Ticket #11169: fix_pbfwriter.patch

pbf/src/org/openstreetmap/josm/plugins/pbf/io/PbfWriter.java

@@ -97 +97 @@
                         stable.incr(tag.getKey());
                         stable.incr(tag.getValue());
                     }
-                    if (!omit_metadata && i.getUser() != null) {
-                        stable.incr(i.getUser().getName());
+                    if (!omit_metadata) {
+                        String userId = i.getUser() != null ? i.getUser().getName() : null;
+                        if (userId == null)
+                                userId = "";
+                        stable.incr(userId);
                     }
                 }
             }
@@ -116 +119 @@
                 for (OsmPrimitive e : entities) {
 
                     int uid = e.getUser() == null ? -1 : (int) e.getUser().getId();
-                    int userSid = stable.getIndex(e.getUser() == null ? "" : e.getUser().getName());
+                    String userId = e.getUser() != null ? e.getUser().getName() : null;
+                    if (userId == null)
+                        userId = "";
+                    int userSid = stable.getIndex(userId);
                     int timestamp = (int) (e.getTimestamp().getTime() / date_granularity);
                     int version = e.getVersion();
                     long changeset = e.getChangesetId();
@@ -476 +482 @@
 
         public void process(DataSet ds) {
             processor.processSources(ds.dataSources);
-            for (Node n : ds.getNodes()) {
-                processor.processNode(n);
-            }
-            for (Way w : ds.getWays()) {
-                processor.processWay(w);
-            }
-            for (Relation r : ds.getRelations()) {
-                processor.processRelation(r);
-            }
+                        ds.getNodes().stream().sorted((n1, n2) -> Long.compare(n1.getUniqueId(), n2.getUniqueId()))
+                                        .filter(n -> n.getCoor() != null).forEach((n) -> processor.processNode(n));
+                        ds.getWays().stream().sorted((w1, w2) -> Long.compare(w1.getUniqueId(), w2.getUniqueId()))
+                                        .filter(w -> w.getNodesCount() > 0).forEach((w) -> processor.processWay(w));
+                        ds.getRelations().stream().sorted((r1, r2) -> Long.compare(r1.getUniqueId(), r2.getUniqueId()))
+                                        .filter(r -> r.getMembersCount() > 0).forEach((r) -> processor.processRelation(r));
         }
 
         public void complete() {

pbf/test/unit/org/openstreetmap/josm/plugins/pbf/io/PbfExporterTest.java

@@ -32 +32 @@
      * @throws Exception if an error occurs
      */
     @Test
-    @Ignore("TODO: to fix")
     public void testTicket11169() throws Exception {
         try (InputStream is = Compression.ZIP.getUncompressedInputStream(
                 new FileInputStream(TestUtils.getRegressionDataFile(11169, "Portsmouth_Area.osm.zip")))) {