Ticket #14013: fix14013.diff

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

@@ -44 +44 @@
 import org.openstreetmap.josm.plugins.fixAddresses.OSMStreet;
 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;
 
@@ -60 +61 @@
     private JTable unresolvedTable;
     private JTable streetTable;
 
+    private ApplyAllGuessesAction applyGuessesAction = AddressActions.getApplyGuessesAction();
     private AbstractAddressEditAction[] actions = new AbstractAddressEditAction[] {
         AddressActions.getResolveAction(),
         AddressActions.getGuessAddressAction(),
-        AddressActions.getApplyGuessesAction(),
+        applyGuessesAction,
+        AddressActions.getRemoveTagsAction(),
         AddressActions.getSelectAction(),
-        AddressActions.getRemoveTagsAction(),
-        AddressActions.getConvertToRelationAction(),
-        AddressActions.getConvertAllToRelationAction()
+        //AddressActions.getConvertToRelationAction(),
+        //AddressActions.getConvertAllToRelationAction()
     };
 
     private JLabel streetLabel;
@@ -125 +127 @@
             unresolvedTable.getSelectionModel().setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
             unresolvedTable.getSelectionModel().addListSelectionListener(this);
             unresolvedTable.getSelectionModel().addListSelectionListener(new IncompleteAddressListener());
-            unresolvedTable.addMouseListener(AddressActions.getApplyGuessesAction());
+            unresolvedTable.addMouseListener(applyGuessesAction);
 
             JTableHeader header = unresolvedTable.getTableHeader();
             header.addMouseListener(uaModel.new ColumnListener(unresolvedTable));
@@ -145 +147 @@
 
             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));
 
                 unresolvedPanel.add(unresolvedButtons, BorderLayout.SOUTH);
@@ -203 +195 @@
             this.getContentPane().add(new JLabel(tr("(No data)")), BorderLayout.CENTER);
         }
 
-        for (int i = 0; i < actions.length; i++) {
-            actions[i].setContainer(addressEditContainer);
-        }
-
         JPanel buttonPanel = new JPanel(new GridLayout(1, 10));
         JButton ok = new JButton(OK_COMMAND, ImageProvider.getIfAvailable(null, "ok"));
         ok.addActionListener(this);