Changeset 1511 for Scheduling/trunk

Timestamp:

Aug 9, 2012, 1:30:57 PM (12 years ago)

Author:

Sam Habiel

Message:

Fixed AddAppointmentEnabled to check for the presence of resources in a resource group; otherwise you are given the ability to make an appointment on the resource group itself which causes a crash. Fixes bug EHS#000342.

File:

• Scheduling/trunk/cs/bsdx0200GUISourceCode/CGView.cs (modified) (2 diffs)

Scheduling/trunk/cs/bsdx0200GUISourceCode/CGView.cs

@@ -1627 +1627 @@
         private bool AddAppointmentEnabled()
         {
+            //new in 1.7: If there are no resources in the resource group, just say false.
+            //otherwise, we end up with being able to add appointments to empty resource groups.
+            if (this.Document.Resources.Count == 0)
+                return false;
+
+            //No cells selected for appointment. False.
             if (this.calendarGrid1.SelectedRange.Cells.CellCount < 1)
                 return false;
 
+            //If manager, can always make appointment
             bool bManager = this.DocManager.ScheduleManager;
             if (bManager == true)
@@ -1635 +1642 @@
                 return (true);
             }
+            // otherwise, check permissions, then check slots.
             else
             {