Dynamic Price Updater

[mc12345678]

This is the function in question at the end of the file:

  /*
   * Formats the response and flushes with the appropriate headers
   * This should be called last as it issues an exit
   *
   * @return void
   */
  protected function dumpOutput($outputType = "XML") {
    if ($outputType == "XML") {
    // output the header for XML
    header("content-type: text/xml");
    // set the XML file DOCTYPE
    echo '<?xml version="1.0" encoding="UTF-8" ?>' . "\n";
    // set the responseType
    echo '<root>' . "\n" . '<responseType>' . $this->responseType . '</responseType>' . "\n";
    // now loop through the responseText nodes
    foreach ($this->responseText as $key => $val) {
      echo '<responseText' . (!is_numeric($key) && !empty($key) ? ' type="' . $key . '"' : '') . '><![CDATA[' . $val . ']]></responseText>' . "\n";
    }
    die('</root>');
    } elseif ($outputType == "JSON") {
      $data = array();
      // output the header for JSON
      header('Content-Type: application/json');
      // DO NOT set a JSON file DOCTYPE as there is none to be included.
//      echo '<?xml version="1.0" encoding="UTF-8" ' . "\n";
      // set the responseType
      $data['responseType'] = $this->responseType;
      // now loop through the responseText nodes
      foreach ($this->responseText as $key => $val) {
          if (!is_numeric($key) && !empty($key)) {
            $data['data'][$key] = $val;
          }
      }
      die(json_encode($data));
    }
  }

$data is an array that is created in the function, $this->responseText is populated throughout the class, but it contains only the key/response information captured throughout the process and XML is not at all captured in that process from what I see of the file. The die at the end of each if statement is present to prevent processing any further information.

Is an older version or mix of files being used for some reason?

[Calljj]

nope, all latest, the json i presented was the request to the ajax, not the response, i presumed it was getting the hump with the request seeing as the result was an error.
i posted the response back earlier in the thread:

pasted here again for clarity:

responseText: "\r\n\ufeff{\"responseType\":\"success\",\"data\":{\"priceTotal\":\"<font size=\\\"0.5\\\" color=\\\"#636363\\\">Price: <\\/font>&pound;1,209.00\",\"preDiscPriceTotalText\":\"<font size=\\\"0.5\\\" color=\\\"#636363\\\">Price: <\\/font>\",\"preDiscPriceTotal\":\"&pound;1,007.50\",\"stock_quantity\":\"1\",\"weight\":\"7\"}}"

[mc12345678]

nope, all latest, the json i presented was the request to the ajax, not the response, i presumed it was getting the hump with the request seeing as the result was an error

I'm currently away from a device that can show me related information. Will respond back when able. Note, that between now and then, if this site is not navigated, you will not receive notification of my response if you only read your emails.

[Calljj]

This is the function in question at the end of the file:

$data is an array that is created in the function, $this->responseText is populated throughout the class, but it contains only the key/response information captured throughout the process and XML is not at all captured in that process from what I see of the file. The die at the end of each if statement is present to prevent processing any further information.

outputs:

 Array
(
    [responseType] => success
    [data] => Array
        (
            [priceTotal] => <font size="0.5" color="#636363">Price: </font>&pound;909.00
            [preDiscPriceTotalText] => <font size="0.5" color="#636363">Price: </font>
            [preDiscPriceTotal] => &pound;757.50
            [stock_quantity] => 1
            [weight] => 7
        )

)

[mc12345678]

outputs:

 Array
(
    [responseType] => success
    [data] => Array
        (
            [priceTotal] => <font size="0.5" color="#636363">Price: </font>&pound;909.00
            [preDiscPriceTotalText] => <font size="0.5" color="#636363">Price: </font>
            [preDiscPriceTotal] => &pound;757.50
            [stock_quantity] => 1
            [weight] => 7
        )

)

Would you mind trying something?

file: includes/modules/pages/product_info/jscript_dynamic_price_updater.php
Line: 137

Change:

_this.responseJSON = jqXHR.responseJSON;

to:

_this.responseJSON = response;

Also, could you in the watch screen enter the following and identify what version of software we are talking about?

jQuery.fn.jquery

[Calljj]

Would you mind trying something?

file: includes/modules/pages/product_info/jscript_dynamic_price_updater.php
Line: 137

Change:

_this.responseJSON = jqXHR.responseJSON;

to:

_this.responseJSON = response;

Also, could you in the watch screen enter the following and identify what version of software we are talking about?

jQuery.fn.jquery

so changing the line not made any difference, the error is thrown back as a result of the request, whereas that line is processing a successfull request?

jquery gives 3.2.1

thanks for continued help on this.

[mc12345678]

so changing the line not made any difference, the error is thrown back as a result of the request, whereas that line is processing a successfull request?

jquery gives 3.2.1

thanks for continued help on this.

That seem to indicate that for some reason returning from the class back to the page, the data is basically being stripped...

By making that change, the variable _this.responseJSON was supposed to be set to the response data, the if statement is intended to offer an alternate way of processing as has/had been identified maybe possible; however, now the responseText seems to not be formatted within the parse call to support making that conversion. So need to see what can be done/altered for that.

Can you also provide the value of responseText as identified just before that error? I'm going to try to see what I can offer as a modified suggestion, but I'm also wondering if some other header information needs to be sent from the class to prevent "screening" the data away.

[mc12345678]

so changing the line not made any difference, the error is thrown back as a result of the request, whereas that line is processing a successfull request?

jquery gives 3.2.1

thanks for continued help on this.

Could you try:
in includes/classes/dynamic_price_updater.php
at/around line 746
just below:

      // output the header for JSON
      header('Content-Type: application/json');

adding:

      header('X-Content-Type-Options: nosniff');

so that area of the code would look like:

      // output the header for JSON
      header('Content-Type: application/json');
      header('X-Content-Type-Options: nosniff');

By the way is this happening on one browser only or any?

[Calljj]

Could you try:
so that area of the code would look like:

      // output the header for JSON
      header('Content-Type: application/json');
      header('X-Content-Type-Options: nosniff');

By the way is this happening on one browser only or any?

Changed code,no difference.
happening on all browsers.

[mc12345678]

Changed code,no difference.
happening on all browsers.

Done a little more leg work... Looks like there might be an extra "blank" line somewhere in the process.

Check the following file to ensure that before the first <?php that there is no "hidden" character or return. If necessary delete the entire first line and retype the text of that line using your favorite plain text editor. Or should be able to copy the file from the latest distribution or from github.

includes/classes/ajax/zcDPU_Ajax.php

Also be sure that there is no closing ?> in the files. Having such a closing followed by a return could cause the same issue.

Further, I highly suspect the includes/classes/ajax/zcDPU_Ajax.php file as having from a while back the UTF8 BOM (Byte Order Mark) "character" that was accidentally added in when I was working on various filetypes some time ago. The file may look the same as the current version, but it is not and it will cause the problem described. I'll try to see if there is anything there in the current fileset, but if installation was done by comparison, then that character may have been missed. It is not a visible character like the text on this screen, but it would provide the opening content of the responseText where it shows: \r\n\ufeff

Once the responseText is "parsed" it turns that section into a carriage returned new line with the BOM character being the first character on the second line that is causing the issue.

Whatever method you use, could you please explain what was tried, what worked, and what didn't? I know someone else was having a somewhat similar problem, but it hadn't dawned on me as much about what was going on.