We use cookies to improve your experience. No personal information is gathered and we don't serve ads. Cookies Policy.

ExpressionEngine Logo ExpressionEngine
Features Pricing Support Find A Developer
Partners Upgrades
Blog Add-Ons Learn
Docs Forums University
Log In or Sign Up
Log In Sign Up
ExpressionEngine Logo
Features Pro new Support Find A Developer
Partners Upgrades
Blog Add-Ons Learn
Docs Forums University Blog
  • Home
  • Forums

Combo Load - Plugin Low Variables - 3.3.1 Errors

Development and Programming

motoracer's avatar
motoracer
8 posts
about 5 years ago
motoracer's avatar motoracer

Hi, When using the Low Variables plugin, an error appears in the console:

<div class=”err-wrap notice”> <h1>Notice</h1>

A non well formed numeric value encountered

ee/legacy/libraries/Javascript_loader.php, line 148 <ul> <li>Severity: E_NOTICE</li> </ul> </div>

Badly url - admin.php?S=0&D=cp&C=javascript&M=combo_load&plugin=ui.touch.punch&package=low_variables:low_variables&v=3.3.1&v=1548331170

The error appears from this line - https://github.com/ExpressionEngine/ExpressionEngine/blob/5.1.3/system/ee/legacy/libraries/Javascript_loader.php#L148

The error appeared suddenly. Has anyone encountered such a thing?

I’m using version EE 5.1.3 Low Variables - 3.3.1

       
Rob Allen's avatar
Rob Allen
2,950 posts
about 5 years ago
Rob Allen's avatar Rob Allen

Have you upgraded EE or Low variables recently to make this happen?

       
motoracer's avatar
motoracer
8 posts
about 5 years ago
motoracer's avatar motoracer

I did update Low Variables but before this was also this error.

       
Tom Jaeger's avatar
Tom Jaeger
228 posts
about 5 years ago
Tom Jaeger's avatar Tom Jaeger

If possible I would recommend making sure both EE and Low Variables are upgraded to the most recent versions, and test again.

       
motoracer's avatar
motoracer
8 posts
about 5 years ago
motoracer's avatar motoracer

Unfortunately, I receive an error at the end of the update.

When will I remove from url

admin.php?S=0&D=cp&C=javascript&M=combo_load&plugin=ui.touch.punch&package=low_variables:low_variables&v=3.3.1&v=1548331170

admin.php?S=0&D=cp&C=javascript&M=combo_load&plugin=ui.touch.punch&package=low_variables:low_variables&v=1548331170

&v=3.3.1 - the php error disappears

       
Tom Jaeger's avatar
Tom Jaeger
228 posts
about 5 years ago
Tom Jaeger's avatar Tom Jaeger

We dug into this one a bit more in-depth. It looks like both low variables and EE are using the v GET variable. On the EE side we opened a ticket to add additional validation that it is a timestamp (and get it from the server accordingly if it isn’t).

I’ve also reached out to Low and asked him to consider renaming the his GET variable.

       
motoracer's avatar
motoracer
8 posts
about 5 years ago
motoracer's avatar motoracer

Problem solved. Errors disappeared after adding the code.

https://github.com/intoeetive/ExpressionEngine/commit/9a3aa941ea9a3238ae9edc0f097f3ff4dd807733

       

Reply

Sign In To Reply

ExpressionEngine Home Features Pro Contact Version Support
Learn Docs University Forums
Resources Support Add-Ons Partners Blog
Privacy Terms Trademark Use License

Packet Tide owns and develops ExpressionEngine. © Packet Tide, All Rights Reserved.