Changeset 2129 in josm

Timestamp:

Sep 14, 2009 11:37:34 AM (4 years ago)

Author:

stoecker

Message:

see #3475 - minor cleanup

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/search/SearchAction.java (modified) (3 diffs)
• data/osm/Filter.java (modified) (4 diffs)
• data/osm/Filters.java (modified) (3 diffs)

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

@@ -83 +83 @@
 
         JPanel left = new JPanel(new GridBagLayout());
-
-        JTextField finput = null;
-        if(initialValues instanceof Filter){
-           JLabel fLabel = new JLabel(tr("Please enter a filter name."));
-           finput = new JTextField(((Filter)initialValues).filterName);
-           left.add(fLabel, GBC.eop());
-           left.add(finput, GBC.eop().fill(GBC.HORIZONTAL));
-        }
-
         left.add(label, GBC.eop());
         left.add(input, GBC.eop().fill(GBC.HORIZONTAL));
@@ -136 +127 @@
                 initialValues instanceof Filter ? tr("Filter") : tr("Search"),
                 new String[] {
-                   initialValues instanceof Filter ? tr("Make filter") : tr("Start Search"),
+                   initialValues instanceof Filter ? tr("Submit filter") : tr("Start Search"),
                 tr("Cancel")}
         );
@@ -151 +142 @@
                 : (remove.isSelected() ? SearchAction.SearchMode.remove : SearchAction.SearchMode.in_selection));
         if(initialValues instanceof Filter){
-           return new Filter(input.getText(), mode, caseSensitive.isSelected(), regexSearch.isSelected(), finput.getText());
+           return new Filter(input.getText(), mode, caseSensitive.isSelected(), regexSearch.isSelected());
         } else {
            return new SearchSetting(input.getText(), mode, caseSensitive.isSelected(), regexSearch.isSelected());

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

@@ -11 +11 @@
 public class Filter extends SearchSetting {
    private final String version = "1";
-   public String filterName = "";
    public Boolean filtered = false;
    public Boolean disabled = true;
@@ -19 +18 @@
        super("", SearchMode.add, false, false);
    }
-   public Filter(String text, SearchMode mode, boolean caseSensitive, boolean regexSearch, String filterName) {
+   public Filter(String text, SearchMode mode, boolean caseSensitive, boolean regexSearch) {
        super(text, mode, caseSensitive, regexSearch);
-       this.filterName = filterName;
    }
 
@@ -36 +34 @@
       caseSensitive = Boolean.parseBoolean(prfs[3]);
       regexSearch = Boolean.parseBoolean(prfs[4]);
-      filterName = prfs[5];
       filtered = Boolean.parseBoolean(prfs[6]);
       disabled = Boolean.parseBoolean(prfs[7]);
@@ -47 +44 @@
       return version + ";" + 
           text + ";" + mode + ";" + caseSensitive + ";" + regexSearch + ";" +
-          filterName + ";" + filtered + ";" + disabled + ";" + 
+          "legacy" + ";" + filtered + ";" + disabled + ";" + 
           inverted + ";" + applyForChildren;
    }

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

@@ -134 +134 @@
 
    public String getColumnName(int column){
-      String[] names = { tr("F"), tr("D"), tr("Name"), tr("C"), tr("I"), tr("M") };
+      String[] names = { tr("F"), tr("D"), tr("Text"), tr("C"), tr("I"), tr("M") };
       return names[column];
    }
@@ -159 +159 @@
                  filter();
                  return;
-         case 2: f.filterName = (String)aValue;
+         case 2: f.text = (String)aValue;
                  savePref(row);
                  return;
@@ -178 +178 @@
          case 0: return f.filtered;
          case 1: return f.disabled;
-         case 2: return f.filterName;
+         case 2: return f.text;
          case 3: return f.applyForChildren;
          case 4: return f.inverted;
          case 5:
                  switch(f.mode){
-                    case replace: return "∅";
-                    case add: return "∪";
-                    case remove: return "∖";
-                    case in_selection: return "∩";
+                    case replace: return tr("R");
+                    case add: return tr("A");
+                    case remove: return tr("D");
+                    case in_selection: return tr("F");
                  }
       }