Feeling Lucky
?
?
[#39720] - Update md5 Deprecation
- Fixed in Code Base
- 4 Feb 2023
- Medium
- Build: 4.2-dev
- # 39720
- Diff
- harsitagarwalla187:harsitagarwalla187/issue39701
User tests: Successful: Unsuccessful:
Pull Request for Issue #39701.
Summary of Changes
Updated md5() from passing null to parameter.
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
laoneo
- comment
- 31 Jan 2023
I have tested this item
Tested with null as secret in configuration. Not sure how this can happen, but the fix works.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39720.
Quy
- comment
- 31 Jan 2023
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39720.
| Status | Pending | ⇒ | Ready to Commit |
Quy
- comment
- 31 Jan 2023
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39720.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-02-04 09:57:05 |
| Closed_By | ⇒ | fancyFranci |
fancyFranci
- comment
- 4 Feb 2023
Thank you!
The second argument of md5 is a boolean and not a string. Why do you add a string here?