Changeset 234 in josm for src/org/openstreetmap

Timestamp:

2007-05-08T22:58:40+02:00 (17 years ago)

Author:

framm

Message:

Fixes bug introduced a bug with 233 that would never clear the modified flag after saving to file.

File:

• src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java (modified) (1 diff)

src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

@@ -225 +225 @@
          * @param processed A list of all objects, that were actually uploaded. 
          *              May be <code>null</code>, which means nothing has been uploaded but 
-         *              saved to disk instead.
+         *              saved to disk instead. Note that an empty collection for "processed"
+         *      means that an upload has been attempted but failed.
          */
         public void cleanData(final Collection<OsmPrimitive> processed, boolean dataAdded) {
 
-                // return immediately if nothing really happened (typically when upload
-                // aborts due to a server connection error)
-                if ((processed == null || processed.isEmpty()) && !dataAdded)
+                // return immediately if an upload attempt failed
+                if (processed != null && processed.isEmpty() && !dataAdded)
                         return;