Re: Dynamic Price Updater
Quote:
Originally Posted by
rainestudios
I'm still getting this error in my logs constantly:
It doesn't seem to affect the functionality, everything is displaying fine in my shop, but this can't be good, right?
thanks,
amy
Quote:
Originally Posted by
Chrome
Hi
@amy: You're quite right, that shouldn't be... I'm quite surprised it's actually still working :blink:... I'll check out your site see if I can see any weird data getting thrown about
Cheers
Dan
Hi Dan,
Just checked my error logs today and it seems I am getting the exact same error that Amy was getting in her quoted post above and your reply in which you replied with the above.
Like Amy everything seems to be working fine apart from this error appearing in the error log which is as follows:
Code:
[17-Jul-2011 11:48:27] PHP Warning: reset() [<a href='function.reset'>function.reset</a>]: Passed variable is not an array or object in /home/my_site/public_html/includes/classes/dynamic_price_updater.php on line 194
[17-Jul-2011 11:48:27] PHP Warning: Variable passed to each() is not an array or object in /home/my_site/public_html/includes/classes/dynamic_price_updater.php on line 195
Now on lines 194 and 195 it states the following.
Code:
reset($this->_shoppingCart->contents[$this->_post['products_id']]['attributes']);
while (list($option, $value) = each($this->_shoppingCart->contents[$this->_post['products_id']]['attributes'])) {
So I am wondering if by any chance you have had the time to figure out what is causing this error and how to fix it?
Thanks.
PS: Still haven't had time to check the tax fix yet but haven't forgotten about it and will look into it very soon :).
Re: Dynamic Price Updater
Hi Dan,
Do you know if DPU would be affected by a Javascript loader and in particular this version of the loader?
https://github.com/yellow1912/cjload...files/includes
As I was getting the errors mentioned in my previous post every minute in the error directory after upgrading to this version of the loader. I had to upgrade to this version of the loader for the Tab Products Pro mod to work as I was getting errors posted in the error directory relating to this mod and I tracked it down to being caused by an earlier version of this Javascript loader. Then when I upgraded the Javascritp loader the errors for the Tab Products Pro mod went away. But it caused the errors to increase for DPU.
Here is some more information on this Javascript loader.
http://wiki.rubikintegration.com/zen...t_loader/about
So for the time being unfortuantly I had to disable DPU until I can figure out how I can get it to work along side Tab Products Pro and this new version of the Javascript loader.
If you any suggestions or ideas I would greatly appreciate hearing them :).
Thanks.
Re: Dynamic Price Updater
Hi CybaGirl
Do you have a URL where I can see the conflicting module in action? I should be able to work out a switching method from that
Cheers
Dan
Re: Dynamic Price Updater
Hi Dan and thanks for getting back to me.
Sorry to read about all this trouble though. I hope something can be worked out and this thread along with DPU is not deleted as I believe it to be a most worthwhile addition to ZenCart and at the very least I would like to say thank you for creating it and giving up what you have in supporting it so that others can use it.
Please give me a day or two to re-enable the mod or install it on my test cart as I have a couple of things I am working on at the moment. So I will get back to you shortly.
Thank you again!
Re: Dynamic Price Updater
Moderator note: all threads in the last 24 hours related to where this module lives have been deleted.
Fresh start:
Chrome, you are a valued member of the community.
The community has rules. Everyone is subject to them.
One of the rules is that contributions be submitted to the Free Software Add Ons area with a GPL license.
People are allowed to attach zip files to posts, but these are intended to be temporary (to permit beta testing and so forth), not long lasting.
As the moderator of that area, one of the things I do is reject changes that third parties make to mods which are still being actively supported. So when you submit a mod, as long as you support your mod, you will be the only one allowed to change it.
Please go ahead and submit your mod using the "Submit your file" link at the top right of
http://www.zen-cart.com/index.php?ma...index&cPath=40
Once that's done, the zip will be removed from the first entry of this thread and replaced with a link to the Free Software Add Ons area.
No hard feelings. Let's all work together for the good of the community.
Re: Dynamic Price Updater
Hi CybaGirl
I apologise, because of the other conversation I missed your reply. Thank you for your support :smile:. Please, test in your own time, I really appreciate the feedback; it's because of a lack of time that I'm unable to create these test environments (I tried taking the laptop to bed to see if I could sleep-code but... Well, let's just not speak of it)
As soon as you have time please let me know your findings and I'll get on it
Dan
Re: Dynamic Price Updater
Dan has agreed to submit his excellent work to the Free Software Add Ons as soon as possible.
Thank you, Dan!
Re: Dynamic Price Updater
Hi
Thanks! :smile: I'll get it uploaded as soon as I alter some text
I'm going to try and fix the outstanding problems this week; they shouldn't exist
Cheers!
Dan
Re: Dynamic Price Updater
PHEW!!!
Glad all this trouble is now sorted out :).
Hi Dan, thanks for your reply and no need to apologise ever! I am sure you have a life outside of watching this thread and waiting for peoples comments about DPU :).
Ok I have enabled DPU in the test store and I have doubled checked the error directory to see if the errors returned and they have. Here is what the errors are saying for your reference.
Code:
[25-Jul-2011 14:29:33] PHP Warning: reset() [<a href='function.reset'>function.reset</a>]: Passed variable is not an array or object in /home/my_site/public_html/shop/includes/classes/dynamic_price_updater.php on line 194
[25-Jul-2011 14:29:33] PHP Warning: Variable passed to each() is not an array or object in /home/my_site/public_html/shop/includes/classes/dynamic_price_updater.php on line 195
I will send you a PM with the address so you can access this store.
Thank you once again for your continued support of DPU and I am glad to see that the thread and DPU is still intact :).
Re: Dynamic Price Updater
I too am glad to see this is all worked out.. (I SILENTLY watch this thread :laugh::laugh:)
I like this mod too and would hate to see Dan and his work go away..