Feeling Lucky
?
[#18334] - [3.9] remove mcrypt check
- Fixed in Code Base
- 2 Nov 2017
- Medium
- Build: 3.9-dev
- # 18334
- Diff
- brianteeman:3.9-mcrypt
User tests: Successful: Unsuccessful:
We were checking and reporting on the presence of mcrypt. We no longer need to do that as it is no longer a requirement and mcrypt itself is not present in php 7.2 so the checks would fail anyway then.
Didnt see the point in wrapping it in a conditional php version check as we dont need the info any more
See #18169 for the other mcrypt removal
joomla-cms-bot
- | Category | ⇒ | Administration com_joomlaupdate Language & Strings |
| Status | New | ⇒ | Pending |
zero-24
- comment
- 14 Oct 2017
Quy
- comment
- 17 Oct 2017
I have tested this item
Code review
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18334.
franz-wohlkoenig
- comment
- 26 Oct 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18334.
| Status | Pending | ⇒ | Ready to Commit |
franz-wohlkoenig
- comment
- 26 Oct 2017
RTC after two successful tests.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-11-02 23:20:14 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
brianteeman
- comment
- 2 Nov 2017
Thanks
wilsonge
- comment
- 2 Nov 2017
Thanks!