Use multi-argument constructor.

brettryan · Mar 11, 2016 · 83982a8 · 83982a8
1 parent fe092d8
commit 83982a8
Showing 1 changed file with 14 additions and 15 deletions.
diff --git a/...n/java/com/drunkendev/confluence/plugins/attachments/ConfigurePurgeAttachmentsAction.java b/...n/java/com/drunkendev/confluence/plugins/attachments/ConfigurePurgeAttachmentsAction.java
@@ -81,22 +81,21 @@ public String doDefault() throws Exception {
         return INPUT;
     }
 
-	@Override
-	public String execute() throws Exception {
+    @Override
+    public String execute() throws Exception {
         System.out.println("Saving settings: " + ageRuleEnabled);
-        PurgeAttachmentSettings s = new PurgeAttachmentSettings();
-        s.setAgeRuleEnabled(ageRuleEnabled);
-        s.setMaxDaysOld(maxDaysOld);
-        s.setMaxRevisions(maxRevisions);
-        s.setMaxSizeRuleEnabled(maxSizeRuleEnabled);
-        s.setMaxTotalSize(maxTotalSize);
-        s.setMode(mode);
-        s.setReportEmailAddress(reportEmailAddress);
-        s.setReportOnly(reportOnly);
-        s.setRevisionCountRuleEnabled(revisionCountRuleEnabled);
-        settingSvc.setSettings(getSpaceKey(), s);
-		return super.execute();
-	}
+        settingSvc.setSettings(getSpaceKey(),
+                               new PurgeAttachmentSettings(mode,
+                                                           ageRuleEnabled,
+                                                           maxDaysOld,
+                                                           revisionCountRuleEnabled,
+                                                           maxRevisions,
+                                                           maxSizeRuleEnabled,
+                                                           maxTotalSize,
+                                                           reportOnly,
+                                                           reportEmailAddress));
+        return super.execute();
+    }
 
     //
     // Settings properties follow