[#41986] - [5.3] simplify class usage
- Pending
- Medium
- Build: 5.0-dev
- # 41986
- Diff
- Denitz:5.0_Simplify_class_usage
- Pending continuous-integration/drone/pr Build is pending Details
User tests: Successful: Unsuccessful:
Summary of Changes
Prefer Classname::class over class string usage to simplify the class usage find in IDE.
Prefer static::class over get_called_class() for simplicity.
Testing Instructions
Apply patch.
Actual result BEFORE applying this Pull Request
See complex code, a class can't be missed in usage find via IDE.
Expected result AFTER applying this Pull Request
Simple code, class usage is complete in IDE.
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
joomla-cms-bot
- | Category | ⇒ | Administration com_installer Libraries Modules Front End Plugins |
| Status | New | ⇒ | Pending |
| Labels |
Added:
PR-5.0-dev
|
||
| Title |
|
||||||
| Labels |
Added:
Feature
|
||
This pull request has been automatically rebased to 5.2-dev.
| Title |
|
||||||
| Labels |
Added:
bug
PR-5.1-dev
PR-5.2-dev
Removed: Feature |
||
| Category | Administration com_installer Libraries Modules Front End Plugins | ⇒ | Administration com_installer Libraries Front End Plugins |
I'm sorry that this didn't go into 5.2.0. Since this is not really a bugfix, but a code-refactoring, I'm going to push this over to 5.3, but I'm going to ask for people to finally test this, so that we can merge it then.
| Title |
|
||||||
This pull request has been automatically rebased to 5.1-dev.