MRBS Version: 1.4.7
Server: HP OpenVMS on Itanium
Setting these variables to FALSE (as in the default configuration) means they are actually TRUE because of the use of empty in a test like this:
$repeats_allowed = $is_admin || empty($auth['only_admin_can_book_repeat']);
The empty function returns TRUE if the variable is 0, FALSE or not set, If the test was changed to:
$repeats_allowed = $is_admin || $auth['only_admin_can_book_repeat']
This would give the desired results whether the variable is FALSE or not set.
There are 5 cases of these tests in the code: 3 in edit_entry.php, 1 in edit_entry_handler.php and 1 in view_entry.php