User tests: Successful: Unsuccessful:
Added missing $user
variable declaration in administrator/modules/mod_submenu/tmpl/default.php
administrator/components/com_content/presets/content.xml
quicktask-permission
attribute, i.e. change it to read:<menuitem
title="COM_CONTENT_MENUS_ARTICLE_MANAGER"
type="component"
element="com_content"
link="index.php?option=com_content&view=articles"
quicktask="index.php?option=com_content&task=article.add"
quicktask-title="COM_CONTENT_MENUS_NEW_ARTICLE"
quicktask-permission="core.create;com_content"
/>
/administrator/index.php?option=com_cpanel&view=cpanel&dashboard=content
)The page breaks with an error: “Call to a member function authorise() on null”
The page works.
None
This issue was discovered while trying to set up a dashboard with a custom preset for my own component. That is to say, it affects all core and third party extensions since it's a very obvious omission when the view template was copied from mod_menu to mod_submenu and subsequently refactored.
Also note that the static code analysis in PhpStorm also caught the problem when I opened the affected file.
Status | New | ⇒ | Pending |
Category | ⇒ | Modules Administration |
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-09-14 12:41:18 |
Closed_By | ⇒ | wilsonge | |
Labels |
Added:
?
|
Thank you! I'm really hoping to move to Dashboards over the next few months.
Speaking of which, I have noticed a few things which can be improved in the InstallationScript to facilitate 3PDs using dashboards. Should I open an issue about my ideas to discuss it?
I have noticed a few things which can be improved in the InstallationScript to facilitate 3PDs using dashboards. Should I open an issue about my ideas to discuss it?
Please do. Not suprised it can be improved as it was an afterthought
Merged on review. Thanks!