[power-architect] r3793 committed - Updated the critic manager to mark new critics as errors so they are i...

[codesite-noreply]

Revision: 3793
Author: thomasobrie...@gmail.com
Date: Tue Jul 27 15:09:13 2010
Log: Updated the critic manager to mark new critics as errors so they are  
included as enabled in default settings.

The duplicate column name critic has been disabled to start to prevent  
unnecessary errors as most users will want that critic disabled by default.
http://code.google.com/p/power-architect/source/detail?r=3793

Modified:
 /trunk/src/main/java/ca/sqlpower/architect/ddl/critic/CriticManager.java

=======================================
---  
/trunk/src/main/java/ca/sqlpower/architect/ddl/critic/CriticManager.java	 
Fri Jul 16 07:18:35 2010
+++  
/trunk/src/main/java/ca/sqlpower/architect/ddl/critic/CriticManager.java	 
Tue Jul 27 15:09:13 2010
@@ -349,9 +349,9 @@
     public void loadDefaults() {
         Preferences prefs =  
Preferences.userNodeForPackage(CriticManager.class);
         for (CriticGrouping group : getCriticGroupings()) {
-             
group.setEnabled(prefs.getBoolean(CriticGrouping.class.getSimpleName()  
+ "." + group.getPlatformType() + ".enabled", false));
+             
group.setEnabled(prefs.getBoolean(CriticGrouping.class.getSimpleName()  
+ "." + group.getPlatformType() + ".enabled", true));
             for (CriticAndSettings settings : group.getSettings()) {
-                String severity =  
prefs.get(settings.getClass().getSimpleName() + ".severity",  
Severity.IGNORE.name());
+                String severity =  
prefs.get(settings.getClass().getSimpleName() + ".severity",  
Severity.ERROR.name());
                 settings.setSeverity(Severity.valueOf(severity));
             }
         }