One-Page Checkout [Support Thread]

Missing Configuration Key

OPC v2.4.1 (apparently)

@lat9, re the issue raised on Github OPC, I see that you referred to the 'Payment Methods Handling Form Submittal' configuration switch introduced in OPC v2.4.0

Even though I have been upgrading through the versions to v2.4.1, I don't have that setting in my admin but I see that the code to add it is in init_checkout_one_upgrade.php, can you suggest a way for me to manually add or re-run that SQL code to add this setting and perhaps also check for any others that might be missing?

I don't usually muck up my upgrades, for some reason OPC has been my bugbear.

simon1066:

Missing Configuration Key

OPC v2.4.1 (apparently)

@lat9, re the issue raised on Github OPC, I see that you referred to the 'Payment Methods Handling Form Submittal' configuration switch introduced in OPC v2.4.0

Even though I have been upgrading through the versions to v2.4.1, I don't have that setting in my admin but I see that the code to add it is in init_checkout_one_upgrade.php, can you suggest a way for me to manually add or re-run that SQL code to add this setting and perhaps also check for any others that might be missing?

I don't usually muck up my upgrades, for some reason OPC has been my bugbear.

It's ok, I checked the init_ files and manually added the missing configuration setting to the database - not sure why I thought it would be troublesome.

simon1066:

