Feeling Lucky
RTC
PR-5.3-dev
[#44973] - [5.3] Change list() to array destruct for libraries code
- Fixed in Code Base
- 26 Feb 2025
- Medium
- Build: 5.3-dev
- # 44973
- Diff
- joomdonation:use-list-to-array-destruct-for-libraries
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
This PR uses ListToArrayDestructRector rector rule to change code form using list function to user more modern array destruct syntax in our libraries code. This is done automatically by rector, no manual change included.
Testing Instructions
Code review
Actual result BEFORE applying this Pull Request
Works
Expected result AFTER applying this Pull Request
Works, with cleaner code using modern syntax
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Labels |
Added:
PR-5.3-dev
|
||
viocassel
- comment
- 24 Feb 2025
I have tested this item ✅ successfully on be28f27
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44973.
| Status | Pending | ⇒ | Ready to Commit |
QuyTon
- comment
- 24 Feb 2025
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44973.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-02-26 18:15:15 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
RTC
|
||
HLeithner
- comment
- 26 Feb 2025
thanks
I have tested this item ✅ successfully on be28f27
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44973.