[#44893] - [5.3] Change list to array destruct for components code
- Fixed in Code Base
- 15 Feb 2025
- Medium
- Build: 5.3-dev
- # 44893
- Diff
- joomdonation:use-list-to-array-destruct-rector
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 components 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 | ⇒ | Administration com_associations com_banners com_contact com_content com_fields com_media com_menus com_newsfeeds com_tags com_workflow Front End |
| Labels |
Added:
PR-5.3-dev
|
||
| Title |
|
||||||
| Title |
|
||||||
I have tested this item โ successfully on 95b797e
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44893.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44893.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-02-15 23:42:08 |
| Closed_By | ⇒ | LadySolveig | |
| Labels |
Added:
RTC
|
||
Thank you @joomdonation and also for your reviews @alikon and @QuyTon ๐
I have tested this item โ successfully on 95b797e
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44893.