Re: Dynamic Price Updater
Dan, Matt,
To correct the issue with Lightbox (assuming that is the issue) I modified the jscript_updater.php as follows:
There are two places in the code that check if: (prArr[i] !== null)
I changed these two instances to: (prArr[i] == '')
This seems to work. I have not fully tested all use cases, but so far so good.
Dan, do you foresee any issues with using (prArr[i]=='') in place of (prArr[i]!==null)?
Thanks,
Clark
Re: Dynamic Price Updater
The site:
Edited: PM sent
I wonder if this has anything to do with lightbox.
Re: Dynamic Price Updater
Hi
Clark, time will tell on that... The Updater checks for a explicit NULL value and Lightbox could be affecting that... I'm genuinely not sure how (I thought it may be an event registration problem)... Please keep me updated as to how it goes; it may well work for other people
Feedback would be much appreciated :smile:
Yellow, I'm going to have a look at the site now... I'll let you know if I find anything... You might want to try Clark's solution to see if it works for you
News as I have it
Cheers
Dan
Re: Dynamic Price Updater
I tried the changes suggested by Clark but it doesnt seem to fix the error. I turned off lightbox for now, Price Updater is still not working however. But it may make things easier to debug.
Re: Dynamic Price Updater
Quote:
Originally Posted by
clarkjarvis
Dan, Matt,
To correct the issue with Lightbox (assuming that is the issue) I modified the jscript_updater.php as follows:
There are two places in the code that check if: (prArr[i] !== null)
I changed these two instances to: (prArr[i] == '')
This seems to work. I have not fully tested all use cases, but so far so good.
Dan, do you foresee any issues with using (prArr[i]=='') in place of (prArr[i]!==null)?
Thanks,
Clark
YES!!!!!! Thank you soooo much Clark!! That did it and now Lightbox and Updater are playing nicely together!:clap::clap::clap:
Matt
Re: Dynamic Price Updater
Nice one Clark! Looks like you may have fixed that one :smile:... I have no idea at all why that is :lol:
Yellow, I do have one idea for the moment... It may not work, but try finding the line that says
var seeker = new RegExp(/\(([+-]?)([^0-9]*)([0-9,]+\.[0-9]+)([^0-9)]*)\)/);
and change it to:
var seeker = new RegExp(/\\(\s*([+-]?)([^0-9]*)([0-9,]+\.[0-9]+)([^0-9)]*)\s*\)/);
Let me know what happens
Dan
Re: Dynamic Price Updater
Chrome: I applied your change your change and it price doesnt get updated anymore.
So I reverted it back, re-apply Clark's fix and it's partially working right now. Say if you add an attribute that costs 30 you will get:
Your price: +$30.00 (1)
Weird.
Re: Dynamic Price Updater
Hi Yellow
I think I might have had an idea... This does mean altering your attributes slightly, but try removing the brackets from around the text in the Extra Fabric attribute
Sorry this is being such a pain
Dan
Re: Dynamic Price Updater
No, I must thank you for taking your precious time to help me here.
"try removing the brackets from around the text in the Extra Fabric attribute"
If the brackets are causing the error, wouldn't the one for Custom Topper, which does not have the brackets should work fine?
BTW, you can visit the store before upgrading to 1.3.8 and adding several features by removing the "testsite" part from the link and you will see the Price Updater is working nicely.
The only possible problem I can think of now is the Lightbox.
Re: Dynamic Price Updater
One of the changes I made was to remove some sideboxes, and on the PU debug I saw:
createSB: Sidebox could not be created!
Can this be a problem?