Changeset 3671 in josm for trunk/src/org/openstreetmap/josm/gui/preferences/ValidatorPreference.java

Timestamp:

2010-11-25T09:45:38+01:00 (13 years ago)

Author:

bastiK

Message:

adapt coding style (to some degree); there shouldn't be any semantic changes in this commit

File:

• trunk/src/org/openstreetmap/josm/gui/preferences/ValidatorPreference.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/gui/preferences/ValidatorPreference.java

@@ -23 +23 @@
  * @author frsantos
  */
-public class ValidatorPreference implements PreferenceSetting
-{
+public class ValidatorPreference implements PreferenceSetting {
 
     public static class Factory implements PreferenceSettingFactory {
+        @Override
         public PreferenceSetting createPreferenceSetting() {
             return new ValidatorPreference();
@@ -66 +66 @@
     private Collection<Test> allTests;
 
+    @Override
     public void addGui(PreferenceTabbedPane gui)
     {
@@ -89 +90 @@
 
         allTests = OsmValidator.getTests();
-        for(Test test: allTests)
-        {
+        for (Test test: allTests) {
             test.addGui(testPanel);
         }
@@ -103 +103 @@
     }
 
-    public boolean ok()
-    {
+    @Override
+    public boolean ok() {
         StringBuilder tests = new StringBuilder();
         StringBuilder testsBeforeUpload = new StringBuilder();
-        Boolean res = false;
 
-        for (Test test : allTests)
-        {
-            if(test.ok())
-                res = false;
+        for (Test test : allTests) {
             String name = test.getClass().getSimpleName();
-            tests.append( ',' ).append( name ).append( '=' ).append( test.enabled );
-            testsBeforeUpload.append( ',' ).append( name ).append( '=' ).append( test.testBeforeUpload );
+            tests.append(',').append(name).append('=').append(test.enabled);
+            testsBeforeUpload.append(',').append(name).append('=').append(test.testBeforeUpload);
         }
 
-        if (tests.length() > 0 ) tests = tests.deleteCharAt(0);
-        if (testsBeforeUpload.length() > 0 ) testsBeforeUpload = testsBeforeUpload.deleteCharAt(0);
+        if (tests.length() > 0) {
+            tests = tests.deleteCharAt(0);
+        }
+        if (testsBeforeUpload.length() > 0) {
+            testsBeforeUpload = testsBeforeUpload.deleteCharAt(0);
+        }
 
-        OsmValidator.initializeTests( allTests );
+        OsmValidator.initializeTests(allTests);
 
-        Main.pref.put( PREF_TESTS, tests.toString());
-        Main.pref.put( PREF_TESTS_BEFORE_UPLOAD, testsBeforeUpload.toString());
-        Main.pref.put( PREF_USE_IGNORE, prefUseIgnore.isSelected());
-        Main.pref.put( PREF_OTHER_UPLOAD, prefOtherUpload.isSelected());
-        Main.pref.put( PREF_LAYER, prefUseLayer.isSelected());
+        Main.pref.put(PREF_TESTS, tests.toString());
+        Main.pref.put(PREF_TESTS_BEFORE_UPLOAD, testsBeforeUpload.toString());
+        Main.pref.put(PREF_USE_IGNORE, prefUseIgnore.isSelected());
+        Main.pref.put(PREF_OTHER_UPLOAD, prefOtherUpload.isSelected());
+        Main.pref.put(PREF_LAYER, prefUseLayer.isSelected());
         return false;
     }