Changeset 33963 in osm

Timestamp:

2017-12-28T15:37:27+01:00 (7 years ago)

Author:

donvip

Message:

fix #josm14013 - buttons not working (patch by javiersanp)

File:

• applications/editors/josm/plugins/FixAddresses/src/org/openstreetmap/josm/plugins/fixAddresses/gui/AddressEditDialog.java (modified) (5 diffs)

applications/editors/josm/plugins/FixAddresses/src/org/openstreetmap/josm/plugins/fixAddresses/gui/AddressEditDialog.java

@@ -45 +45 @@
 import org.openstreetmap.josm.plugins.fixAddresses.StringUtils;
 import org.openstreetmap.josm.plugins.fixAddresses.gui.actions.AbstractAddressEditAction;
+import org.openstreetmap.josm.plugins.fixAddresses.gui.actions.ApplyAllGuessesAction;
 import org.openstreetmap.josm.plugins.fixAddresses.gui.actions.AddressActions;
 import org.openstreetmap.josm.tools.ImageProvider;
@@ -61 +62 @@
     private JTable streetTable;
 
+    private ApplyAllGuessesAction applyGuessesAction = AddressActions.getApplyGuessesAction();
     private AbstractAddressEditAction[] actions = new AbstractAddressEditAction[] {
         AddressActions.getResolveAction(),
         AddressActions.getGuessAddressAction(),
-        AddressActions.getApplyGuessesAction(),
-        AddressActions.getSelectAction(),
+        applyGuessesAction,
         AddressActions.getRemoveTagsAction(),
-        AddressActions.getConvertToRelationAction(),
-        AddressActions.getConvertAllToRelationAction()
+        AddressActions.getSelectAction()
     };
 
@@ -126 +126 @@
             unresolvedTable.getSelectionModel().addListSelectionListener(this);
             unresolvedTable.getSelectionModel().addListSelectionListener(new IncompleteAddressListener());
-            unresolvedTable.addMouseListener(AddressActions.getApplyGuessesAction());
+            unresolvedTable.addMouseListener(applyGuessesAction);
 
             JTableHeader header = unresolvedTable.getTableHeader();
@@ -146 +146 @@
             try {
                 JPanel unresolvedButtons = new JPanel(new GridLayout(2, 5, 5, 5));
-                SideButton assign = new SideButton(AddressActions.getResolveAction());
-                unresolvedButtons.add(assign);
-
-                SideButton guess = new SideButton(AddressActions.getGuessAddressAction());
-                unresolvedButtons.add(guess);
-                SideButton applyAllGuesses = new SideButton(AddressActions.getApplyGuessesAction());
-                unresolvedButtons.add(applyAllGuesses);
-
-                SideButton removeAddressTags = new SideButton(AddressActions.getRemoveTagsAction());
-                unresolvedButtons.add(removeAddressTags);
-
-                unresolvedButtons.add(new JPanel());
-
-                SideButton selectInMap = new SideButton(AddressActions.getSelectAction());
-                unresolvedButtons.add(selectInMap);
+                for (AbstractAddressEditAction action : actions) {
+                    action.setContainer(addressEditContainer);
+                    SideButton button = new SideButton(action);
+                    unresolvedButtons.add(button);
+                }
                 headerPanel2.setMinimumSize(new Dimension(100, 70));
 
@@ -202 +192 @@
         } else {
             this.getContentPane().add(new JLabel(tr("(No data)")), BorderLayout.CENTER);
-        }
-
-        for (int i = 0; i < actions.length; i++) {
-            actions[i].setContainer(addressEditContainer);
         }