It's ok, I checked the init_ files and manually added the missing configuration setting to the database - not sure why I thought it would be troublesome.
It's only troublesome in that the setting didn't get added automatically on the upgrade.:(

Hah! When we were kids, you had to REMOVE and RE-INSTALL to get the database updates.
You tell that to kids these days and they don't believe you!

FEATURE REQUEST:

Hi lat9. I would like to request a feature (really a modification) to OPC to be included in a possible future update. I am always on the lookout for how to simplify things, because a simple shopping experience is the best shopping experience.

My suggestion: Under the Gift Certificate section of OPC, have the gift certificate balance automatically fill into the "Apply Amount" box instead of having a separate "Available Balance" line. The way it is currently setup is a little confusing and I think that this will simplify things for the customers. Let me know what you think of this suggestion!

swguy:

Hah! When we were kids, you had to REMOVE and RE-INSTALL to get the database updates
You tell that to kids these days and they don't believe you!My first computer was 16k of ram and a cassette tape for storage. It did have the latest technology in a 300 baud modem built in! :P

njcyx:

Hi @lat9, my site received several errors during the holiday, from one Australia customer.

[02-Jul-2022 09:31:55 America/New_York] Request URI: /index.php?main_page=checkout_one, IP address: 1.129.27.xxx
#1 trigger_error() called at [/public_html/includes/classes/OnePageCheckout.php:1058]
#2 OnePageCheckout->getAddressValuesFromDb() called at [/public_html/includes/classes/OnePageCheckout.php:1031]
#3 OnePageCheckout->getAddressValues() called at [/public_html/includes/templates/template_default/templates/tpl_modules_opc_address_block.php:26]
#4 require(/public_html/includes/templates/template_default/templates/tpl_modules_opc_address_block.php) called at [/public_html/includes/templates/template_default/templates/tpl_modules_opc_shipping_address.php:31]
#5 require(/public_html/includes/templates/template_default/templates/tpl_modules_opc_shipping_address.php) called at [/public_html/includes/templates/template_default/templates/tpl_checkout_one_default.php:54]
#6 require(/public_html/includes/templates/template_default/templates/tpl_checkout_one_default.php) called at [/public_html/includes/templates/responsive_classic/common/tpl_main_page.php:185]
#7 require(/public_html/includes/templates/responsive_classic/common/tpl_main_page.php) called at [/public_html/index.php:94]
--> PHP Fatal error: unknown address_book_id () for customer_id (109389) in /public_html/includes/classes/OnePageCheckout.php on line 1058.

> 
> I received 7 log files and 6 files are the above content. One is the following:
> ```php
[02-Jul-2022 09:39:35 America/New_York] Request URI: /ajax.php?act=ajaxOnePageCheckout&method=updateShipping, IP address: 1.129.27.xxx
#1 trigger_error() called at [/public_html/includes/classes/OnePageCheckout.php:992]
#2 OnePageCheckout->setTempShippingToBilling() called at [/public_html/includes/classes/ajax/zcAjaxOnePageCheckout.php:56]
#3 zcAjaxOnePageCheckout->updateShipping() called at [/public_html/ajax.php:85]
--> PHP Fatal error: Invalid access; $_SESSION['billto'] is not set. in /public_html/includes/classes/OnePageCheckout.php on line 992.

Here are the codes near line 1058:

    if ($address_info->EOF) {
        trigger_error("unknown address_book_id (" . $address_book_id . ') for customer_id (' . $_SESSION['customer_id'] . ')', E_USER_ERROR);
        exit();
    }

> 
> My OPC debug is ON so I have the log file at that time. Please DM me your email so I can send it to you. 
> 
> Thanks in advance.

Hi @lat9, may I have any updates for this issue?

lolwaut:

My suggestion: Under the Gift Certificate section of OPC, have the gift certificate balance automatically fill into the "Apply Amount" box instead of having a separate "Available Balance" line. The way it is currently setup is a little confusing and I think that this will simplify things for the customers. Let me know what you think of this suggestion!I would venture to guess that most folks want control over their funds.

I would think that the number complaining that you "took their money" would far outweigh those who thaked you for using it.

Some are holding on to that for Aunt Sally's birthday present.

With the fact that most folks skim over most any form, I think you would wind up with a lot of irate folks.

I don't have the time to look it up but, I'm thinking it might not even be legal to capture those funds without the customers' approval.

lolwaut:

FEATURE REQUEST:

Hi lat9. I would like to request a feature (really a modification) to OPC to be included in a possible future update. I am always on the lookout for how to simplify things, because a simple shopping experience is the best shopping experience.

My suggestion: Under the Gift Certificate section of OPC, have the gift certificate balance automatically fill into the "Apply Amount" box instead of having a separate "Available Balance" line. The way it is currently setup is a little confusing and I think that this will simplify things for the customers. Let me know what you think of this suggestion!

dbltoe:

I would venture to guess that most folks want control over their funds.

I would think that the number complaining that you "took their money" would far outweigh those who thaked you for using it.

Some are holding on to that for Aunt Sally's birthday present.

With the fact that most folks skim over most any form, I think you would wind up with a lot of irate folks.

I don't have the time to look it up but, I'm thinking it might not even be legal to capture those funds without the customers' approval.

As @dbltoe indicated, setting the input-field to contain the current Gift Certificate amount would result in a possibly-unwanted use of GC funds for the order. That's why the amount is indicated as text below the input field itself.

lat9:

As @dbltoe indicated, setting the input-field to contain the current Gift Certificate amount would result in a possibly-unwanted use of GC funds for the order. That's why the amount is indicated as text below the input field itself.

I don't see how that is possible, since you cannot apply more gift certificate funds than the total of an order. The system won't let you do it. If you apply $50 in gift certificate funds to a $30 order, you will still be left with $20 in your account. And the funds aren't applied until the user specifically clicks the APPLY button, so I feel this is not an issue as it relates to my suggestion.

Hi Cindy - I have a little workflow issue and I'm wondering if it's something that can be addressed with this wonderful module, or something else.

Here's the issue. There's a limitation that happens when a customer decides to enter their credit card information BEFORE they end up adding other order-related things (like applying Reward Points, or adding Insurance). When that happens, it results in the module erasing the CC information and throwing the error about it.

How to reproduce it on our website:

1. On the checkout page, the customer clicks the Credit Card option and fills in their credit card information.
2. They immediately realize that they want to add insurance, so they click the Insurance checkbox. This automatically updates the page and order total (which is what we want), but the module doesn't realize that Credit Card Information is still typed in the box......so it deletes it, and then the error is thrown about having blank Credit Card Info.

Screenshot: https://i.postimg.cc/kMRtwz55/screenshot-131.png

NOTE: This works fine if the customer first adds Insurance, and then enters the Credit Card information AFTERWARDS. But if the Credit Card information is first, and then they decide to add Insurance or apply Reward Points (both of which will correctly auto-update the order total), it will result in the blank credit card information and error.

(We only noticed this when a customer left us a frustrating comment about trying six times to checkout but kept having his info deleted.

IN CONCLUSION: Is there anyway to have the module NOT CLEAR the Credit Card Fields if there's text in the "Card Number:" and "CVV Number:" fields?

I assume that would bypass the error and frustration of customers who are using this checkout workflow of entering payment information before they add insurance or rewards, etc.

Hi Jeff

I don't no if this would help but under the Configuration OPC Payment-Block Action on Shipping Change settings, some card payment modules require the no-update setting, this enables the card payment fields to retain the inputted data whilst things like shipping is changed. When the customer comes back to the card payment fields the data is still there. If it's set on update or refresh try the no update setting and see what happens.

lolwaut:

FEATURE REQUEST:

Hi lat9. I would like to request a feature (really a modification) to OPC to be included in a possible future update. I am always on the lookout for how to simplify things, because a simple shopping experience is the best shopping experience.

My suggestion: Under the Gift Certificate section of OPC, have the gift certificate balance automatically fill into the "Apply Amount" box instead of having a separate "Available Balance" line. The way it is currently setup is a little confusing and I think that this will simplify things for the customers. Let me know what you think of this suggestion!

i LOVE this idea.

who cares about aunt sally.

i submitted a new plugin to do just this.

support thread is here:

https://www.zen-cart.com/showthread.php?228898-autofill-gift-certificate-value-support-thread

hopefully will get approved soon.

best.

I never even played a lawyer on tv but, i would certainly consult legal counsel before making this anything but OPTIONAL for the customer.

You don't know how aunt Sally can be.:cool: But, I know how I'd be if you decided where and when I spent my gift!

carlwhat:

i LOVE this idea.

who cares about aunt sally.

i submitted a new plugin to do just this.

support thread is here:

https://www.zen-cart.com/showthread.php?228898-autofill-gift-certificate-value-support-thread

hopefully will get approved soon.

best.

:bow: THANK YOU!

Zen Car Version 1.5.7d
ZCA Bootstrap 4
One Page Checkout 2.3.10

This error message has appeared recently on several occasions can someone explain what the issue is please.

[07-Jul-2022 10:22:25 UTC] PHP Fatal error: Uncaught Error: Call to a member function add_session() on null in /includes/classes/observers/class.checkout_one_observer.php:167
Stack trace:
#0 /includes/autoload_func.php(44): checkout_one_observer->__construct()
#1 /includes/application_top.php(222): require('/var/www/vhosts...')
#2 /index.php(25): require('/var/www/vhosts...')
#3 {main}
thrown in /includes/classes/observers/class.checkout_one_observer.php on line 167

[07-Jul-2022 10:22:25 UTC] Request URI: /index.php?main_page=account, IP address:
--> PHP Fatal error: Uncaught Error: Call to a member function add_session() on null in /includes/classes/observers/class.checkout_one_observer.php:167
Stack trace:
#0 /includes/autoload_func.php(44): checkout_one_observer->__construct()
#1 /includes/application_top.php(222): require('/var/www/vhosts...')
#2 /index.php(25): require('/var/www/vhosts...')
#3 {main}
thrown in /includes/classes/observers/class.checkout_one_observer.php on line 167.

Thanks

lolwaut:

I don't see how that is possible, since you cannot apply more gift certificate funds than the total of an order. The system won't let you do it. If you apply $50 in gift certificate funds to a $30 order, you will still be left with $20 in your account. And the funds aren't applied until the user specifically clicks the APPLY button, so I feel this is not an issue as it relates to my suggestion.
My point (and I believe @dbltoe's as well) is that pre-populating that input field with the current GC amount will result in the order being placed with GC funds if the customer's not 'watching'. I'll note that that's not necessarily what the customer wants to do.

ianhg:

Zen Car Version 1.5.7d
ZCA Bootstrap 4
One Page Checkout 2.3.10

This error message has appeared recently on several occasions can someone explain what the issue is please.

[07-Jul-2022 10:22:25 UTC] PHP Fatal error: Uncaught Error: Call to a member function add_session() on null in /includes/classes/observers/class.checkout_one_observer.php:167
Stack trace:
#0 /includes/autoload_func.php(44): checkout_one_observer->__construct()
#1 /includes/application_top.php(222): require('/var/www/vhosts...')
#2 /index.php(25): require('/var/www/vhosts...')
#3 {main}
thrown in /includes/classes/observers/class.checkout_one_observer.php on line 167

[07-Jul-2022 10:22:25 UTC] Request URI: /index.php?main_page=account, IP address:
--> PHP Fatal error: Uncaught Error: Call to a member function add_session() on null in /includes/classes/observers/class.checkout_one_observer.php:167
Stack trace:
#0 /includes/autoload_func.php(44): checkout_one_observer->__construct()
#1 /includes/application_top.php(222): require('/var/www/vhosts...')
#2 /index.php(25): require('/var/www/vhosts...')
#3 {main}
thrown in /includes/classes/observers/class.checkout_one_observer.php on line 167.

Thanks
Right, OPC is currently at v2.4.1. The issue you've reported was corrected in OPC v2.3.11.

Jeff_Mash:

Hi Cindy - I have a little workflow issue and I'm wondering if it's something that can be addressed with this wonderful module, or something else.

Here's the issue. There's a limitation that happens when a customer decides to enter their credit card information BEFORE they end up adding other order-related things (like applying Reward Points, or adding Insurance). When that happens, it results in the module erasing the CC information and throwing the error about it.

How to reproduce it on our website:

1. On the checkout page, the customer clicks the Credit Card option and fills in their credit card information.
2. They immediately realize that they want to add insurance, so they click the Insurance checkbox. This automatically updates the page and order total (which is what we want), but the module doesn't realize that Credit Card Information is still typed in the box......so it deletes it, and then the error is thrown about having blank Credit Card Info.

Screenshot: https://i.postimg.cc/kMRtwz55/screenshot-131.png

NOTE: This works fine if the customer first adds Insurance, and then enters the Credit Card information AFTERWARDS. But if the Credit Card information is first, and then they decide to add Insurance or apply Reward Points (both of which will correctly auto-update the order total), it will result in the blank credit card information and error.

(We only noticed this when a customer left us a frustrating comment about trying six times to checkout but kept having his info deleted.

IN CONCLUSION: Is there anyway to have the module NOT CLEAR the Credit Card Fields if there's text in the "Card Number:" and "CVV Number:" fields?

I assume that would bypass the error and frustration of customers who are using this checkout workflow of entering payment information before they add insurance or rewards, etc.
That clearing of the credit-card fields, IIRC, was a PCI-compliance-related change. What I **will **look into is the possibility of clearing *only *the CVV number field, so that all the information doesn't need to be re-entered.

lat9:

My point (and I believe @dbltoe's as well) is that pre-populating that input field with the current GC amount will result in the order being placed with GC funds if the customer's not 'watching'. I'll note that that's not necessarily what the customer wants to do.

You're right. I misunderstood.

Actually, now that I am fully testing this, if the gift card field is populated with anything but $0.00 and the order is submitted WITHOUT clicking the "Apply" button, Zencart actually throws an error: Your order's details have changed. Please review the current values and re-submit.

@carlwhat For this to be implemented properly, the checkout submit process would need to ignore that field. This might be more complicated than I initially thought. Does your plugin address this?