Feeling Lucky
?
bug
PR-4.3-dev
[#40133] - Remove useless strpos() checking
- Fixed in Code Base
- 4 May 2023
- Medium
- Build: 4.3-dev
- # 40133
- Diff
- Fedik:rm-strpos-ev
User tests: Successful: Unsuccessful:
Summary of Changes
The patch removes useless strpos() check.
This event is set up in AssetRegistry by class name already. Also it is not a system wide event, and never should be.
Testing Instructions
Apply patch, and check that the site works as before.
Actual result BEFORE applying this Pull Request
Works.
Expected result AFTER applying this Pull Request
Works.
Link to documentations
Please select:
- No documentation changes for docs.joomla.org needed
- No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Status | New | ⇒ | Pending |
Fedik
- comment
- 16 Mar 2023
done
| Labels |
Added:
PR-4.3-dev
|
||
toivo
- comment
- 16 Mar 2023
I have tested this item
Tested successfully in Joomla 4.3.0-beta5-dev of 16 March using PHP 8.1.10
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40133.
joomdonation
- comment
- 17 Mar 2023
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40133.
| Status | Pending | ⇒ | Ready to Commit |
joomdonation
- comment
- 17 Mar 2023
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40133.
| Labels |
Added:
?
bug
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-05-04 19:25:02 |
| Closed_By | ⇒ | obuisard |
Tested and nothing is broken. But please remove the unused use statement