[mapguide-commits] r6003 - trunk/Tools/Maestro/Maestro.Editors/MapDefinition

[svn_mapguide at osgeo.org]

Author: jng
Date: 2011-07-26 05:16:09 -0700 (Tue, 26 Jul 2011)
New Revision: 6003

Modified:
   trunk/Tools/Maestro/Maestro.Editors/MapDefinition/FiniteScaleListCtrl.cs
Log:
#1748: Must remind myself that the ItemDeleted type for the BindingList<T>.ListChanged event is completely useless and should be avoided at all costs.


Modified: trunk/Tools/Maestro/Maestro.Editors/MapDefinition/FiniteScaleListCtrl.cs
===================================================================
--- trunk/Tools/Maestro/Maestro.Editors/MapDefinition/FiniteScaleListCtrl.cs	2011-07-26 12:01:59 UTC (rev 6002)
+++ trunk/Tools/Maestro/Maestro.Editors/MapDefinition/FiniteScaleListCtrl.cs	2011-07-26 12:16:09 UTC (rev 6003)
@@ -74,11 +74,6 @@
                         AddScaleToMap(_scales[e.NewIndex]);
                     }
                     break;
-                case ListChangedType.ItemDeleted:
-                    {
-                        RemoveScaleFromMap(_scales[e.NewIndex]);
-                    }
-                    break;
                 case ListChangedType.Reset:
                     {
                         ClearScales();
@@ -95,6 +90,7 @@
 
         private void RemoveScaleFromMap(double scale)
         {
+            _scales.Remove(scale);
             _map.InitBaseMap();
             _map.BaseMap.RemoveFiniteDisplayScale(scale);
         }
@@ -143,7 +139,7 @@
             if (lstDisplayScales.SelectedItem != null)
             {
                 double scale = (double)lstDisplayScales.SelectedItem;
-                _scales.Remove(scale);
+                RemoveScaleFromMap(scale);
             }
         }