[#45039] - [5.3] Remove useless empty ($form) check
- Pending
- Medium
- Build: 5.3-dev
- # 45039
- Diff
- joomdonation:remove_empty_form_check
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
The loadForm method from FormBehaviorTrait (which is used by our models) always return a Form object on success and throw Exception on failure, so all empty($form) check in our model classes are useless, it never returns false, thus should be moved from models code.
Testing Instructions
This will require careful code review from our testers and maintainer.
Actual result BEFORE applying this Pull Request
Works
Expected result AFTER applying this Pull Request
Works, slight faster because the useless code is removed, thus not being executed.
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_banners com_categories com_config com_contact com_content com_fields com_finder com_installer com_languages com_media com_menus com_messages com_modules com_newsfeeds com_plugins com_redirect |
| Title |
|
||||||
| Labels |
Added:
PR-5.3-dev
|
||