From 8ef3b83adf1adf0e4b49c66b2b207009fd555a36 Mon Sep 17 00:00:00 2001
From: Nathan Coad <nathan@nath15mbpr.local>
Date: Sat, 5 May 2018 21:13:52 +1000
Subject: [PATCH] invert logic

---
 booking.variety_admin.inc | 23 ++++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/booking.variety_admin.inc b/booking.variety_admin.inc
index 6d73dee..b84d191 100644
--- a/booking.variety_admin.inc
+++ b/booking.variety_admin.inc
@@ -765,6 +765,27 @@ function booking_variety_regn_edit_form_submit($form, &$form_state) {
       array(':nid' => $values['nid']))->fetchObject();    
     $previous_variety_ids = drupal_json_decode($person->booking_variety_ids);
 
+    foreach ($booking_variety_ids as $new_tid => $new_sid) {
+
+      //check if there was a previous session ID for this person in this timeslot
+      if (! isset($previous_variety_ids[$new_tid])) {
+        watchdog('booking_debug', 'Person @nid in timeslot @tid had no previous session id registered but is now @new.', array(
+          '@nid' => $values['nid'], '@tid' => $new_tid, '@new' => $new_sid));              
+      }
+      else {
+        //compare session id in this timeslot to the previous value
+        if ($new_sid != $previous_variety_ids[$new_tid]) {
+          watchdog('booking_debug', 'Person @nid in timeslot @tid previously registered for session id @vid but new value is @new', array(
+            '@nid' => $values['nid'], '@tid' => $new_tid, '@vid' => $previous_variety_ids[$new_tid], '@new' => $new_sid));      
+        }
+        else {
+          watchdog('booking_debug', 'Person @nid in timeslot @tid has not changed variety session @vid', array(
+            '@nid' => $values['nid'], '@tid' => $new_tid, '@vid' => $new_sid));              
+        }
+      }
+
+    }
+/*
     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(
@@ -816,7 +837,7 @@ function booking_variety_regn_edit_form_submit($form, &$form_state) {
 			->condition('booking_person_nid', $values['nid'])
 			->execute();
     watchdog('booking', "Updated variety session registration for person ID !nid", array('!nid' => $values['nid']));
-
+*/
 		$form_state['redirect'] = $redirect_path;
 	}
 }