You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by sg...@apache.org on 2005/08/28 13:23:21 UTC
svn commit: r263863 -
/portals/jetspeed-2/trunk/components/portal-site/src/java/org/apache/jetspeed/portalsite/impl/PortalSiteSessionContextImpl.java
Author: sgala
Date: Sun Aug 28 04:23:17 2005
New Revision: 263863
URL: http://svn.apache.org/viewcvs?rev=263863&view=rev
Log:
I managed to get a NPE out of here. Changing the comparison logic so that when the comparison throws an Exceotion it returns false instead of propagating it up.
Modified:
portals/jetspeed-2/trunk/components/portal-site/src/java/org/apache/jetspeed/portalsite/impl/PortalSiteSessionContextImpl.java
Modified: portals/jetspeed-2/trunk/components/portal-site/src/java/org/apache/jetspeed/portalsite/impl/PortalSiteSessionContextImpl.java
URL: http://svn.apache.org/viewcvs/portals/jetspeed-2/trunk/components/portal-site/src/java/org/apache/jetspeed/portalsite/impl/PortalSiteSessionContextImpl.java?rev=263863&r1=263862&r2=263863&view=diff
==============================================================================
--- portals/jetspeed-2/trunk/components/portal-site/src/java/org/apache/jetspeed/portalsite/impl/PortalSiteSessionContextImpl.java (original)
+++ portals/jetspeed-2/trunk/components/portal-site/src/java/org/apache/jetspeed/portalsite/impl/PortalSiteSessionContextImpl.java Sun Aug 28 04:23:17 2005
@@ -791,7 +791,7 @@
}
// compare control and navigation locator properties
- if ((properties0 != null) && (properties1 != null))
+ try
{
for (int i = 0, limit = properties0.length; (i < limit); i++)
{
@@ -806,6 +806,9 @@
return false;
}
}
+ } catch(Throwable t) {
+ t.printStackTrace();
+ return false;
}
}
return true;
---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org