Refactor history

Author: pcastornii

patternfx-core/src/main/java/com/techsenger/patternfx/core/ComponentHistory.java

@@ -27,16 +27,16 @@
 public interface ComponentHistory extends Serializable {
 
     /**
-     * Returns whether this history instance is fresh, meaning it was newly created and has not yet been used to
-     * save or restore any component state. A fresh history contains no previously stored data and should not be
+     * Returns whether this history instance is new, meaning it was newly created and has not yet been used to
+     * save or restore any component state. A new history contains no previously stored data and should not be
      * passed to a component for restoration.
      *
      * <p>Once the component's state has been saved into this history, the flag becomes {@code false}, indicating that
      * the history now holds valid data that can be used to restore the component's state in future instances.
      *
-     * @return {@code true} if this history is fresh and has not yet been used, {@code false} otherwise
+     * @return {@code true} if this history is new and has not yet been used, {@code false} otherwise
      */
-    boolean isFresh();
+    boolean isNew();
 
     /**
      * Method called before the component is serialized. This can be used to prepare the object's state

patternfx-mvp/src/main/java/com/techsenger/patternfx/mvp/AbstractHistory.java

@@ -24,11 +24,11 @@
  */
 public abstract class AbstractHistory implements ComponentHistory {
 
-    private boolean fresh = true;
+    private boolean isNew = true;
 
     @Override
-    public boolean isFresh() {
-        return fresh;
+    public boolean isNew() {
+        return isNew;
     }
 
     @Override
@@ -41,7 +41,7 @@ public void preSerialize() {
 
     }
 
-    void setFresh(boolean fresh) {
-        this.fresh = fresh;
+    void setNew(boolean value) {
+        this.isNew = value;
     }
 }

patternfx-mvp/src/main/java/com/techsenger/patternfx/mvp/AbstractPresenter.java

@@ -161,8 +161,8 @@ protected final void restoreHistory() {
         var policy = getHistoryPolicy();
         logger.debug("{} History policy during restore: {}", getDescriptor().getLogPrefix(), policy);
         if (policy != NONE) {
-            if (history.isFresh()) {
-                logger.debug("{} History is fresh. Skipping restoration", getDescriptor().getLogPrefix());
+            if (history.isNew()) {
+                logger.debug("{} History is new. Skipping restoration", getDescriptor().getLogPrefix());
             } else {
                 switch (policy) {
                     case DATA:
@@ -223,7 +223,7 @@ protected final void saveHistory() {
      *
      */
     protected void saveData() {
-        getHistory().setFresh(false);
+        getHistory().setNew(false);
     }
 
     /**
@@ -232,7 +232,7 @@ protected void saveData() {
      *
      */
     protected void saveAppearance() {
-        getHistory().setFresh(false);
+        getHistory().setNew(false);
     }
 
     protected abstract Descriptor createDescriptor();

patternfx-mvvm/src/main/java/com/techsenger/patternfx/mvvm/AbstractHistory.java

@@ -24,11 +24,11 @@
  */
 public abstract class AbstractHistory implements ComponentHistory {
 
-    private boolean fresh = true;
+    private boolean isNew = true;
 
     @Override
-    public boolean isFresh() {
-        return fresh;
+    public boolean isNew() {
+        return isNew;
     }
 
     @Override
@@ -41,7 +41,7 @@ public void preSerialize() {
 
     }
 
-    void setFresh(boolean fresh) {
-        this.fresh = fresh;
+    void setNew(boolean value) {
+        this.isNew = value;
     }
 }

patternfx-mvvm/src/main/java/com/techsenger/patternfx/mvvm/AbstractViewModel.java

@@ -101,8 +101,8 @@ protected final void restoreHistory() {
         var policy = getHistoryPolicy();
         logger.debug("{} History policy during restore: {}", getDescriptor().getLogPrefix(), policy);
         if (policy != NONE) {
-            if (history.isFresh()) {
-                logger.debug("{} History is fresh. Skipping restoration", getDescriptor().getLogPrefix());
+            if (history.isNew()) {
+                logger.debug("{} History is new. Skipping restoration", getDescriptor().getLogPrefix());
             } else {
                 switch (policy) {
                     case DATA:
@@ -175,7 +175,7 @@ protected final void saveHistory() {
      * @param viewModel
      */
     protected void saveData() {
-        getHistory().setFresh(false);
+        getHistory().setNew(false);
     }
 
     /**
@@ -185,7 +185,7 @@ protected void saveData() {
      * @param viewModel
      */
     protected void saveAppearance() {
-        getHistory().setFresh(false);
+        getHistory().setNew(false);
     }
 
     protected abstract Descriptor createDescriptor();

patternfx-mvvmx/src/main/java/com/techsenger/patternfx/mvvmx/AbstractComponentViewModel.java

@@ -95,8 +95,8 @@ protected final void restoreHistory() {
         var policy = getHistoryPolicy();
         logger.debug("{} History policy during restore: {}", getMediator().getLogPrefix(), policy);
         if (policy != NONE) {
-            if (history.isFresh()) {
-                logger.debug("{} History is fresh. Skipping restoration", getMediator().getLogPrefix());
+            if (history.isNew()) {
+                logger.debug("{} History is new. Skipping restoration", getMediator().getLogPrefix());
             } else {
                 switch (policy) {
                     case DATA:
@@ -169,7 +169,7 @@ protected final void saveHistory() {
      * @param viewModel
      */
     protected void saveData() {
-        getHistory().setFresh(false);
+        getHistory().setNew(false);
     }
 
     /**
@@ -179,7 +179,7 @@ protected void saveData() {
      * @param viewModel
      */
     protected void saveAppearance() {
-        getHistory().setFresh(false);
+        getHistory().setNew(false);
     }
 
     void prepareHistory() {

patternfx-mvvmx/src/main/java/com/techsenger/patternfx/mvvmx/AbstractHistory.java

@@ -24,10 +24,11 @@
  */
 public abstract class AbstractHistory implements ComponentHistory {
 
-    private boolean fresh = true;
+    private boolean isNew = true;
 
-    public boolean isFresh() {
-        return fresh;
+    @Override
+    public boolean isNew() {
+        return isNew;
     }
 
     @Override
@@ -40,7 +41,7 @@ public void preSerialize() {
 
     }
 
-    void setFresh(boolean fresh) {
-        this.fresh = fresh;
+    void setNew(boolean value) {
+        this.isNew = value;
     }
 }