Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use $ismoo instend of calling jinput #14047

Merged
merged 4 commits into from
Feb 13, 2017
Merged

Conversation

zero-24
Copy link
Contributor

@zero-24 zero-24 commented Feb 13, 2017

Pull Request for Issue #13306

Summary of Changes

  • Use $ismoo instend of calling jinput
  • a bit CS

Testing Instructions

  1. Open an administrator form that has a User form field on it (i.e. User Notes)
  2. Type a user name or keyword into the search box and submit
  3. Click one of the resulting users; the dialog is dismissed

Expected result

The user selected in the pop-up should populate the user form field.

Actual result

In 3.6.5 it is broken as the $ismoo value is missing.
In 3.7-dev it is working but don't uses the $ismoo it gets the value from the jinput object

Documentation Changes Required

None

@joomdonation
Copy link
Contributor

I have tested this item ✅ successfully on abccdaf


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14047.

@joomdonation
Copy link
Contributor

joomdonation commented Feb 13, 2017

Since you made this change, could you please this line https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_users/views/users/tmpl/modal.php#L23 to

$input = JFactory::getApplication()->input;

and remove the line above https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_users/views/users/tmpl/modal.php#L22

(The $app variable is used one time only, thus it should be removed)

@zero-24
Copy link
Contributor Author

zero-24 commented Feb 13, 2017

Done thanks @joomdonation

@joomdonation
Copy link
Contributor

I have tested this item ✅ successfully on 29dab64


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14047.

1 similar comment
@dgrammatiko
Copy link
Contributor

I have tested this item ✅ successfully on 29dab64


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14047.

@dgrammatiko
Copy link
Contributor

RTC

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 13, 2017
@zero-24 zero-24 added this to the Joomla 3.7.0 milestone Feb 13, 2017
@wilsonge wilsonge merged commit 8e81979 into joomla:staging Feb 13, 2017
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Feb 13, 2017
@zero-24 zero-24 deleted the zero-24-patch-3 branch February 13, 2017 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants