MenuManager

A plugin to enable management of menus within October CMS

Back to MenuManager Support

UpTop
UpTop

The reserved indicator "@" cannot start a plain scalar; you need to quote the scalar at line 9 (near "form: @/plugins/benfreke/menumanager/models/menu/fields.yaml").

Stable release breaks the controller. Can you fix this please?

RaTT
RaTT

To fix it you must just modify files config_form.yaml and config_list.yaml found in plugins/benfreke/menumanager/controllers/menus

The current values are:

config_form.yaml: @/plugins/benfreke/menumanager/models/menu/fields.yaml
config_list.yaml: @/plugins/benfreke/menumanager/models/menu/columns.yaml

The correct values are:

config_form.yaml: $/benfreke/menumanager/models/menu/fields.yaml
config_list.yaml: $/benfreke/menumanager/models/menu/columns.yaml

Also you must modify file Menus.php found in plugins/benfreke/menumanager/controllers

The current value is:

$toolbarConfig->buttons = '@/plugins/benfreke/menumanager/controllers/menus/_reorder_toolbar.htm';

The correct value is:

$toolbarConfig->buttons = '$/benfreke/menumanager/controllers/menus/_reorder_toolbar.htm';

Last updated

UpTop
UpTop

Thanks for your reply with a fix for the problem. I was hoping BenFreke could update the plugin. because although your fix works, I don't want to keep fixing it for every website we develop :)

Ludovic
Ludovic

For me, the 13 of august 2016, the values were :

in config_list.yaml : list: @/plugins/benfreke/menumanager/models/menu/columns.yaml to be changed to list: $/benfreke/menumanager/models/menu/columns.yaml

in config_form.yaml form: @/plugins/benfreke/menumanager/models/menu/fields.yaml to be changed to form: $/benfreke/menumanager/models/menu/fields.yaml

  • make the modification recommended by RaTT (see above)

then it works.

benfreke
benfreke

Feel free to submit a pull request with those fixes. Now I know what the fix is it should be fairly simple for me to implement, but updating this plugin isn't very high on my to do list sorry.

benfreke
benfreke

Thanks for the merge requests. If github had of emailed me I would have merged them earlier. Sorry for the delay, I've just spent several hours getting my local environment up and running to fix this. :/

1-6 of 6