Add new scheduling rule

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.

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.

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

Hi, I used Chrome browser to set the scheduling rule and after I clicked “Save”, it moved to the page like https://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.

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: https://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)

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.

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.

Thanks Nate!