Changeset 1446 in josm for trunk/src/org/openstreetmap/josm

Timestamp:

2009-02-25T16:09:21+01:00 (15 years ago)

Author:

stoecker

Message:

fixed #2236 - editing membership of untagged ways. patch by singularita

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/PropertiesDialog.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/PropertiesDialog.java

@@ -719 +719 @@
         }
 
-        boolean hasTags = !newSelection.isEmpty() && propertyData.getRowCount() > 0;
-        boolean hasSelection = !newSelection.isEmpty();
-        btnAdd.setEnabled(hasSelection);
-        btnEdit.setEnabled(hasTags);
-        btnDel.setEnabled(hasTags);
-        propertyTable.setVisible(hasSelection);
-        propertyTable.getTableHeader().setVisible(hasSelection);
-        selectSth.setVisible(!hasSelection);
-        if(hasTags) propertyTable.changeSelection(0, 0, false, false);
-
-        checkPresets(nodes, ways, relations, closedways);
-
         // re-load membership data
         // this is rather expensive since we have to walk through all members of all existing relationships.
@@ -760 +748 @@
         membershipTable.setVisible(membershipData.getRowCount() > 0);
 
+        boolean hasSelection = !newSelection.isEmpty();
+        boolean hasTags = hasSelection && propertyData.getRowCount() > 0;
+        boolean hasMemberships = hasSelection && membershipData.getRowCount() > 0;
+        btnAdd.setEnabled(hasSelection);
+        btnEdit.setEnabled(hasTags || hasMemberships);
+        btnDel.setEnabled(hasTags || hasMemberships);
+        propertyTable.setVisible(hasSelection);
+        propertyTable.getTableHeader().setVisible(hasSelection);
+        selectSth.setVisible(!hasSelection);
+        if(hasTags) propertyTable.changeSelection(0, 0, false, false);
+        else if(hasMemberships) membershipTable.changeSelection(0, 0, false, false);
+
+        checkPresets(nodes, ways, relations, closedways);
+
         if(propertyData.getRowCount() != 0 || membershipData.getRowCount() != 0) {
             setTitle(tr("Properties: {0} / Memberships: {1}",