Changeset 18571 in josm for trunk

Timestamp:

2022-10-10T21:15:56+02:00 (2 years ago)

Author:

taylor.smock

Message:

Fix #22439: Filter deleting always deletes the first filter regardless of selection

Location:

trunk

Files:

• src/org/openstreetmap/josm/data/osm/FilterModel.java (modified) (1 diff)
• test/unit/org/openstreetmap/josm/gui/dialogs/FilterDialogTest.java (added)

trunk/src/org/openstreetmap/josm/data/osm/FilterModel.java

@@ -302 +302 @@
         List<Filter> result = new ArrayList<>(rowIndexes.length);
         for (int i = rowIndexes.length - 1; i >= 0; i--) {
-            result.add(filters.remove(i));
+            result.add(filters.remove(rowIndexes[i]));
         }
         // Reverse the list so that users can iterate through the filters in the order that they were in the model.