small refactorings

src/main/java/hudson/plugins/global_build_stats/FieldFilter.java

@@ -1,14 +1,13 @@
 package hudson.plugins.global_build_stats;
 
-
 public interface FieldFilter {
 
 	boolean isFieldValueValid(String fieldValue);
 
-
 	public static final FieldFilter ALL = new FieldFilter() {
+		@Override
 		public boolean isFieldValueValid(String fieldValue) {
 			return true;
 		}
 	};
-}
+}

src/main/java/hudson/plugins/global_build_stats/GlobalBuildStatsPlugin.java

@@ -11,7 +11,7 @@
 import hudson.model.listeners.RunListener;
 import hudson.plugins.global_build_stats.business.GlobalBuildStatsBusiness;
 import hudson.plugins.global_build_stats.model.*;
-import hudson.plugins.global_build_stats.rententionstrategies.RetentionStragegy;
+import hudson.plugins.global_build_stats.rententionstrategies.RetentionStrategy;
 import hudson.plugins.global_build_stats.validation.GlobalBuildStatsValidator;
 import hudson.security.Permission;
 import hudson.util.ChartUtil;
@@ -21,7 +21,6 @@
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
-//import java.util.logging.Logger;
 
 import javax.servlet.ServletException;
 
@@ -45,8 +44,6 @@
 @ExportedBean
 public class GlobalBuildStatsPlugin extends Plugin {
 
-//    private static final Logger LOGGER = Logger.getLogger(GlobalBuildStatsPlugin.class.getName());
-
     /**
      * List of aggregated job build results
      * This list will grow over time, but will be monthly sharded in different files to keep
@@ -71,7 +68,7 @@ public class GlobalBuildStatsPlugin extends Plugin {
     /**
      * List of retention strategies applied on job results
      */
-    private List<RetentionStragegy> retentionStrategies = new ArrayList<RetentionStragegy>();
+    private List<RetentionStrategy> retentionStrategies = new ArrayList<RetentionStrategy>();
 
 	/**
 	 * Business layer for global build stats
@@ -382,10 +379,10 @@ public void doMoveDownConf(StaplerRequest req, StaplerResponse res) throws Servl
     public void doUpdateRetentionStrategies(StaplerRequest req, StaplerResponse res) throws ServletException, IOException {
         Hudson.getInstance().checkPermission(getRequiredPermission());
 
-        List<RetentionStragegy> selectedStrategies = new ArrayList<RetentionStragegy>();
+        List<RetentionStrategy> selectedStrategies = new ArrayList<RetentionStrategy>();
         if(req.getParameterValues("retentionStrategies") != null){
             for(String selectedStrategyId : req.getParameterValues("retentionStrategies")){
-                RetentionStragegy retentionStrategy = RetentionStragegy.valueOf(selectedStrategyId);
+                RetentionStrategy retentionStrategy = RetentionStrategy.valueOf(selectedStrategyId);
                 retentionStrategy.updateState(req.getParameterMap());
                 selectedStrategies.add(retentionStrategy);
             }
@@ -464,19 +461,19 @@ public void reloadJobBuildResults(List<JobBuildResult> results) {
         this.jobBuildResultsSharder = new JobBuildResultSharder(this.jobBuildResultsSharder, results);
     }
 
-    public List<RetentionStragegy> getAvailableRetentionStrategies(){
-        return RetentionStragegy.values();
+    public List<RetentionStrategy> getAvailableRetentionStrategies(){
+        return RetentionStrategy.values();
     }
 
     public boolean isStrategySelected(String strategyId){
-        return retentionStrategies.contains(RetentionStragegy.valueOf(strategyId));
+        return retentionStrategies.contains(RetentionStrategy.valueOf(strategyId));
     }
 
-    public void setRetentionStrategies(List<RetentionStragegy> retentionStrategies) {
+    public void setRetentionStrategies(List<RetentionStrategy> retentionStrategies) {
         this.retentionStrategies = retentionStrategies;
     }
 
-    public List<RetentionStragegy> getRetentionStrategies() {
+    public List<RetentionStrategy> getRetentionStrategies() {
         return retentionStrategies;
     }
 }

src/main/java/hudson/plugins/global_build_stats/business/GlobalBuildStatsBusiness.java

@@ -10,7 +10,7 @@
 import hudson.plugins.global_build_stats.JobBuildResultFactory;
 import hudson.plugins.global_build_stats.model.*;
 import hudson.plugins.global_build_stats.model.AbstractBuildStatChartDimension.LegendItemData;
-import hudson.plugins.global_build_stats.rententionstrategies.RetentionStragegy;
+import hudson.plugins.global_build_stats.rententionstrategies.RetentionStrategy;
 import hudson.plugins.global_build_stats.util.CollectionsUtil;
 import hudson.util.DataSetBuilder;
 import hudson.util.ShiftedCategoryAxis;
@@ -53,12 +53,12 @@ public GlobalBuildStatsBusiness(GlobalBuildStatsPlugin _plugin){
      * Records the result of a build.
      */
 	public void onJobCompleted(final AbstractBuild build) {
-        for(RetentionStragegy s : plugin.getRetentionStrategies()){
+        for(RetentionStrategy s : plugin.getRetentionStrategies()){
             s.onBuildCompleted(build, pluginSaver);
         }
         this.pluginSaver.updatePlugin(new GlobalBuildStatsPluginSaver.BeforeSavePluginCallback(){
+        	@Override
             public void changePluginStateBeforeSavingIt(GlobalBuildStatsPlugin plugin) {
-
                 plugin.getJobBuildResultsSharder().queueResultToAdd(JobBuildResultFactory.INSTANCE.createJobBuildResult(build));
             }
         });
@@ -92,6 +92,7 @@ private int searchBuildStatConfigIndexById(String id){
 	public void recordBuildInfos() throws IOException {
 
         this.pluginSaver.updatePlugin(new GlobalBuildStatsPluginSaver.BeforeSavePluginCallback(){
+        	@Override
             public void changePluginStateBeforeSavingIt(GlobalBuildStatsPlugin plugin) {
 
                 List<JobBuildResult> jobBuildResultsRead = new ArrayList<JobBuildResult>();
@@ -121,7 +122,7 @@ public void handleItem(List<JobBuildResult> results, TopLevelItem item){
 
 	public JFreeChart createChart(BuildStatConfiguration config){
 		List<AbstractBuildStatChartDimension> dimensions = createDataSetBuilder(config);
-        return createChart(config, dimensions, config.getBuildStatTitle());
+        return createChart(dimensions, config.getBuildStatTitle());
 	}
 
 	public List<JobBuildSearchResult> searchBuilds(BuildHistorySearchCriteria searchCriteria){
@@ -145,6 +146,7 @@ public void updateBuildStatConfiguration(final String oldBuildStatId,
                                              final boolean regenerateId) throws IOException {
 
         this.pluginSaver.updatePlugin(new GlobalBuildStatsPluginSaver.BeforeSavePluginCallback(){
+        	@Override
             public void changePluginStateBeforeSavingIt(GlobalBuildStatsPlugin plugin) {
                 if(regenerateId){
                     String newBuildStatId = ModelIdGenerator.INSTANCE.generateIdForClass(BuildStatConfiguration.class);
@@ -233,7 +235,7 @@ public static String escapeAntiSlashes(String value){
 		}
 	}
 
-    private JFreeChart createChart(final BuildStatConfiguration config, List<AbstractBuildStatChartDimension> dimensions, String title) {
+    private JFreeChart createChart(List<AbstractBuildStatChartDimension> dimensions, String title) {
 
     	final JFreeChart chart = ChartFactory.createStackedAreaChart(title, null, "", 
     			new DataSetBuilder<String, DateRange>().build(), PlotOrientation.VERTICAL, true, true, false);
@@ -298,12 +300,12 @@ private static LegendItemCollection sortLegendItems(LegendItemCollection legendI
     	return sortedLegendItems;
     }
 
-    public void updateRetentionStrategies(final List<RetentionStragegy> selectedStrategies) {
+    public void updateRetentionStrategies(final List<RetentionStrategy> selectedStrategies) {
         this.pluginSaver.updatePlugin(new GlobalBuildStatsPluginSaver.BeforeSavePluginCallback(){
             @Override
             public void changePluginStateBeforeSavingIt(GlobalBuildStatsPlugin plugin) {
                 plugin.setRetentionStrategies(selectedStrategies);
-                for(RetentionStragegy s : selectedStrategies){
+                for(RetentionStrategy s : selectedStrategies){
                     s.strategyActivated(pluginSaver);
                 }
             }
@@ -393,7 +395,7 @@ public void reloadPlugin() {
         this.pluginSaver.reloadPlugin();
 
         // If job results are empty, let's perform an initialization !
-        if(this.plugin.getJobBuildResults()==null || this.plugin.getJobBuildResults().size() == 0){
+        if(this.plugin.getJobBuildResults()==null || this.plugin.getJobBuildResults().isEmpty()){
             try {
                 this.recordBuildInfos();
             } catch (IOException e) {
@@ -403,7 +405,7 @@ public void reloadPlugin() {
     }
 
     public void onBuildDeleted(AbstractBuild build) {
-        for(RetentionStragegy s : plugin.getRetentionStrategies()){
+        for(RetentionStrategy s : plugin.getRetentionStrategies()){
             s.onBuildDeleted(build, pluginSaver);
         }
     }

src/main/java/hudson/plugins/global_build_stats/business/GlobalBuildStatsPluginSaver.java

@@ -6,7 +6,7 @@
 import hudson.plugins.global_build_stats.rententionstrategies.DiscardResultsOlderThanDays;
 import hudson.plugins.global_build_stats.rententionstrategies.DoNotKeepBuildResultWhenDiscarded;
 import hudson.plugins.global_build_stats.rententionstrategies.KeepExistingJobResultsOnly;
-import hudson.plugins.global_build_stats.rententionstrategies.RetentionStragegy;
+import hudson.plugins.global_build_stats.rententionstrategies.RetentionStrategy;
 import hudson.plugins.global_build_stats.xstream.GlobalBuildStatsXStreamConverter;
 import hudson.security.Permission;
 import hudson.util.DaemonThreadFactory;
@@ -119,7 +119,7 @@ public void reloadPlugin() {
      */
     public void updatePlugin(BeforeSavePluginCallback callback){
         final List<BuildStatConfiguration> configsBeforeStateChange = new ArrayList<BuildStatConfiguration>(plugin.getBuildStatConfigs());
-        final List<RetentionStragegy> retentionStrategiesBeforeStateChange = new ArrayList<RetentionStragegy>(plugin.getRetentionStrategies());
+        final List<RetentionStrategy> retentionStrategiesBeforeStateChange = new ArrayList<RetentionStrategy>(plugin.getRetentionStrategies());
         callback.changePluginStateBeforeSavingIt(plugin);
         LOGGER.log(Level.FINER, "Global build stats state update queued !");
 

src/main/java/hudson/plugins/global_build_stats/model/JobBuildResult.java

@@ -13,7 +13,7 @@
 public class JobBuildResult implements Serializable {
 
 	private static final long serialVersionUID = -4697202185011561179L;
-	public static long EMPTY_DURATION = -1;
+	public static final long EMPTY_DURATION = -1;
 	public static final String EMPTY_NODE_NAME = null;
 	public static final String EMPTY_USER_NAME = null;
 	public static final String MASTER_NODE_NAME = "master";

src/main/java/hudson/plugins/global_build_stats/rententionstrategies/DiscardResultsOlderThanDays.java

@@ -17,7 +17,7 @@
 /**
  * @author fcamblor
  */
-public class DiscardResultsOlderThanDays extends RetentionStragegy<DiscardResultsOlderThanDays>
+public class DiscardResultsOlderThanDays extends RetentionStrategy<DiscardResultsOlderThanDays>
                 implements ParameterizedStrategy<DiscardResultsOlderThanDays>, BuildCompletedListener {
 
     private static final long PURGE_FREQUENCY = 1000L * 3600L * 24L; // Let's purge job build results once a day

src/main/java/hudson/plugins/global_build_stats/rententionstrategies/DoNotKeepBuildResultWhenDiscarded.java

@@ -10,7 +10,7 @@
 /**
  * @author fcamblor
  */
-public class DoNotKeepBuildResultWhenDiscarded extends RetentionStragegy<DoNotKeepBuildResultWhenDiscarded>
+public class DoNotKeepBuildResultWhenDiscarded extends RetentionStrategy<DoNotKeepBuildResultWhenDiscarded>
                 implements BuildDeletedListener {
 
     @Override

src/main/java/hudson/plugins/global_build_stats/rententionstrategies/KeepExistingJobResultsOnly.java

@@ -12,7 +12,7 @@
 /**
  * @author fcamblor
  */
-public class KeepExistingJobResultsOnly extends RetentionStragegy<KeepExistingJobResultsOnly> {
+public class KeepExistingJobResultsOnly extends RetentionStrategy<KeepExistingJobResultsOnly> {
     @Override
     public String getConfigPage() {
         return "keepExistingJobResultsOnly.jelly";

src/main/java/hudson/plugins/global_build_stats/rententionstrategies/RetentionStragegy.java → src/main/java/hudson/plugins/global_build_stats/rententionstrategies/RetentionStrategy.java

@@ -13,15 +13,15 @@
 /**
  * @author fcamblor
  */
-public abstract class RetentionStragegy<T extends RetentionStragegy> {
+public abstract class RetentionStrategy<T extends RetentionStrategy> {
 
-    protected static final List<RetentionStragegy> RETENTION_STRATEGIES_IMPLEMENTATIONS = new ArrayList<RetentionStragegy>(){{
+    protected static final List<RetentionStrategy> RETENTION_STRATEGIES_IMPLEMENTATIONS = new ArrayList<RetentionStrategy>(){{
         add(new DiscardResultsOlderThanDays());
         add(new DoNotKeepBuildResultWhenDiscarded());
         add(new KeepExistingJobResultsOnly());
     }};
 
-    protected RetentionStragegy(){
+    protected RetentionStrategy(){
     }
 
     public String getId(){
@@ -30,8 +30,8 @@ public String getId(){
 
     public abstract String getConfigPage();
 
-    public static RetentionStragegy valueOf(String strategyId){
-        for(RetentionStragegy strategy : RETENTION_STRATEGIES_IMPLEMENTATIONS){
+    public static RetentionStrategy valueOf(String strategyId){
+        for(RetentionStrategy strategy : RETENTION_STRATEGIES_IMPLEMENTATIONS){
             if(strategyId.equals(strategy.getId())){
                 return strategy;
             }
@@ -65,7 +65,7 @@ public void onBuildCompleted(AbstractBuild build, GlobalBuildStatsPluginSaver pl
         }
     }
 
-    public static List<RetentionStragegy> values(){
+    public static List<RetentionStrategy> values(){
         return RETENTION_STRATEGIES_IMPLEMENTATIONS;
     }
 

src/main/java/hudson/plugins/global_build_stats/rententionstrategies/strategybehaviours/ParameterizedStrategy.java

@@ -1,13 +1,13 @@
 package hudson.plugins.global_build_stats.rententionstrategies.strategybehaviours;
 
-import hudson.plugins.global_build_stats.rententionstrategies.RetentionStragegy;
+import hudson.plugins.global_build_stats.rententionstrategies.RetentionStrategy;
 
 import java.util.Map;
 
 /**
  * @author fcamblor
  */
-public interface ParameterizedStrategy<T extends RetentionStragegy> {
+public interface ParameterizedStrategy<T extends RetentionStrategy> {
     public void updateStrategyParameters(Map<String, String[]> parameters);
     public void updateStrategyParameters(T strategyToCopy);
 }