Fixes #75 - Add ability to add PhaseListeners to AjaxLifecycle

manorrock · Nov 27, 2023 · 4ff1e97 · 4ff1e97
1 parent 84c3ab4
commit 4ff1e97
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/lifecycle/ajax/src/main/java/com/manorrock/oyena/lifecycle/ajax/AjaxLifecycle.java b/lifecycle/ajax/src/main/java/com/manorrock/oyena/lifecycle/ajax/AjaxLifecycle.java
@@ -32,6 +32,7 @@
 import jakarta.faces.event.PhaseListener;
 import jakarta.faces.lifecycle.Lifecycle;
 import jakarta.inject.Named;
+import java.util.ArrayList;
 
 /**
  * The AJAX lifecycle.
@@ -42,6 +43,11 @@
 @Named("com.manorrock.oyena.lifecycle.ajax.AjaxLifecycle")
 public class AjaxLifecycle extends Lifecycle {
 
+    /**
+     * Stores the phase listeners.
+     */
+    private ArrayList<PhaseListener> phaseListeners = new ArrayList<>();
+
     /**
      * Constructor.
      */
@@ -60,7 +66,7 @@ public AjaxLifecycle() {
      */
     @Override
     public void addPhaseListener(PhaseListener phaseListener) {
-        // phase listeners are not supported.
+        phaseListeners.add(phaseListener);
     }
 
     /**
@@ -76,16 +82,11 @@ public void execute(FacesContext facesContext) throws FacesException {
     /**
      * Get the phase listeners.
      *
-     * <p>
-     * As phase listeners are not supported by the Action lifecycle this will
-     * always return a zero length array.
-     * </p>
-     *
      * @return the empty array of phase listeners.
      */
     @Override
     public PhaseListener[] getPhaseListeners() {
-        return new PhaseListener[0];
+        return (PhaseListener[]) phaseListeners.toArray();
     }
 
     /**
@@ -99,7 +100,7 @@ public PhaseListener[] getPhaseListeners() {
      */
     @Override
     public void removePhaseListener(PhaseListener phaseListener) {
-        // phase listeners are not supported.
+        phaseListeners.remove(phaseListener);
     }
 
     /**