Quick PHP 8 fix for MenuManager please!

[Charles Butcher]

I would be very grateful if someone could tell me how to patch the MenuManager module to work with PHP 8. The "each" at line 111:

Code: Select all

while (list($key) = each($children)) {

breaks the site. I tried:

Code: Select all

foreach( $children as $key ) {

without success. I know I should probably be using Navigator instead, but I'm not a developer and until I can get my head round that I'd like to keep it working with MenuManager and PHP 8 if possible.

Any hints appreciated.

[MantaPro]

Try

Code: Select all

 foreach ($children as $key => $value)

In the old code the each returns the key & value pairs. The list assigns the key but ignores the value.

In your attempted code you were looping through the children and assigning the value to a field called key. Whereas you want the field called key to hold the key.

Good luck

[Charles Butcher]

Thank you very much indeed! That works. Now the errors have moved to old modules like CGGoogleMaps – but it's progress :-)

[DIGI3]

Just a reminder that source code changes are not permitted on the forum (there's pinned posts outlining the reasoning so I won't get into it here). Also keep in mind MenuManager was deprecated years ago, and will stop being included very soon. Migrating to Navigator is the best option.

[Charles Butcher]

Thanks DIGI3, and apologies for breaking that rule – which makes perfect sense. I'll get to grips with Navigator. I inherited the original MenuManager template in 2009 as part of a complete site design, so I have never needed to understand it. This will be a useful learning experience.