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

Timestamp:

2016-06-22T21:45:43+02:00 (8 years ago)

Author:

Don-vip

Message:

fix #13033 - JoinAreasActionTest: Do not set status line (patch by michael2402) - gsoc-core

Location:

trunk/src/org/openstreetmap/josm/actions

Files:

• JoinAreasAction.java (modified) (1 diff)
• search/SearchAction.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

@@ -500 +500 @@
                 if (ds != null) {
                     ds.setSelected(allWays);
-                    Main.map.mapView.repaint();
                 }
             } else {

trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

@@ -530 +530 @@
 
     static void search(SearchSetting s) {
-        SearchTask.newSearchTask(s).run();
+        SearchTask.newSearchTask(s, new SelectSearchReceiver()).run();
+    }
+
+    /**
+     * Performs the search specified by the search string {@code search} and the search mode {@code mode} and returns the result of the search.
+     *
+     * @param search the search string to use
+     * @param mode the search mode to use
+     * @return The result of the search.
+     * @since 10457
+     */
+    public static Collection<OsmPrimitive> searchAndReturn(String search, SearchMode mode) {
+        final SearchSetting searchSetting = new SearchSetting();
+        searchSetting.text = search;
+        searchSetting.mode = mode;
+        CapturingSearchReceiver receiver = new CapturingSearchReceiver();
+        SearchTask.newSearchTask(searchSetting, receiver).run();
+        return receiver.result;
+    }
+
+    /**
+     * Interfaces implementing this may receive the result of the current search.
+     * @author Michael Zangl
+     * @since 10457
+     */
+    interface SearchReceiver {
+        /**
+         * Receive the search result
+         * @param ds The data set searched on.
+         * @param result The result collection, including the initial collection.
+         * @param foundMatches The number of matches added to the result.
+         * @param setting The setting used.
+         */
+        void receiveSearchResult(DataSet ds, Collection<OsmPrimitive> result, int foundMatches, SearchSetting setting);
+    }
+
+    /**
+     * Select the search result and display a status text for it.
+     */
+    private static class SelectSearchReceiver implements SearchReceiver {
+
+        @Override
+        public void receiveSearchResult(DataSet ds, Collection<OsmPrimitive> result, int foundMatches, SearchSetting setting) {
+            ds.setSelected(result);
+            if (foundMatches == 0) {
+                final String msg;
+                final String text = Utils.shortenString(setting.text, MAX_LENGTH_SEARCH_EXPRESSION_DISPLAY);
+                if (setting.mode == SearchMode.replace) {
+                    msg = tr("No match found for ''{0}''", text);
+                } else if (setting.mode == SearchMode.add) {
+                    msg = tr("Nothing added to selection by searching for ''{0}''", text);
+                } else if (setting.mode == SearchMode.remove) {
+                    msg = tr("Nothing removed from selection by searching for ''{0}''", text);
+                } else if (setting.mode == SearchMode.in_selection) {
+                    msg = tr("Nothing found in selection by searching for ''{0}''", text);
+                } else {
+                    msg = null;
+                }
+                Main.map.statusLine.setHelpText(msg);
+                JOptionPane.showMessageDialog(
+                        Main.parent,
+                        msg,
+                        tr("Warning"),
+                        JOptionPane.WARNING_MESSAGE
+                );
+            } else {
+                Main.map.statusLine.setHelpText(tr("Found {0} matches", foundMatches));
+            }
+        }
+    }
+
+    /**
+     * This class stores the result of the search in a local variable.
+     * @author Michael Zangl
+     */
+    private static final class CapturingSearchReceiver implements SearchReceiver {
+        private Collection<OsmPrimitive> result;
+
+        @Override
+        public void receiveSearchResult(DataSet ds, Collection<OsmPrimitive> result, int foundMatches,
+                SearchSetting setting) {
+                    this.result = result;
+        }
     }
 
@@ -540 +622 @@
         private boolean canceled;
         private int foundMatches;
-
-        private SearchTask(DataSet ds, SearchSetting setting, Collection<OsmPrimitive> selection, Predicate<OsmPrimitive> predicate) {
+        private SearchReceiver resultReceiver;
+
+        private SearchTask(DataSet ds, SearchSetting setting, Collection<OsmPrimitive> selection, Predicate<OsmPrimitive> predicate,
+                SearchReceiver resultReceiver) {
             super(tr("Searching"));
             this.ds = ds;
@@ -547 +631 @@
             this.selection = selection;
             this.predicate = predicate;
-        }
-
-        static SearchTask newSearchTask(SearchSetting setting) {
+            this.resultReceiver = resultReceiver;
+        }
+
+        static SearchTask newSearchTask(SearchSetting setting, SearchReceiver resultReceiver) {
             final DataSet ds = Main.getLayerManager().getEditDataSet();
+            return newSearchTask(setting, ds, resultReceiver);
+        }
+
+        /**
+         * Create a new search task for the given search setting.
+         * @param setting The setting to use
+         * @param ds The data set to search on
+         * @return A new search task.
+         */
+        private static SearchTask newSearchTask(SearchSetting setting, final DataSet ds, SearchReceiver resultReceiver) {
             final Collection<OsmPrimitive> selection = new HashSet<>(ds.getAllSelected());
             return new SearchTask(ds, setting, selection, new Predicate<OsmPrimitive>() {
@@ -557 +652 @@
                     return ds.isSelected(o);
                 }
-            });
+            }, resultReceiver);
         }
 
@@ -614 +709 @@
                         tr("Error"),
                         JOptionPane.ERROR_MESSAGE
-
                 );
             }
@@ -624 +718 @@
                 return;
             }
-            ds.setSelected(selection);
-            if (foundMatches == 0) {
-                final String msg;
-                final String text = Utils.shortenString(setting.text, MAX_LENGTH_SEARCH_EXPRESSION_DISPLAY);
-                if (setting.mode == SearchMode.replace) {
-                    msg = tr("No match found for ''{0}''", text);
-                } else if (setting.mode == SearchMode.add) {
-                    msg = tr("Nothing added to selection by searching for ''{0}''", text);
-                } else if (setting.mode == SearchMode.remove) {
-                    msg = tr("Nothing removed from selection by searching for ''{0}''", text);
-                } else if (setting.mode == SearchMode.in_selection) {
-                    msg = tr("Nothing found in selection by searching for ''{0}''", text);
-                } else {
-                    msg = null;
-                }
-                Main.map.statusLine.setHelpText(msg);
-                JOptionPane.showMessageDialog(
-                        Main.parent,
-                        msg,
-                        tr("Warning"),
-                        JOptionPane.WARNING_MESSAGE
-                );
-            } else {
-                Main.map.statusLine.setHelpText(tr("Found {0} matches", foundMatches));
-            }
+            resultReceiver.receiveSearchResult(ds, selection, foundMatches, setting);
         }
     }