From 2480c13641ec1e8527b17701d4e8886d2772f76d Mon Sep 17 00:00:00 2001 From: Nathan Coad Date: Fri, 13 Jul 2018 13:57:26 +1000 Subject: [PATCH] bugfix --- booking.variety_admin.inc | 22 ++++++++++++---------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/booking.variety_admin.inc b/booking.variety_admin.inc index 2ed34d9..743ef70 100644 --- a/booking.variety_admin.inc +++ b/booking.variety_admin.inc @@ -861,16 +861,18 @@ function booking_variety_regn_edit_form_submit($form, &$form_state) { } //iterate over the old list comparing to the new list in case there are regn counts we need to decrease - foreach ($previous_variety_ids as $previous_tid => $previous_sid) { - if (! isset($booking_variety_ids[$previous_tid])) { - watchdog('booking_debug', 'Person @nid in timeslot @tid is no longer registered for a variety session so reducing count for @vid by 1', array( - '@nid' => $values['nid'], '@tid' => $previous_tid, '@vid' => $previous_sid, - )); - // reduce regn count for old session - db_update('booking_variety_sessions') - ->expression('booking_variety_regncount', 'booking_variety_regncount - :count', array(':count' => 1)) - ->condition('vid', $previous_sid) - ->execute(); + if (isset($previous_variety_ids)) { + foreach ($previous_variety_ids as $previous_tid => $previous_sid) { + if (! isset($booking_variety_ids[$previous_tid])) { + watchdog('booking_debug', 'Person @nid in timeslot @tid is no longer registered for a variety session so reducing count for @vid by 1', array( + '@nid' => $values['nid'], '@tid' => $previous_tid, '@vid' => $previous_sid, + )); + // reduce regn count for old session + db_update('booking_variety_sessions') + ->expression('booking_variety_regncount', 'booking_variety_regncount - :count', array(':count' => 1)) + ->condition('vid', $previous_sid) + ->execute(); + } } }