Feeling Lucky
?
?
[#19710] - [libraries][legacy][request] - fix php 7.1 warning not numeric
- Fixed in Code Base
- 25 Mar 2018
- Medium
- Build: staging
- # 19710
- Diff
- alikon:patch-103
User tests: Successful: Unsuccessful:
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
rotech1
- comment
- 16 Feb 2018
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19710.
Quy
- comment
- 23 Feb 2018
I couldn't reproduce the warnings without this PR.
How about adding the following code at the top of the method so you don't have to cast it 3 times?
$mask = (int) $mask;
| Labels |
Added:
?
|
||
alikon
- comment
- 23 Feb 2018
to reproduce the warnings #19695 (comment)
thanks for the dry tip
Quy
- comment
- 27 Feb 2018
I have tested this item
Code review
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19710.
Quy
- comment
- 27 Feb 2018
I have tested this item
Code review
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19710.
rotech1
- comment
- 20 Mar 2018
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19710.
| Status | Pending | ⇒ | Ready to Commit |
Quy
- comment
- 20 Mar 2018
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19710.
mbabker
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-03-25 14:53:52 |
| Closed_By | ⇒ | mbabker | |
| Labels |
Added:
?
|
||
@rotech1 Please do the following to mark your test as successful:
Test thisbutton above author's photoTested successfullySubmit test result