Days are off

  • Resolved CompKing

    (@compking)


    4 years, 6 months ago

    Thanks for a great plug-in.

    Running into a logic problem.

    %today% incorrectly shows Sunday when today is Monday
    %now% shows the correct time in my current time zone
    %tomorrow% returns a blank result (along with all other variables which refer to tomorrow)

    As such
    I can’t do anything with tomorrow hours, and today’s hours are wrong as they show yesterday’s hours.

Viewing 11 replies - 31 through 41 (of 41 total)
  • Thread Starter CompKing

    (@compking)

    I’ll give it a try tonight.

    Thanks

    Thread Starter CompKing

    (@compking)

    I believe that fixed the days last night. I am unsure if the if “%if_not_open_later%” variable is working properly. I’ll need to test again tonight, as I didn’t stay up late as my hours have been off here personally. ??

    Thanks again..

    Plugin Author Design Extreme

    (@designextreme)

    @compking Thanks for this – it’s already a significant improvement. I’ll wait to hear back from you so that I can be sure all the related issues are resolved.

    Plugin Author Design Extreme

    (@designextreme)

    @compking I’ve released version 1.12 with the recent changes. If there are any issues at all, please drop me a message. Thanks!

    Plugin Author Design Extreme

    (@designextreme)

    @compking I’ve rewritten the code that handles refreshing of opening hour lists and conditional shortcodes – it is much more stable when multiple instances exist in one page and I corrected an issue identifying the current day in some instances (that could result in the wrong open/closed status).

    Please download the new version and send me any feedback or issues. I’ll resolve these as quickly as possible.

    Thread Starter CompKing

    (@compking)

    So far so good for the variable values. The true test will be at GMT switch to next day.

    I am having a problem with %if_not_open_later% not being true, when it should be. This is again on my sample site.

    The variable %if_open_tomorrow% works correctly, as far as I can tell.

    Thread Starter CompKing

    (@compking)

    Further testing shows %if_open_later% as True when it should be False.

    https://test.northmountliquor.ca/sample-page/

    Plugin Author Design Extreme

    (@designextreme)

    @compking Strangely enough I spotted an error with this replacement condition (and its inverse) around an hour ago! I’ve altered the day timestamp to the current timestamp and it’s working properly now.

    I’ve also added support for the “next” opening/closing time and support for 24 hour conditions.

    I’ll release the fix and new features in 1.14 very soon.

    • This reply was modified 4 years, 5 months ago by Design Extreme. Reason: Added inverse mention
    Plugin Author Design Extreme

    (@designextreme)

    @compking Just added version 1.14 – it resolves this issue and adds new functionality and stability too.

    Thread Starter CompKing

    (@compking)

    As far as I can tell things are working properly! You nailed it. Thanks so very much…

    Plugin Author Design Extreme

    (@designextreme)

    Thanks @compking for pointing out the issues you found. This has been really helpful! ??

Viewing 11 replies - 31 through 41 (of 41 total)
  • The topic ‘Days are off’ is closed to new replies.