Viewing 8 replies - 1 through 8 (of 8 total)
  • bforsoft

    (@bforsoft)

    Hi, I had the same problem. The solution is in restaurant-reservations/lib/simple-admin-pages/js/admin.js

    The call for `sap_scheduler_register_events()´on line 29 seems to be incorrect. I have made a small correction and it seems to work now.

    I have made this ‘hack’ in the original code. I am hoping for an official update from the developer.

    NateWr

    (@natewr)

    Hi,

    Thanks for reporting this. Can you give me some more details about your environment? I’m unable to reproduce this issue myself. Useful information you could provide include:

    Version of WordPress
    Browser (Chrome, Firefox, IE)
    Operating System (Win 7, Mac, Linux)

    bforsoft: what about the call to sap_scheduler_register_events() on line 29 seemed incorrect to you and what did you change that made it work?

    If I can reproduce the issue I’ll certainly get a fix out as soon as possible.

    Thread Starter sangwh

    (@sangwh)

    I tested 2 browsers (Firefox and Chrome). Chrome is working but Firefox is NOT working. See below for more detail.

    Version of WordPress: WordPress Version 3.9.1
    Browser: Firefox latest version 29.0.1 -> Clicked but Nothing happened(just adding # in the website address admin.php?page=rtb-settings&tab=rtb-schedule#
    Browser: Chrome Version 34.0.1847.131 m -> Working Fine
    Operating System: Window XP

    Thread Starter sangwh

    (@sangwh)

    Hi, I used Chrome browser to set the scheduling rule and after I clicked “Save”, it moved to the page like http://www.yourwebsite.com/wp-admin/options.php and shows below message.

    Warning: Invalid argument supplied for foreach() in /home/xx/public_html/wp-content/plugins/restaurant-reservations/lib/simple-admin-pages/classes/AdminPageSetting.Scheduler.class.php on line 552

    Warning: Cannot modify header information – headers already sent by (output started at /home/xx/public_html/wp-content/plugins/restaurant-reservations/lib/simple-admin-pages/classes/AdminPageSetting.Scheduler.class.php:552) in /home/xx/public_html/wp-includes/pluggable.php on line 1121

    When I go back to Booking Schedule page in wp-admin /wp-admin/admin.php?page=rtb-settings&tab=rtb-schedule, schedule time that I have just set is there (I set it between 6pm-12pm everyday) but it is NOT working (NO time can be selected) on the front-end booking page.

    bforsoft

    (@bforsoft)

    Nate: Thanks for the rapid response. In Firefox the sap_scheduler_register_events() returns the following error:

    Error: ReferenceError: sap_scheduler_register_events is not defined
    File: http://xxxxxxxxxxxxx/wp-content/plugins/restaurant-reservations/lib/simple-admin-pages/js/admin.js?ver=1.0
    Line: 29

    The fast and dirty fix I did to save time is simply replace the function call with the function code. (did not have enough time to debug the code properly)

    NateWr

    (@natewr)

    Thanks both of you for the feedback. I’ve found and fixed the JavaScript error in Firefox.

    sangwh, I’m unable to reproduce the Warning notice you received, but I’ve put in a check that should prevent it from appearing again.

    I’ll put out a new version with a fix in the next couple hours.

    sangwh, if you used the time 6pm-12pm, no times will be available because 12pm (noon) is considered earlier than 6pm. You probably want to go with 11:45pm, which is the latest time that can be selected in a day. Let me know if that works out for you.

    NateWr

    (@natewr)

    FYI, the issue is in a library that is shared with my Food and Drink Menu plugin. I will update that plugin at the same time, so if you are using both plugins update both to ensure the issue is fixed.

    bforsoft

    (@bforsoft)

    Thanks Nate!

Viewing 8 replies - 1 through 8 (of 8 total)

The topic ‘Add new scheduling rule’ is closed to new replies.