[Commits] r1044 - apps/opengeo/geoexplorer/branches/0.1.x/lib

[commits at geoext.org]

Author: tschaub
Date: 2009-06-11 20:06:33 +0200 (Thu, 11 Jun 2009)
New Revision: 1044

Modified:
   apps/opengeo/geoexplorer/branches/0.1.x/lib/GeoExplorer.js
Log:
Layer title is stored in two places.  We need to set the title in both places.  After records have been added to the store, the store will keep these synchronized (see #85).

Modified: apps/opengeo/geoexplorer/branches/0.1.x/lib/GeoExplorer.js
===================================================================
--- apps/opengeo/geoexplorer/branches/0.1.x/lib/GeoExplorer.js	2009-06-11 15:11:30 UTC (rev 1043)
+++ apps/opengeo/geoexplorer/branches/0.1.x/lib/GeoExplorer.js	2009-06-11 18:06:33 UTC (rev 1044)
@@ -402,11 +402,12 @@
                     // set layer title from config
                     if(conf.title) {
                         /**
-                         * TODO: If LayerStore is changed to listen for title
-                         * changes remove 'layer.name = conf.title'.
-                         * See http://www.geoext.org/trac/geoext/ticket/85
+                         * Because the layer title data is duplicated, we have
+                         * to set it in both places.  After records have been
+                         * added to the store, the store handles this
+                         * synchronization.
                          */
-                        layer.name = conf.title;
+                        layer.setName(conf.title);
                         record.set("title", conf.title);
                     }