lat9 / ZCA-Bootstrap-Template

A Bootstrap-4 template for Zen Cart versions 1.5.8 through 2.1.0. Forked from https://github.com/zcadditions/ZCA-Bootstrap-Template-for-1.5.6-v2.0.0c. See the demo site, below, for additional information.
https://zc158.vinosdefrutastropicales.com/zc158/index.php?main_page=index
GNU General Public License v3.0
4 stars 14 forks source link

ZCA Bootstrap Colors configuration_group_id not passed to admin_pages #49

Closed dbltoe closed 2 years ago

dbltoe commented 3 years ago

Found with mod_list on all the latest 1.5.7c installs. Mod list reports a "Missing Admin Configuration Page" for ZCA Bootstrap Colors on a standard 1.5.7c. The configuration_group_id is created in the configuration_group table and the page_key value of toolsCZABootstrapColors is created in the admin_pages table but, the gID=## page_params is not propagated using the configuration_group_id. Not sure that it affects actual operations but it does stop mod_list from accessing the admin page.

drbyte commented 3 years ago

It breaks pattern somewhat, insomuch as the actual editor for that group isn't on the configuration menu, but rather under the Tools menu (as denoted by the page_key prefix, and the menu_key).

lat9 commented 3 years ago

Might be worth mentioning on the template's Zen Cart support thread (or in the one for Mod List, or both)?

drbyte commented 3 years ago

Or update the mod_list mod to check such anomalies against other menu_key categories where gID= is set.

lat9 commented 3 years ago

Or update the mod_list mod to check such anomalies against other menu_key categories where gID= is set.

That would be a 'good thing', since I use this construct in a couple of customizations.