[#44875] - [5.3] Use str_contains function for components
- Fixed in Code Base
- 16 Feb 2025
- Medium
- Build: 5.3-dev
- # 44875
- Diff
- joomdonation:use_str_contains_for_components
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
This PR uses StrContainsRector rector rule to convert code of our components to use str_contains function. 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, easier to read code
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_categories com_config com_content com_cpanel com_installer com_joomlaupdate com_languages com_login com_menus com_postinstall com_templates |
I have tested this item ✅ successfully on 8a65582
Carefully reviewed every single change.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44875.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
PR-5.3-dev
|
||
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44875.
| Labels |
Added:
RTC
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-02-16 00:11:26 |
| Closed_By | ⇒ | LadySolveig |
Thank you @joomdonation and many thanks also to the diligent testers @QuyTon @richard67 💚 💛 💙 ❤️
I have tested this item ✅ successfully on 8a65582
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44875.