Add parameters to composer

Author: pcastornii

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

@@ -35,10 +35,12 @@ public abstract class AbstractParentFxView<P extends ParentPresenter<?, ?>>
 
     public class Composer implements ParentFxView.Composer {
 
+        private ComposeParameters parameters = createParameters();
+
         private final AbstractParentFxView<?> view = AbstractParentFxView.this;
 
         @Override
-        public void compose(ComposeParameters params) {
+        public void compose() {
             // empty
         }
 
@@ -88,6 +90,15 @@ public String toTreeString() {
         public String toTreeString(BiConsumer<ParentPort, StringBuilder> appender) {
             return view.toTreeString(depthFirstIterator(), appender);
         }
+
+        @Override
+        public ComposeParameters getParameters() {
+            return this.parameters;
+        }
+
+        protected ComposeParameters createParameters() {
+            return null;
+        }
     }
 
     private final ObservableList<ChildFxView<?>> modifiableChildren = FXCollections.observableArrayList();

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

@@ -31,11 +31,8 @@ public abstract class AbstractParentPresenter<V extends ParentView, C extends Pa
 
     private C composer;
 
-    private ComposeParameters parameters;
-
     public AbstractParentPresenter(V view) {
         super(view);
-        this.parameters = createParameters();
     }
 
     @Override
@@ -70,25 +67,10 @@ public void requestFocus() {
     @Override
     protected void postInitialize() {
         super.postInitialize();
-        this.composer.compose(parameters);
-        this.parameters = null;
+        this.composer.compose();
     }
 
     protected void setComposer(ParentComposer composer) {
         this.composer = (C) composer;
     }
-
-    /**
-    * Creates the parameters to be passed to {@link ParentComposer#compose(ComposerParameters)} during the static
-    * composition phase. After composition completes, the instance is set to {@code null} and is no longer accessible.
-    *
-    * @return the parameters for composition, or {@code null} if no parameters are required
-    */
-    protected ComposeParameters createParameters() {
-        return null;
-    }
-
-    protected ComposeParameters getParameters() {
-        return parameters;
-    }
 }

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

@@ -17,8 +17,6 @@
 package com.techsenger.patternfx.mvp;
 
 /**
- * Marker interface for parameters passed to {@link ParentComposer#compose(Parameters)}.
- * Implementations carry initialization data required by child components during static composition.
  *
  * @author Pavel Castornii
  */

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

@@ -26,6 +26,13 @@
  */
 public interface ParentComposer {
 
+    /**
+     * Returns the parameters of the composer.
+     *
+     * @return the parameters or null.
+     */
+    ComposeParameters getParameters();
+
     /**
      * Composes the static structure of this component by creating and adding its default child components. This
      * method is intended for static composition, where child components are created once during component
@@ -36,9 +43,8 @@ public interface ParentComposer {
      * that all child components are properly initialized and integrated into the component tree before the parent
      * component starts its normal operation.
      *
-     * @param params the parameters to be used during composition, or {@code null} if no parameters are required
      */
-    void compose(ComposeParameters params);
+    void compose();
 
     /**
      * Returns an unmodifiable observable list of child components.