[tiled] r661 - in trunk: . src/tiled/io/xml

[svn@biggeruniverse.com]

Author: bjorn
Date: 2006-06-05 17:51:40 -0500 (Mon, 05 Jun 2006)
New Revision: 661

Modified:
   trunk/CHANGES
   trunk/src/tiled/io/xml/XMLMapTransformer.java
Log:
Fixed loading of invisible layers.

Modified: trunk/CHANGES
===================================================================
--- trunk/CHANGES	2006-06-05 22:34:28 UTC (rev 660)
+++ trunk/CHANGES	2006-06-05 22:51:40 UTC (rev 661)
@@ -40,6 +40,7 @@
 * Fixed layer merging so that undo/redo of operation is functional
 * Fixed resetting of selected tileset in tile palette on layer changes
 * Fixed saving of tile properties when using a tileset image
+* Fixed loading of invisible layers
 * Rewrote configuration based on the Preferences class, no more tiled.conf
 * Cleaned up TileSet (removed checksumming and rotation/orienation code)
 * Removed experimental hexagonal mapview from new map dialog

Modified: trunk/src/tiled/io/xml/XMLMapTransformer.java
===================================================================
--- trunk/src/tiled/io/xml/XMLMapTransformer.java	2006-06-05 22:34:28 UTC (rev 660)
+++ trunk/src/tiled/io/xml/XMLMapTransformer.java	2006-06-05 22:51:40 UTC (rev 661)
@@ -535,7 +535,6 @@
 
         ml.setOffset(offsetX, offsetY);
         ml.setName(getAttributeValue(t, "name"));
-        ml.setVisible(visible == 1);
 
         if (opacity != null) {
             ml.setOpacity(Float.parseFloat(opacity));
@@ -612,6 +611,13 @@
             }
         }
 
+        // Invisible layers are automatically locked, so it is important to
+        // set the layer to potentially invisible _after_ the layer data is
+        // loaded.
+        // todo: Shouldn't this be just a user interface feature, rather than
+        // todo: something to keep in mind at this level?
+        ml.setVisible(visible == 1);
+
         return ml;
     }