One-Page Checkout [Support Thread]

[g2ktcf]

A cr@p is enough so that someone familiar with this plugin (or any other chunk of code) can figure out what's going on if the person who made the change is no longer available!

I hear you about the code that has no comments at all! Way back, I was tasked with creating a simulation model of a Motorola 68000. I found another developer who had created one and asked for a copy of the code to use as a basis.

When I received the code (a humongous blob, BTW), there was not a single comment. When I asked the developer why, he said "Comments only indicate what the code's intended to do". I hit the delete button and started from scratch.

I used the term "######## load" the other day. Of course EVERYONE knows what that means...but my smart-alecky friends would not leave me alone. Do you remember the GOTO days with line numbers? OMG without comments it was all spaghetti!

well darn, I got spanked for a bad word. The replaced word starts with b...has four letters....geez!

[g2ktcf]

BTW, I really love the one page checkout. Nicely done.

[keneso]

I'm not sure if I should post here, or at the bootstrap thread, in case I apologize.

That said by trying to make some changes to https://www.zen-cart.com/downloads.php?do=file&id=2305 (bootstrap template for OPC) I noticed there are differences in some of php snips, i.e. tpl_modules_opc_shopping_cart.php
My guess is I could use the "original" OPC template_default one, and need to be confirmed.

Also how would I add the price per unit to show in the tpl_modules_opc_shopping_cart.php

Thank you

[lat9]

I'm not sure if I should post here, or at the bootstrap thread, in case I apologize.

That said by trying to make some changes to https://www.zen-cart.com/downloads.php?do=file&id=2305 (bootstrap template for OPC) I noticed there are differences in some of php snips, i.e. tpl_modules_opc_shopping_cart.php
My guess is I could use the "original" OPC template_default one, and need to be confirmed.

Also how would I add the price per unit to show in the tpl_modules_opc_shopping_cart.php

Thank you

You're in the right support-thread!

The BS4/OPC makes changes to every one of OPC's template_default/template files (except for tpl_modules_checkout_one_shipping.php), so you'll want to make your subsequent changes to the tpl_modules_opc_shopping_cart.php that's now present in your clone of the bootstrap template's version.

I've got stuff going on this morning, but will try to get back later today to answer the part about adding the unit-price to that template.

[strelitzia]

Zen Cart 1.5.7b
Advanced Shipper 6
OPC 2.3.7

I'm getting the following message after clicking the Confirm button for an order.
"Your order's details have changed. Please review the current values and re-submit."

No values have changed. Total is the sale before clicking Confirm Order as it is after clicking.

Do you have any suggestions on how I can resolve this?

[strelitzia]

I took a look in the code to find what triggered this error message and saw the

$session_end_hash != $session_start_hash

check.

This led me to send the un hashed content of these two to error logs.

The output was

[09-Apr-2021 15:16:50 Europe/Paris] start total: £12.14
[09-Apr-2021 15:16:55 Europe/Paris] end total: £12.14

Clearly the use of £ vs £ is the issue here.

$session_end_hash is currently set like

$session_end_hash = $checkout_one->hashSession($currencies->format ($order->info['total']));

Can anyone suggest the correct way to resolve this?

[strelitzia]

I tried to use html_entity_decode like this

$session_end_hash = $checkout_one->hashSession(html_entity_decode ($currencies->format ($order->info['total'])));

And while

error_log("end total: ".html_entity_decode ($currencies->format ($order->info['total'])));

gives me the same output as

error_log("start total: ".$_POST['current_order_total']);

the hash values are still different.

[09-Apr-2021 15:56:48 Europe/Paris] session end hash: 35ee63b81d1bbd66ebe5a266291de4f6
[09-Apr-2021 15:56:48 Europe/Paris] session start hash: adc994f5f3b5a08bbada52c1c26ed16c

So clearly html_entity_decode is not the solution.

[lat9]

What CHARSET is in use on the site? Noting that the OPC's observer class (where the hashSession method resides) attempts to convert those HTML entities into their character counterparts:

        // -----
        // Add the order's current total to the blob that's being hashed, so that changes in the total based on
        // payment-module selection can be properly detected (e.g. COD fee).
        //
        // Some currencies use a non-ASCII symbol for its symbol, e.g. £.  To ensure that we don't get into
        // a checkout-loop, make sure that the order's current total is scrubbed to convert any "HTML entities"
        // into their character representation.
        //
        // This is needed since the order's current total, as passed into the confirmation page, is created by
        // javascript that captures the character representation of any symbols.
        //
        // Note: Some templates also include carriage-returns within the total's display, so remove them from
        // the mix, too!
        //
        $current_order_total = str_replace(array("\n", "\r"), '', $current_order_total);
        $session_data['order_current_total'] = html_entity_decode($current_order_total, ENT_COMPAT, CHARSET);

[strelitzia]

What CHARSET is in use on the site?

define('CHARSET', 'utf-8');

[strelitzia]

I've got this in the error logs.

PHP Warning: html_entity_decode(): charset `CHARSET' not supported, assuming utf-8 in /Users/steph/Sites/localhost/balloonmarket/includes/classes/observers/class.checkout_one_observer.php on line 732

Using PHP 7.4