[#44938] - [5.3] Use str_contains function for modules plugins
- Fixed in Code Base
- 22 Feb 2025
- Medium
- Build: 5.3-dev
- # 44938
- Diff
- joomdonation:use_str_contains_for_modules_plugins
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
This PR uses StrContainsRector rule to convert our modules and plugins code to use str_contains function. It does not change any existing behavior, just make the code cleaner and easier to read. The change is done automatically by rector, no manual change included here.
Testing Instructions
Need to have code review for every single change here.
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 | ⇒ | Modules Front End Plugins |
I have tested this item ✅ successfully on 5e552d8
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44938.
| 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/44938.
| Labels |
Added:
RTC
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-02-22 09:55:01 |
| Closed_By | ⇒ | HLeithner |
Thanks
I have tested this item ✅ successfully on 5e552d8
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44938.