Skip to content
This repository has been archived by the owner on Oct 12, 2024. It is now read-only.

Time Selected is "NONE" on order and email #124

Closed
3 tasks
gbettant opened this issue Mar 24, 2021 · 14 comments
Closed
3 tasks

Time Selected is "NONE" on order and email #124

gbettant opened this issue Mar 24, 2021 · 14 comments
Assignees

Comments

@gbettant
Copy link

Describe the bug
Since today, the orders i receive have a time selected as "none".
This is new, everything was correct before.
The customers have selected the time but on the order and email sent, pickup time selected is "none"

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://www.maisonbettant.fr/eshop
  2. Make an order
  3. Select time slot
  4. Received email indicates "none"

Expected behavior
Correct selected time on the order

Isolating the problem (mark completed items with an [x]):

  • I have deactivated other plugins and confirmed this bug occurs when only WooCommerce and this plugin are active.
  • This bug happens with a default WordPress theme active, or Storefront.
  • I can reproduce this bug consistently using the steps above.

WordPress Environment

  • Website URL: https://www.maisonbettant.fr/eshop
  • PHP Version: 7.4.15
  • WordPress Version: 5.7
  • WooCommerce Version: 5.1
  • Plugin Version: 1.3.13
  • WordPress Timezone Setting: Paris
  • WordPress Date Format Setting: j F Y
  • WordPress Time Format Setting: G \h i \m\i\n
  • All Plugin Start/End Pickup Time Settings: 8AM / 7PM
  • Plugin Pickup Time Interval Setting: 30 minutes
  • Plugin Pickup Time Delay Setting: 4 hours
  • Plugin Orders Per Interval: not specified
  • Plugin Pickup Time Open Days Ahead Setting: 30
@gbettant gbettant changed the title Time Selected is "NONE" Time Selected is "NONE" on order and email Mar 24, 2021
@timnolte timnolte self-assigned this Mar 27, 2021
@jquiros2
Copy link

jquiros2 commented Apr 15, 2021

Am having the same issue on some orders, and have not found the reason despite debugging (so far).
So orders this is happening on we are contacting the person who placed the order requesting the time/date for delivery or pickup.

@gbettant
Copy link
Author

I reinstalled the 1.3.12 version instead of the 1.3.13 : it now works properly, and don't have "none" anymore ....

@gbettant
Copy link
Author

So this means the problem is in the 1.3.13 version

@timnolte
Copy link
Member

@gbettant OK, thanks for that information. I'll take a look at what code changes were made between the 2 releases and see what is going on.

@gbettant
Copy link
Author

Ok great, hope installing the future 1.3.14 will get rid of this "none"

@jquiros2
Copy link

jquiros2 commented Apr 15, 2021

OK, thanks for that information. I'll take a look at what code changes were made between the 2 releases and see what is going on.

@timnolte Thanks so much! I also reinstalled 1.3.12 and will report if it "sometimes" (in my case) happens again as that might help see if it's an issue with my environment or not.

@gbettant
Copy link
Author

gbettant commented Apr 15, 2021

it was also sometimes for me with 1.3.13, i'd say about 3 or 4 times out of 5 i had "none".
Now with 1.3.12 all seems fine.

@mike-battistella
Copy link

In my installation it also occurs that sometimes the time field is "None" - I've experienced this after upgrading from WooCommerce 4 to 5 — but not sure if I've also upgraded this plugin at the same time.

@OHOHere
Copy link

OHOHere commented May 25, 2021

I'm having the same issue. No PickUp Time Selected information is saved. But I have noticed that if it is the customers first order that the selected Pick Up Time is not saved so it does not get to the email nor the order information. But the second time the customer orders, or if still logged in from the first order, the selected Pick Up Time is shown and saved correctly.

@MaW-cmkbk
Copy link

Same issue. From 12 orders there was 5 with None in pickup-time.

@timnolte
Copy link
Member

With the latest release I'm pretty sure that this issue should be resolved.

@jquiros2
Copy link

jquiros2 commented Nov 4, 2022

Am adding this in case it helps track it down as I just spent most of the day trying and am just not good enough!

I decided to update because I started having issues with 1.3.12 (thinking all the updates in Wordpress, woocommerce, etc could be causing it).

I updated to the latest - 1.4.2 and I am having this issue again.

I'm tracking the variable/field/order meta and logging with the following:

$order = wc_get_order($order_id);
$pickup_timeTMP = $order->get_meta( '_local_pickup_time_select', true);
$jq_fp = fopen("/var/www/html/berzerker/wp-content/themes/divi-child/myqvars.log","a");
$jq_date = date("Y-m-d_H:i:s");
fwrite($jq_fp, $jq_date.$order_id."-A_order is ".$order."\n");
fwrite($jq_fp, $jq_date.$order_id."-A_order_id is ".$order_id."\n");
fwrite($jq_fp, $jq_date.$order_id."-A_pickup_timeTMP is ".$pickup_timeTMP."\n");
fclose($jq_fp);

For some reason, when I enter an order as a shop owner or admin, the _local_pickup_time_select IS set, but when adding as a customer it IS NOT set.

So- might it have something to do with "protected" in the code?

Thanks again for any pointers!

@jquiros2
Copy link

jquiros2 commented Nov 5, 2022

Ouch! reverting to 1.3.12 does not fix it anymore, probably due to all the other woo/wp updates.
@MaW-cmkbk , @OHOHere , @mike-battistella , @gbettant how are you guys doing with this?

@rolodoom
Copy link

rolodoom commented Aug 23, 2023

I'm getting still this error, so I think this thread should be opened again. There is also no pickup date on the order not on the email

  • Plugin Version: 1.4.2
  • PHP Version: 7.4.33
  • WordPress Version: 6.3
  • WooCommerce Version: 8.0.2

I opened a detail bug explanation on #225

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants