You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mb...@apache.org on 2005/09/05 09:53:45 UTC
svn commit: r278687 -
/myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java
Author: mbr
Date: Mon Sep 5 00:53:40 2005
New Revision: 278687
URL: http://svn.apache.org/viewcvs?rev=278687&view=rev
Log:
fix for MYFACES-493 and MYFACES-494
Modified:
myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java
Modified: myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java
URL: http://svn.apache.org/viewcvs/myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java?rev=278687&r1=278686&r2=278687&view=diff
==============================================================================
--- myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java (original)
+++ myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java Mon Sep 5 00:53:40 2005
@@ -377,6 +377,10 @@
public void addPhaseListener(PhaseListener phaseListener)
{
+ if(phaseListener == null)
+ {
+ throw new NullPointerException("PhaseListener must not be null.");
+ }
if (_phaseListenerList == null)
{
_phaseListenerList = new ArrayList();
@@ -391,6 +395,10 @@
public void removePhaseListener(PhaseListener phaseListener)
{
+ if(phaseListener == null)
+ {
+ throw new NullPointerException("PhaseListener must not be null.");
+ }
if (_phaseListenerList == null)
{
_phaseListenerList = new ArrayList();