Feeling Lucky
?
[#17566] - Correct the usage of getInt in two places
- Fixed in Code Base
- 20 Aug 2017
- Medium
- Build: staging
- # 17566
- Diff
- zero-24:getInt
- Pending hound Hound is busy reviewing changes... Details
User tests: Successful: Unsuccessful:
Summary of Changes
Correct the usage of getInt in two places
Testing Instructions
review
Expected result
getInt just accepts two parameters see: https://api.joomla.org/cms-3/classes/JInput.html#method_getInt
Actual result
getInt is called with three parameters
Documentation Changes Required
none
Reporter to the JSST
joomla-cms-bot
- | Category | ⇒ | Front End com_content com_users |
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
sandewt
- comment
- 20 Aug 2017
I would also like to test. Do you have instructions how? (I have installed the patchtester.)
wilsonge
- comment
- 20 Aug 2017
Check that editing your user profile still works in the frontend, and that when editing an article in the frontend you are still correctly redirected to the correct place :)
sandewt
- comment
- 20 Aug 2017
I have tested this item
Tested in Joomla! 3.8.0-beta3-dev.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17566.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-20 11:55:09 |
| Closed_By | ⇒ | wilsonge |
wilsonge
- comment
- 20 Aug 2017
Thanks guys :)
I have tested this item✅ successfully on f1fc93a
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17566.