When receiving times from a JCC GPLAN backend, all times are in the evening because the comparisons are against Unix epoch. The intended comparison was probably against e.g. midday + unix_date.

The latter comparison would (I believe) result in a 1 h offset for afternoon/evening due to DST in summertime. Attached patch simply looks at the hour component of the time.

CommentFileSizeAuthor
#15 do-2507505-wfm_appointment_day_part_calculation-2.patch1.76 KBralphvdhoudt
#10 do-2507505-wfm_appointment_day_part_calculation.patch1.96 KBheine
#8 wfm_appointment_part_of_day_calculation.patch2.02 KBheine
wfm_appointment_part_of_day_calculation.patch1.75 KBheine

Comments

frankschaap’s picture

frankschaap commented

I wonder what the solution is to get this working in Almelo, because it appears to be working there.

And yes, we need to use some weird offsets to for Qmatic in Vught. Hearing about your case makes me wonder if these systems even use timezones... so far we have not even needed to adjust for DST o_0

heine’s picture

heine commented

In Almelo, I guess +unix_date was added to the comparison. Anything past 17:00 is marked as evening there however, which I believe is not expected and due to DST differences between January and June. Coming October Almelo will probably start showing the slot 17:00 - 18:00 as afternoon again.

Do you happen to have a dump of values received in webform_appointment_date_time_element_process() from a Qmatic backend? I would assume both client adapters would result in identical datetime formats.

heine’s picture

heine commented

Scratch that :)

Evening is derived from new DateTime('1970-01-01 17:00', $timezone);

So, evening means anything after 17:00. Is that correct? I would expect 18:00 as the cutoff.

frankschaap’s picture

frankschaap commented

Regular opening hours are 9-17. So yeah, for the municipality evening opening hours are > 17:00.

frankschaap’s picture

frankschaap commented

Sticking to the organisation's interpretation of time of day may in fact not be the most clear and productive way to service the customer. Should we reconsider?

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented

@Heine,

elseif ($hour < 17) {

or

elseif ($hour < 18) {

edit: my mistake, didn't notice the new comments today

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented

You can make a variable for morning, midday and evening with as default the official standaards.

heine’s picture

heine commented
StatusFileSize
new wfm_appointment_part_of_day_calculation.patch2.02 KB

Attached patch introduces variables.

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented
Status: Needs review » Needs work

Nice, only the variables will be called every run inside "foreach ($dates as $date => $time_array)"

heine’s picture

heine commented
Status: Needs work » Needs review
StatusFileSize
new do-2507505-wfm_appointment_day_part_calculation.patch1.96 KB

Indeed.

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented
Status: Needs review » Reviewed & tested by the community

Works perfect!

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented
Component: Code » Appointments
heine’s picture

heine commented
Project: Drupal voor Gemeenten » DvG Appointments
Version: 7.x-1.x-dev »
Component: Appointments » Documentation

Moving to fork.

mrharolda’s picture

mrharolda commented
Version: » 7.x-1.x-dev
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Needs reroll because of the rebranding ...

ralphvdhoudt’s picture

ralphvdhoudt commented
Status: Needs work » Needs review
Issue tags: -Needs reroll
StatusFileSize
new do-2507505-wfm_appointment_day_part_calculation-2.patch1.76 KB

Better patch, now against dvg appointments

mrharolda’s picture

mrharolda commented
Status: Needs review » Reviewed & tested by the community

Back to RbtC since there were no changes, except for using renamed functions.

  • MrHaroldA committed b7ab889 on 7.x-1.x authored by ralphvdhoudt 
    Issue #2507505 by Heine, ralphvdhoudt: Available timeslots are always in...
mrharolda’s picture

mrharolda commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.