[Tomasz Frydrych]

Hello support,

I upgraded PHP on my server to 8.1 and I found a problem with merging top and main menu functionality on your theme. To display menu sprintf function from PHP is used. This method takes string format as first argument and arguments to be placed in string as following arguments. For single menu the string format is as follows:

<ul id="%1$s" class="%2$s">%3$s</ul>

and it works correctly. If you select an option to merge top and main menu on mobile devices then the string format is extended by adding top menu html definition at the end. In my case I have there button to logout with some url containing url escaped characters (like %3D) and because of that sprintf causes fatal error with one of following messages:

– Uncaught ArgumentCountError: 7 arguments are required, 4 given

– Uncaught ValueError: Unknown format specifier “D”

Can you please fix the issue? Temporarly I disabled this merging functionality but it is only an workaround.

 

Thank you in advance

Tomasz

[vako]

Hi Tomasz,

 

At first, thank you for choosing our theme and service.

We assigned this thread to our development team, they will check the mentioned case and reply shortly.

Kind Regards

[George]

Hello,

Can you please give us temporary access to your site, better if it be staging and recreate the issue so I debug/fix it there directly ?

Don’t forget to set it as private reply.

Thanks in advance.

Kind regars,
George

[Tomasz Frydrych]

This reply has been marked as private.

[George]

Hello,

Sorry for this very late response, can you reopen access to test site ?

Kind regards,
George

[Tomasz Frydrych]

This reply has been marked as private.

[Author]

Posts