Dynamic Price Updater

Quote:

---

Originally Posted by mc12345678

Just tried using the latest version as provided on github. Had store set to not display the cart after adding product to the cart and switched to showing the cart after adding product. In both cases was not able to reproduce the issue with continuing with shopping.

Further please identify what other plugins may be installed and the "path" taken to get to the shopping cart where the continue shopping button is providing a return to address comprised of having the ajax command on the uri.

---

I have ceon seo url also installed, definately related to dpu module and the zen_back_link() url function as it works fine from any other page apart from a product page.
I simply go to a product page, then click to view the shopping cart page (with a product in cart) , then the "continue shopping" button href link is "?act=DPU_Ajax&method=dpu_update"
If i switch off ceon, same issue , url is now : index.php?main_page=index&act=DPU_Ajax&method=dpu_update

Quote:

---

That said, regarding the price displaying as reported at the referenced post, what version of DPU is installed? Has the fix that was suggested related to webchills post been applied either as he provided it or as it was applied to the most recent modifications to the github version of the software?.

---

Tried webchills, no result.
Latest version of DPU, but have to keep reverting as just giving me 0.00

Further to this, can force the error.
includes/modules/pages/product_info/jscript_dynamic_price_update.php

Code:

---

// use the spans to see if there is a discount occuring up in this here house
  var test = thePrice.getElementsByTagName("span");
  var psp = false;
    var pdpt = false;

  for (var a=0,b=test.length; a<b; a++) {
     
  if (test[a].className === "normalprice") {
      pdpt = test[a];

---

Leaving like this on a sale/special/reduced priced product, that is priced by attributes (product price is zero, attributes do all the work) gives:
Price 0.00 Price 123
Price 0.00 Price 234
Price 0.00 Price 456

Code:

---

<h2 id="productPrices" class="productGeneral">&lt;font size="0.5" color="#636363"&gt;Price: &lt;/font&gt;<font size="0.5" color="#636363">Price: </font><span class="normalprice">£0.00</span>&nbsp;<span class="productSpecialPrice"><font size="0.5" color="#636363">Price: </font>£726.75</span><span class="productPriceDiscount"><br>Save:&nbsp;25% off</span>
 </h2>

---

Comment out the

Code:

---

pdpt = test[a];

---

And gives the same (lowest priced attribute) starting price for all the different priced attributes

price 999 price 123
price 999 price 234
price 999 price 456


My html on page is as follows:

Code:

---

<h2 id="productPrices" class="productGeneral">
<font size="0.5" color="#636363">Price: </font><span class="normalprice">£969.00 </span>&nbsp;<span class="productSpecialPrice"><font size="0.5" color="#636363">Price: </font>£951.75</span><span class="productPriceDiscount"><br>Save:&nbsp;25% off</span>
 </h2>

---

/languages/english/extra_definitions/dynamic_price_updater.php contents:

Code:

---

<?php
/**
* @package Dynamic Price Updater
* @author Design75
* @version 3.0
* @licence This module is released under the GNU/GPL licence
*/

define('BOX_HEADING_DYNAMIC_PRICE_UPDATER_SIDEBOX', 'Price Breakdown'); // the heading that shows in the Updater sidebox
define('DPU_BASE_PRICE', 'Base price');

define('UPDATER_PREFIX_TEXT', '<font size="0.5" color="#636363">Price: </font>');
define('UPDATER_PREFIX_TEXT_STARTING_AT', 'Starting at: ');
define('UPDATER_PREFIX_TEXT_AT_LEAST', 'At least: ');
define('DPU_SHOW_QUANTITY_FRAME', '&nbsp;(%s)');
define('DPU_SIDEBOX_QUANTITY_FRAME', '&nbsp;x&nbsp;%s'); // how the weight is displayed in the sidebox.  Default is ' x 1'... set to '' for no display... %s is the quantity itself
define('DPU_SIDEBOX_PRICE_FRAME', '&nbsp;(%s)'); // how the attribute price is displayed
define('DPU_SIDEBOX_TOTAL_FRAME', '<span class="DPUSideboxTotalText">Total: </span><span class="DPUSideboxTotalDisplay">%s</span>'); // this is how the total should be displayed.  %s is the price itself as displayed in the
define('DPU_SIDEBOX_FRAME', '<span class="DPUSideBoxName">%1$s</span>%3$s%2$s<br />'); // the template for the sidebox display.  Instructions below
/*
 *DPU_SIDEBOX_FRAME has 3 variables you can use... They are:
 * %1$s - The attribute name
 * %2$s - The quantity display
 * %3$s - The individual price display
 * You can position these anywahere around the DPU_SIDEBOX_FRAME string or even remove them to prevent them from displaying
 */

---

Ok, apologies for all the responses, i'm kinda brainstorming aloud!
So it seems, that the comment out of the

pdpt = test[a];

Also has an effect on my other issue on the shopping cart page!
So if i comment out that line to stop the zeros occuring on the product page, then it affects the 'continue button' on the shopping cart page.
If i uncomment that line, all my sale items get messed up, however the 'continue button' then works fine ?!?!

I then changed the small amount of text within the 'UPDATER_PREFIX_TEXT' incase this was causing and issue, but no, the 0.00
s return.

Of note, the dpu loads far far far quicker with the

//pdpt = test[a];

in place.

When uncommented it takes 2 -3 seconds for the prices to load on screen.

Quote:

---

Originally Posted by Calljj

Further to this, can force the error.
includes/modules/pages/product_info/jscript_dynamic_price_update.php

Code:

---

// use the spans to see if there is a discount occuring up in this here house
  var test = thePrice.getElementsByTagName("span");
  var psp = false;
    var pdpt = false;

  for (var a=0,b=test.length; a<b; a++) {
     
  if (test[a].className === "normalprice") {
      pdpt = test[a];
   

---

Leaving like this on a sale/special/reduced priced product, that is priced by attributes (product price is zero, attributes do all the work) gives:
Price 0.00 Price 123
Price 0.00 Price 234
Price 0.00 Price 456

Code:

---

<h2 id="productPrices" class="productGeneral"><font size="0.5" color="#636363">Price: </font><font size="0.5" color="#636363">Price: </font><span class="normalprice">£0.00</span>&nbsp;<span class="productSpecialPrice"><font size="0.5" color="#636363">Price: </font>£726.75</span><span class="productPriceDiscount"><br>Save:&nbsp;25% off</span>
 </h2>

---

Comment out the

Code:

---

pdpt = test[a];

---

And gives the same (lowest priced attribute) starting price for all the different priced attributes

price 999 price 123
price 999 price 234
price 999 price 456


My html on page is as follows:

Code:

---

<h2 id="productPrices" class="productGeneral">
<font size="0.5" color="#636363">Price: </font><span class="normalprice">£969.00 </span>&nbsp;<span class="productSpecialPrice"><font size="0.5" color="#636363">Price: </font>£951.75</span><span class="productPriceDiscount"><br>Save:&nbsp;25% off</span>
 </h2>

---

/languages/english/extra_definitions/dynamic_price_updater.php contents:

Code:

---

<?php
/**
* @package Dynamic Price Updater
* @author Design75
* @version 3.0
* @licence This module is released under the GNU/GPL licence
*/

define('BOX_HEADING_DYNAMIC_PRICE_UPDATER_SIDEBOX', 'Price Breakdown'); // the heading that shows in the Updater sidebox
define('DPU_BASE_PRICE', 'Base price');

define('UPDATER_PREFIX_TEXT', '<font size="0.5" color="#636363">Price: </font>');
define('UPDATER_PREFIX_TEXT_STARTING_AT', 'Starting at: ');
define('UPDATER_PREFIX_TEXT_AT_LEAST', 'At least: ');
define('DPU_SHOW_QUANTITY_FRAME', '&nbsp;(%s)');
define('DPU_SIDEBOX_QUANTITY_FRAME', '&nbsp;x&nbsp;%s'); // how the weight is displayed in the sidebox.  Default is ' x 1'... set to '' for no display... %s is the quantity itself
define('DPU_SIDEBOX_PRICE_FRAME', '&nbsp;(%s)'); // how the attribute price is displayed
define('DPU_SIDEBOX_TOTAL_FRAME', '<span class="DPUSideboxTotalText">Total: </span><span class="DPUSideboxTotalDisplay">%s</span>'); // this is how the total should be displayed.  %s is the price itself as displayed in the
define('DPU_SIDEBOX_FRAME', '<span class="DPUSideBoxName">%1$s</span>%3$s%2$s<br />'); // the template for the sidebox display.  Instructions below
/*
 *DPU_SIDEBOX_FRAME has 3 variables you can use... They are:
 * %1$s - The attribute name
 * %2$s - The quantity display
 * %3$s - The individual price display
 * You can position these anywahere around the DPU_SIDEBOX_FRAME string or even remove them to prevent them from displaying
 */

---

---

I've been thinking about this issue over the last week, trying to figure out how it is possible that with a fully installed series of software that
1) The normalprice would get updated but.
2) That the price presented as an update is 0.00.

Just a few minutes ago, I thought about comparing the above provided code that is currently bolded to see if there was anything obvious considering that at least recently no web page has been presented for independent review (ie. posted with the likes of mysiteDOTcom). And the thing is that I do not see a history along the master path of the software that shows this code section as presented:

Code:

---

// use the spans to see if there is a discount occuring up in this here house
  var test = thePrice.getElementsByTagName("span");
  var psp = false;
    var pdpt = false;

  for (var a=0,b=test.length; a<b; a++) {
     
  if (test[a].className === "normalprice") {
      pdpt = test[a];

---

There is some "key" code missing and if it has been relocated for some reason, that too has not been explained. The code snippet in this area should be:

Code:

---

  // use the spans to see if there is a discount occuring up in this here house
  var test = thePrice.getElementsByTagName("span");
  var psp = false;
  var a;
  var b = test.length;
  var pdpt = false;

  for (a = 0; a < b; a += 1) {
    if (test[a].className === "normalprice") {
      pdpt = test[a];

---

The for loop had been modified 10 months ago as part of 3.2.0 and before that the if statement checking for normalprice didn't exist, so there has been some amount of editing performed. Even within the code, there are other cases of using the variable a and a similar loop performed; however, the loop does not evaluate the normalprice className. If I remember correctly that was on purpose in the event of some sort of communication failure that at least a price remained visible, but I could be wrong there throughout. I would have to look again to see if there really is a reason to not evaluate normalprice in the other situations.

Regardless, there is something that has been modified from the provided files and those changes have not been explained nor presented. As such I would not be surprised that the modifications (or lack of incorporation of changes) have caused the issues observered. I'm open to trying to further investigate but am not seeing the behavior that has been described when the provided software is loaded in full as provided.

just replaced all files from fresh donwload of 3.2
issue is exactly the same.
happy to pm url for you to see.

Quote:

---

Originally Posted by Calljj

just replaced all files from fresh donwload of 3.2
issue is exactly the same.
happy to pm url for you to see.

---

Not fully done with the review, but I have at least found a few things with the applicable site. Appears that the jscript_responsive_framework.php file from includes/templates/template_default/jscript is not placed in the site's template directory. This means that the old XHR style of communication is used instead of the zcJS ajax method that is more built into ZC. I also noticed that there is a javascript error on at least one of the pages. In using Facebook pixel, the price (value) of the product > 999 is pushed to the page without any escape of the comma or quotes around the value. This results in a javascript error of SCRIPT1003: Expected ':' where the content is similar to the following:

Code:

---

fbq('track', 'ViewContent', {
content_ids: ['1234'],
content_type: 'product',
value: 1,026.97,
currency: 'GBP'
});

---

Basically could be seen as:

Code:

---

fbq('track', 'ViewContent', {
content_ids: ['1234'],
content_type: 'product',

value: 1,
026.97,

currency: 'GBP'
});

---

Based on standard formatting it would "look" like the above and appear to be missing the right side of the parameter in the second row of red.

My preliminary review so far is that there is something in this troubled store that has either modified the shopping_cart class or through observation modifies the operation. I can see that the data returned from the inside calculations at this site is indicating that a sale product has an "original" price

('preDiscPriceTotal')

of 0.00, but I don't know what is causing that and whether it is related to changes made to attempt to adjust for tax.

Possibly more to follow.

Quote:

---

Originally Posted by mc12345678

Possibly more to follow.

---

Thanks for looking.
moved the file across.
removed the comma from facebook pixel (although facebook was reading it fine through the Chrome Pixel addon checker thingy)
still no change.

Google reports errors 400 Dynamic Price Updater (Version Installed) 3.2.0

https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update
https://www.MYDOMAIN/ajax.php?produc...hod=dpu_update

Well, this is expected behavior based on the code associated with ajax requested and as incorporated in ZC 1.5.6 (which has been incorporated into this version of DPU).

Essentially it has been determined that there is no value in indexing the ajax response.
(Especially when the response requires data to be posted )