From: Alan Jack Pippin <ajp@pippin.(none)>
Date: Sun, 7 Oct 2007 21:39:12 +0000 (-0600)
Subject: Fixed some bugs in the scheduling appointment email logic.
X-Git-Tag: release_0_1_0~15
X-Git-Url: http://git.pippins.net/embedvideo/%27%20.%20%24this-%3Ephpgw_js_url%20.%20%27/jscalendar/static/checkmark.gif?a=commitdiff_plain;h=5f9a574d6d8aa73a22afe75d08ed1c16f4af81f4;p=eq%2F.git

Fixed some bugs in the scheduling appointment email logic.
---

diff --git a/inc/class.eq.inc.php b/inc/class.eq.inc.php
index b0a9fd0..850129b 100644
--- a/inc/class.eq.inc.php
+++ b/inc/class.eq.inc.php
@@ -763,7 +763,7 @@ class eq
 	     . " AND date='".$activity['date']."' AND notes='".$activity['notes']."'";
 	  $this->db->query($sql,__LINE__,__FILE__);
 	  if($this->db->next_record()) {
-	    print "activity: " . $this->db->f('activity') . "<br>";
+	    //print "activity: " . $this->db->f('activity') . "<br>";
 	    $activity['activity'] = $this->db->f('activity');
 	  }
 	  
@@ -3524,40 +3524,64 @@ class eq
 		 $time = $hour.':'.$minute.':'.'00';
 		 $uid = 0;
 
+		 // Zero out the family or elder if date = NULL
+		 if($date == "") {
+		   $elder = 0;
+		   $family = 0;
+		 }
+		 
 		 // Update an existing appointment
 		 if($appointment < $this->max_appointments)
 		   {
 		     //Only perform a database update if we have made a change to this appointment
 		     $sql = "SELECT * FROM eq_appointment where " .
 			"appointment='$appointment'" .
-			"and elder='$elder'" .
-			"and family='$family'" .
-			"and date='$date'" .
-			"and time='$time'";
+			" and presidency='$presidency'" .
+			" and elder='$elder'" .
+			" and family='$family'" .
+			" and date='$date'" .
+			" and time='$time'";
 		     $this->db->query($sql,__LINE__,__FILE__);
 		     if(!$this->db->next_record()) {
-		       $this->db->query("UPDATE eq_appointment set" .
+		       $old_date = $this->db->f('date');
+		       $old_time = $this->db->f('time');
+		       $this->db2->query("UPDATE eq_appointment set" .
 					" family=" . $family . 
 					" ,elder=" . $elder . 
 					" ,date='" . $date . "'" .
 					" ,time='" . $time . "'" .
+					" ,presidency='" . $presidency . "'" .
 					" WHERE appointment=" . $appointment,__LINE__,__FILE__);
 		       
 		       // Email the appointment
-		       if(($date != "") && ($time != "")) { 
-			 $this->email_appt($appointment);
-		       }
+		       $this->email_appt($appointment);
 		     }
 		   }
 		 
 		 // Add a new appointment
 		 else if(($appointment >= $this->max_appointments) && ($date != "") && ($time != ""))
 		   {
-		     $this->db->query("INSERT INTO eq_appointment (appointment,presidency,family,elder,date,time,uid) "
+		     //print "adding entry: appt=$appointment date: $date time: $time elder: $elder family: $family<br>";
+		     $this->db2->query("INSERT INTO eq_appointment (appointment,presidency,family,elder,date,time,uid) "
 			   . "VALUES (NULL,'" . $presidency . "','" . $family . "','"
 			   . $elder . "','" . $date . "','" . $time  . "','" . $uid ."')",__LINE__,__FILE__);
-		     
-		     //print "adding entry: appt=$appointment date: $date time: $time elder: $elder family: $family<br>";		     
+
+		     // Now reselect this entry from the database to see if we need
+		     // to send an appointment out for it.
+		     $sql = "SELECT * FROM eq_appointment where " .
+			"elder='$elder'" .
+			" and family='$family'" .
+			" and presidency='$presidency'" .
+			" and date='$date'" .
+			" and time='$time'" .
+			" and uid='$uid'";
+		     $this->db3->query($sql,__LINE__,__FILE__);
+		     if($this->db3->next_record()) {
+		       // Email the appointment if warranted
+		       if(($date != "") && ($time != "") && (($elder > 0) || $family > 0)) { 
+			 $this->email_appt($this->db3->f('appointment'));
+		       }
+		     }
 		   }
 	       }
 	   }