[#48050] - [7.0] Convert getError() in controllers to exceptions
- Pending
- Medium
- Build: 7.0-dev
- # 48050
- Diff
- Hackwar:7.0-exceptions
User tests: Successful: Unsuccessful:
- I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.
Summary of Changes
This converts all calls to getError() in controllers to instead use exceptions.
Testing Instructions
Codereview
Link to documentations
Please select:
-
Documentation link for guide.joomla.org:
-
No documentation changes for guide.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_banners com_config com_contact com_content com_contenthistory com_finder com_installer com_languages com_menus com_messages com_redirect com_templates com_users com_workflow |
No, those are the helpers to allow for transitioning from LegacyErrorHandlingTrait to Exceptions and they will only be removed in 8.0. They are mandatory to convert to exceptions now.
No, those are the helpers to allow for transitioning from LegacyErrorHandlingTrait to Exceptions and they will only be removed in 8.0. They are mandatory to convert to exceptions now.
Then you should adapt the phpstan baseline file in this PR because as you can see, phpstan fails for this PR.
@Hackwar Phpstan complains about the „setUseException“ calls. Don’t we want to remove them in 7.0?