bos Posted December 1, 2022 Share Posted December 1, 2022 After upgrading to 6.4.7, we (or the Paypal gateway) can no longer alter orders to "Processing" status. Paid orders are now set as "Pending" instead of "Processing" and if we manually change it to "Processing" we get a blank screen with the following in the error log: Quote [<strong>Exception</strong>] /customers/2/b/5/domain/httpd.www/store/modules/gateway/PayPal/gateway.class.php:23 - Attempt to modify property "basket" on null How do we fix this? (And before anyone mentions it, we tried to replace the Paypal Gateway with the new Paypal-plugin (that has an incredibly annoying nag screen on all orders, despite clicking "X") but Paypal will charge us 10% extra since we need to upgrade to a business account, and therefore we cannot do that, and therefore we have to use Paypal Gateway still) Quote Link to comment Share on other sites More sharing options...
havenswift-hosting Posted December 1, 2022 Share Posted December 1, 2022 As per Al's comments on the github issue you raised, this is almost certainly down to a faulty or partially completed upgrade 1 Quote Link to comment Share on other sites More sharing options...
bsmither Posted December 1, 2022 Share Posted December 1, 2022 (edited) This may be related to an open issue: https://github.com/cubecart/v6/issues/3057 If the error was copied from CubeCart's admin, System Error Log, if you hover the mouse over the error, there may be a popup that gives the trace PHP made to get to the code that errored. If that popup appears, try to copy/paste the contents to a reply here. Edited December 1, 2022 by bsmither 1 Quote Link to comment Share on other sites More sharing options...
bsmither Posted December 1, 2022 Share Posted December 1, 2022 There is a reference in the issue to a conversation: https://forums.cubecart.com/topic/58033-set-order-to-processing-then-nothing/ that mentions an edit to line 501 of order.class.php. I do not know why the that person had a problem - it seems to be a really simple edit. Quote Link to comment Share on other sites More sharing options...
Al Brookbanks Posted December 1, 2022 Share Posted December 1, 2022 Sorry if I was rude on GitHub @bos. My kids were driving me mental this morning and I was at boiling point. If you want to open a support ticket I can look at this. Otherwise these gentlemen may be able to help.. heck it could even be a bug but it doesn't look like it form the offset. .. back to support tickets for me .. it's oddly busy ATM. Quote Link to comment Share on other sites More sharing options...
bos Posted January 16 Author Share Posted January 16 On 12/1/2022 at 10:50 AM, bsmither said: If the error was copied from CubeCart's admin, System Error Log, if you hover the mouse over the error, there may be a popup that gives the trace PHP made to get to the code that errored. If that popup appears, try to copy/paste the contents to a reply here. Hovering the lines in System Error Log does not reveal any popups for me. Is there any other way to get a stacktrace? Quote Link to comment Share on other sites More sharing options...
bsmither Posted January 16 Share Posted January 16 There is still a question regarding "modify property 'basket' on null" error for PayPal. https://github.com/cubecart/v6/issues/3057 The "unsupported operand type" errors has a recently posted fix at: https://github.com/cubecart/v6/issues/3133 Quote Link to comment Share on other sites More sharing options...
bos Posted March 1 Author Share Posted March 1 Update: we now found out that the same error goes for the Bank Transfer gateway, meaning there are now no means at all for us getting paid since all gateways goes dead. Quote Link to comment Share on other sites More sharing options...
bsmither Posted March 1 Share Posted March 1 The image above suggests the error was logged when the admin brought the order up for inspection from the Unsettled Orders tab in the Dashboard. And the Bank Transfer gateway does leave the order at Pending. It doesn't provide a way for the customer to complete a transaction. Please read the following issue in the Github and implement the fixes detailed there - all of them, as the second set of changes undoes some of the first set of changes. Or, upgrade to CC6410 if you haven't done so already. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.