You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by an...@apache.org on 2006/04/29 19:44:42 UTC
svn commit: r398210 -
/cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/java/org/apache/cocoon/forms/formmodel/UnionDefinition.java
Author: antonio
Date: Sat Apr 29 10:44:41 2006
New Revision: 398210
URL: http://svn.apache.org/viewcvs?rev=398210&view=rev
Log:
Remove commented legacy code. It has been commented for more than 2 years.
Modified:
cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/java/org/apache/cocoon/forms/formmodel/UnionDefinition.java
Modified: cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/java/org/apache/cocoon/forms/formmodel/UnionDefinition.java
URL: http://svn.apache.org/viewcvs/cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/java/org/apache/cocoon/forms/formmodel/UnionDefinition.java?rev=398210&r1=398209&r2=398210&view=diff
==============================================================================
--- cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/java/org/apache/cocoon/forms/formmodel/UnionDefinition.java (original)
+++ cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/main/java/org/apache/cocoon/forms/formmodel/UnionDefinition.java Sat Apr 29 10:44:41 2006
@@ -24,32 +24,6 @@
public class UnionDefinition extends AbstractContainerDefinition {
private String caseWidgetId;
- /*
- public void setDatatype(Datatype datatype) {
- if (!String.class.isAssignableFrom(datatype.getTypeClass()))
- throw new RuntimeException("Only datatype string is allowed for this widget at " + getLocation() + ".");
- super.setDatatype(datatype);
- }
-
- public void setDefault(Object value) throws Exception {
- if (!(value == null || String.class.isAssignableFrom(value.getClass())))
- throw new Exception("UnionDefinition: Default case must be supplied as a string (" + getLocation() + ")");
- if (value == null || value.equals("")) {
- if (isRequired())
- throw new Exception("UnionWidget: Union is marked required, but no default case was supplied (" + getLocation() + ")");
- this.defaultValue = "";
- } else {
- if (!hasWidget((String)value))
- throw new Exception("UnionWidget: The default value \"" + value + "\" does not match a union case (" + getLocation() + ")");
- this.defaultValue = (String)value;
- }
- }
-
- public Object getDefaultValue() {
- return defaultValue;
- }
- */
-
/**
* initialize this definition with the other, sort of like a copy constructor
*/
@@ -61,7 +35,7 @@
this.caseWidgetId = other.caseWidgetId;
} else {
- throw new Exception("Definition to inherit from is not of the right type! (at "+getLocation()+")");
+ throw new Exception("Definition to inherit from is not of the right type! (at "+ getLocation()+").");
}
}
@@ -79,3 +53,4 @@
return unionWidget;
}
}
+