Just enquiring why a small bit of code in admin/modules.php was changed

Open to all! Ask other shopowners for help.
14Steve14
Senior Contributor
Posts: 942
Joined: Fri Oct 25, 2019 7:01 pm
Phoenix Version: v1.0.9.1
Has thanked: 17 times
Been thanked: 111 times

Just enquiring why a small bit of code in admin/modules.php was changed

Post by 14Steve14 »

This is probably one for @burt or @ecartz

I am updating my current site to the latest 1.1.0.7, not yet updated to 1.1.0.8, version, and in testing we noticed that on the modules page in admin when viewing the content section the dropdown list was missing. Just wondering if this was intentional as there was a problem with the code, or whether it was just an oversight.

The bit of code in question is around lines 126-126 and originally is

Code: Select all

<?php
  $cfgm = "uncallable_cfgm_$set";
  if (is_callable([$cfgm, 'menu'])) {
?>
When a simple change back to the same as was in the older versions the dropdown selection list shows

Code: Select all

<?php
  $cfgm = "cfgm_$set";
  if (is_callable([$cfgm, 'menu'])) {
?>



Join The Code Co-op to get access to your library in the Code Co-op Forum
User avatar
BrockleyJohn
Certified Developer
Posts: 189
Joined: Mon Mar 01, 2021 5:37 pm
Phoenix Version:
Has thanked: 2 times
Been thanked: 36 times

Re: Just enquiring why a small bit of code in admin/modules.php was changed

Post by BrockleyJohn »

I think that change was released in 1.0.9.6
https://github.com/CE-PhoenixCart/Phoen ... 60488b6ff9

User avatar
burt
Core Team
Posts: 4736
Joined: Tue Oct 29, 2019 9:37 am
Phoenix Version: v1.1.0.8
Has thanked: 279 times
Been thanked: 467 times

Re: Just enquiring why a small bit of code in admin/modules.php was changed

Post by burt »

Intentional (as a quick sticking plaster) to remove a menu which was causing issues.
I am not here to build for you.
I am here to build with you. Let's help each other.



Join The Code Co-op to get access to your library in the Code Co-op Forum
Post Reply