#11652 removing field instanciation checks, moving data integrity to …

…factory
dotCMS · May 20, 2017 · ae31844 · ae31844
1 parent be82b56
commit ae31844
Show file tree

Hide file tree

Showing 7 changed files with 9 additions and 21 deletions.
diff --git a/dotCMS/src/main/java/com/dotcms/contenttype/model/field/DateField.java b/dotCMS/src/main/java/com/dotcms/contenttype/model/field/DateField.java
@@ -42,8 +42,7 @@ public abstract static class Builder implements FieldBuilder {
 
     @Value.Check
     public void check() {
-        super.check();
-
+
         Preconditions.checkArgument(new FieldUtil().validDate(defaultValue()), this.getClass().getSimpleName() + " invalid defualt Value:" + defaultValue());
 
     }

diff --git a/dotCMS/src/main/java/com/dotcms/contenttype/model/field/DateTimeField.java b/dotCMS/src/main/java/com/dotcms/contenttype/model/field/DateTimeField.java
@@ -38,7 +38,6 @@ public abstract static class Builder implements FieldBuilder {}
 
     @Value.Check
     public void check() {
-        super.check();
 
         Preconditions.checkArgument(new FieldUtil().validDateTime(defaultValue()), this.getClass().getSimpleName() + " invalid defualt Value:" + defaultValue());
 

diff --git a/dotCMS/src/main/java/com/dotcms/contenttype/model/field/HostFolderField.java b/dotCMS/src/main/java/com/dotcms/contenttype/model/field/HostFolderField.java
@@ -15,12 +15,7 @@
 @Value.Immutable
 public abstract class HostFolderField extends Field implements OnePerContentType {
 
-	@Value.Check
-	public void check() {
-		super.check();
-		if(iDate().before(legacyFieldDate))return;
-		Preconditions.checkArgument(indexed(),"Host Folder Fields must be indexed");
-	}
+
 	@Value.Default
 	@Override
 	public boolean indexed() {

diff --git a/dotCMS/src/main/java/com/dotcms/contenttype/model/field/SelectableValuesField.java b/dotCMS/src/main/java/com/dotcms/contenttype/model/field/SelectableValuesField.java
@@ -33,7 +33,7 @@ public List<SelectableValue> selectableValues() {
 
 	@Value.Check
 	public void check() {
-		super.check();
+
 		if(iDate().before(legacyFieldDate))return;
 		if(values()!=null){
 	        String[] tempVals = StringUtil.split(values().replaceAll("\r\n","|").trim(), "|");

diff --git a/dotCMS/src/main/java/com/dotcms/contenttype/model/field/TagField.java b/dotCMS/src/main/java/com/dotcms/contenttype/model/field/TagField.java
@@ -26,13 +26,7 @@ public String typeName(){
 		return LegacyFieldTypes.getLegacyName(TagField.class);
 	}
 
-	@Value.Check
-	public void check() {
-		super.check();
-		if(iDate().after(legacyFieldDate)){
-			Preconditions.checkArgument(indexed(),"Tag Fields must be indexed");
-		}
-	}
+
 	@Value.Default
 	@Override
 	public boolean indexed() {

diff --git a/dotCMS/src/main/java/com/dotcms/contenttype/model/field/TextField.java b/dotCMS/src/main/java/com/dotcms/contenttype/model/field/TextField.java
@@ -25,7 +25,7 @@ public  Class type() {
 	@Value.Derived
 	@Override
 	public List<DataTypes> acceptedDataTypes(){
-		return ImmutableList.of(DataTypes.LONG_TEXT,DataTypes.TEXT, DataTypes.FLOAT, DataTypes.INTEGER);
+		return ImmutableList.of(DataTypes.TEXT,DataTypes.LONG_TEXT, DataTypes.FLOAT, DataTypes.INTEGER);
 	}
 	@Value.Default
 	@Override

diff --git a/dotCMS/src/main/java/com/dotcms/contenttype/model/field/TimeField.java b/dotCMS/src/main/java/com/dotcms/contenttype/model/field/TimeField.java
@@ -6,6 +6,7 @@
 
 import com.dotcms.contenttype.util.FieldUtil;
 import com.dotcms.repackage.com.google.common.collect.ImmutableList;
+import com.dotmarketing.util.UtilMethods;
 import com.dotcms.repackage.com.google.common.base.Preconditions;
 import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
@@ -40,9 +41,9 @@ public abstract static class Builder implements FieldBuilder {}
 
     @Value.Check
     public void check() {
-        super.check();
-
-        Preconditions.checkArgument(new FieldUtil().validTime(defaultValue()), this.getClass().getSimpleName() + " invalid defualt Value:" + defaultValue());
 
+        if(UtilMethods.isSet(defaultValue())){
+          Preconditions.checkArgument(new FieldUtil().validTime(defaultValue()), this.getClass().getSimpleName() + " invalid defualt Value:" + defaultValue());
+        }
     }
 }