You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mm...@apache.org on 2005/07/09 15:37:39 UTC
svn commit: r209942 -
/myfaces/impl/trunk/src/java/org/apache/myfaces/el/PropertyResolverImpl.java
Author: mmarinschek
Date: Sat Jul 9 06:37:37 2005
New Revision: 209942
URL: http://svn.apache.org/viewcvs?rev=209942&view=rev
Log:
fixing MyFaces-315, thanks to Duffy Gillman
Modified:
myfaces/impl/trunk/src/java/org/apache/myfaces/el/PropertyResolverImpl.java
Modified: myfaces/impl/trunk/src/java/org/apache/myfaces/el/PropertyResolverImpl.java
URL: http://svn.apache.org/viewcvs/myfaces/impl/trunk/src/java/org/apache/myfaces/el/PropertyResolverImpl.java?rev=209942&r1=209941&r2=209942&view=diff
==============================================================================
--- myfaces/impl/trunk/src/java/org/apache/myfaces/el/PropertyResolverImpl.java (original)
+++ myfaces/impl/trunk/src/java/org/apache/myfaces/el/PropertyResolverImpl.java Sat Jul 9 06:37:37 2005
@@ -54,15 +54,18 @@
{
try
{
- if (base == null || property == null ||
- property instanceof String && ((String)property).length() == 0)
- {
- return null;
- }
- if (base instanceof Map)
- {
- return ((Map) base).get(property);
- }
+ //fix for myfaces-315 - empty string as key to a map-value is allowed
+ //thanks to duffy gillman
+ if (base == null || property == null ||
+ (property instanceof String && ((String)property).length() == 0 &&
+ !(base instanceof Map)))
+ {
+ return null;
+ }
+ if (base instanceof Map)
+ {
+ return ((Map) base).get(property);
+ }
// If none of the special bean types, then process as normal Bean
return getProperty(base, property.toString());