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/21 18:09:20 UTC
svn commit: r1125737 -
/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/binder/SetPropertiesBuilder.java
Author: simonetripodi
Date: Sat May 21 16:09:20 2011
New Revision: 1125737
URL: http://svn.apache.org/viewvc?rev=1125737&view=rev
Log:
removed wrong check, null 'propertyName' is admitted
Modified:
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/binder/SetPropertiesBuilder.java
Modified: commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/binder/SetPropertiesBuilder.java
URL: http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/binder/SetPropertiesBuilder.java?rev=1125737&r1=1125736&r2=1125737&view=diff
==============================================================================
--- commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/binder/SetPropertiesBuilder.java (original)
+++ commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/binder/SetPropertiesBuilder.java Sat May 21 16:09:20 2011
@@ -49,18 +49,11 @@ public final class SetPropertiesBuilder
{
if ( attributeName == null )
{
- reportError( "setProperties().addAlias( String,String ) }", "empty 'attributeName' not allowed" );
+ reportError( "setProperties().addAlias( String,String )", "empty 'attributeName' not allowed" );
}
else
{
- if ( propertyName == null )
- {
- reportError( "setProperties().addAlias( String,String ) }", "empty 'propertyName' not allowed" );
- }
- else
- {
- aliases.put( attributeName, propertyName );
- }
+ aliases.put( attributeName, propertyName );
}
return this;
}