Feeling Lucky
PR-5.2-dev
[#43512] - [5.2] Remove Toolbar::getInstance() from views
- Pending
- Medium
- Build: 5.2-dev
- # 43512
- Diff
- Hackwar:5.2-toolbar
- Pending continuous-integration/drone/pr Build is pending Details
User tests: Successful: Unsuccessful:
Summary of Changes
Toolbar::getInstance() has been deprecated for quite some time now, but the alternative wasn't really clear yet. This PR replaces all occurences with $this->getDocument()->getToolbar() instead. This leaves the Toolbar::getInstance() in the ToolbarHelper class.
Testing Instructions
Codereview
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_admin com_associations com_banners com_cache com_categories com_checkin com_config com_contact com_content com_cpanel com_fields com_finder |
brianteeman
- comment
- 22 May 2024
What the issue @brianteeman ?
- obvious errors due to blindly search/replace (picked up by drone)
- stating that no documentation is required.
| Labels |
Added:
PR-5.2-dev
|
||
What the issue @brianteeman ?