You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2011/05/22 14:23:34 UTC
svn commit: r1125939 -
/incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java
Author: simonetripodi
Date: Sun May 22 12:23:34 2011
New Revision: 1125939
URL: http://svn.apache.org/viewvc?rev=1125939&view=rev
Log:
just added {} blocks
Modified:
incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java
Modified: incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java
URL: http://svn.apache.org/viewvc/incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java?rev=1125939&r1=1125938&r2=1125939&view=diff
==============================================================================
--- incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java (original)
+++ incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java Sun May 22 12:23:34 2011
@@ -74,7 +74,9 @@ public class ListPropertyAccessor
}
if ( name instanceof Number )
+ {
return list.get( ( (Number) name ).intValue() );
+ }
if ( name instanceof DynamicSubscript )
{
@@ -82,13 +84,21 @@ public class ListPropertyAccessor
switch ( ( (DynamicSubscript) name ).getFlag() )
{
case DynamicSubscript.FIRST:
+ {
return len > 0 ? list.get( 0 ) : null;
+ }
case DynamicSubscript.MID:
+ {
return len > 0 ? list.get( len / 2 ) : null;
+ }
case DynamicSubscript.LAST:
+ {
return len > 0 ? list.get( len - 1 ) : null;
+ }
case DynamicSubscript.ALL:
+ {
return new ArrayList( list );
+ }
}
}
@@ -140,7 +150,9 @@ public class ListPropertyAccessor
case DynamicSubscript.ALL:
{
if ( !( value instanceof Collection ) )
+ {
throw new OgnlException( "Value must be a collection" );
+ }
list.clear();
list.addAll( (Collection<?>) value );
return;
@@ -215,7 +227,9 @@ public class ListPropertyAccessor
Method m = OgnlRuntime.getReadMethod( target.getClass(), indexStr );
if ( m != null )
+ {
return super.getSourceAccessor( context, target, index );
+ }
}
catch ( Throwable t )