You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cayenne.apache.org by nt...@apache.org on 2019/09/20 19:37:25 UTC
[cayenne] branch STABLE-4.1 updated: cleanup
This is an automated email from the ASF dual-hosted git repository.
ntimofeev pushed a commit to branch STABLE-4.1
in repository https://gitbox.apache.org/repos/asf/cayenne.git
The following commit(s) were added to refs/heads/STABLE-4.1 by this push:
new 5a1484e cleanup
5a1484e is described below
commit 5a1484e6337610a9be220f1825818e3142fc639d
Author: Nikita Timofeev <st...@gmail.com>
AuthorDate: Fri Sep 20 22:37:17 2019 +0300
cleanup
---
.../modeler/editor/cgen/GeneratorController.java | 100 ++++-----------------
1 file changed, 17 insertions(+), 83 deletions(-)
diff --git a/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/editor/cgen/GeneratorController.java b/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/editor/cgen/GeneratorController.java
index 58bc85e..2243531 100644
--- a/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/editor/cgen/GeneratorController.java
+++ b/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/editor/cgen/GeneratorController.java
@@ -99,33 +99,29 @@ public abstract class GeneratorController extends CayenneController {
}
private ValidationFailure validateEmbeddableAttribute(EmbeddableAttribute attribute) {
- String name = attribute.getEmbeddable().getClassName();
+ return validateAttribute(attribute.getEmbeddable().getClassName(), attribute.getName(), attribute.getType());
+ }
- ValidationFailure emptyName = BeanValidationFailure.validateNotEmpty(name, "attribute.name",
- attribute.getName());
+ private ValidationFailure validateAttribute(String objectName, String attributeName, String attributeType) {
+ ValidationFailure emptyName = BeanValidationFailure.validateNotEmpty(objectName, "attribute.name",
+ attributeName);
if (emptyName != null) {
return emptyName;
}
- ValidationFailure badName = CodeValidationUtil.validateJavaIdentifier(name, "attribute.name",
- attribute.getName());
+ ValidationFailure badName = CodeValidationUtil.validateJavaIdentifier(objectName, "attribute.name",
+ attributeName);
if (badName != null) {
return badName;
}
- ValidationFailure emptyType = BeanValidationFailure.validateNotEmpty(name, "attribute.type",
- attribute.getType());
+ ValidationFailure emptyType = BeanValidationFailure.validateNotEmpty(objectName, "attribute.type",
+ attributeType);
if (emptyType != null) {
return emptyType;
}
- ValidationFailure badType = BeanValidationFailure.validateJavaClassName(name, "attribute.type",
- attribute.getType());
- if (badType != null) {
- return badType;
- }
-
- return null;
+ return BeanValidationFailure.validateJavaClassName(objectName, "attribute.type", attributeType);
}
private ValidationFailure validateEmbeddable(Embeddable embeddable) {
@@ -138,13 +134,7 @@ public abstract class GeneratorController extends CayenneController {
return emptyClass;
}
- ValidationFailure badClass = BeanValidationFailure.validateJavaClassName(name, "className",
- embeddable.getClassName());
- if (badClass != null) {
- return badClass;
- }
-
- return null;
+ return BeanValidationFailure.validateJavaClassName(name, "className", embeddable.getClassName());
}
public void validateEntity(ValidationResult validationBuffer, ObjEntity entity, boolean clientValidation) {
@@ -204,45 +194,14 @@ public abstract class GeneratorController extends CayenneController {
}
if (entity.getSuperClassName() != null) {
- ValidationFailure badSuperClass = BeanValidationFailure.validateJavaClassName(name, "superClassName",
- entity.getSuperClassName());
- if (badSuperClass != null) {
- return badSuperClass;
- }
+ return BeanValidationFailure.validateJavaClassName(name, "superClassName", entity.getSuperClassName());
}
return null;
}
private ValidationFailure validateAttribute(ObjAttribute attribute) {
-
- String name = attribute.getEntity().getName();
-
- ValidationFailure emptyName = BeanValidationFailure.validateNotEmpty(name, "attribute.name",
- attribute.getName());
- if (emptyName != null) {
- return emptyName;
- }
-
- ValidationFailure badName = CodeValidationUtil.validateJavaIdentifier(name, "attribute.name",
- attribute.getName());
- if (badName != null) {
- return badName;
- }
-
- ValidationFailure emptyType = BeanValidationFailure.validateNotEmpty(name, "attribute.type",
- attribute.getType());
- if (emptyType != null) {
- return emptyType;
- }
-
- ValidationFailure badType = BeanValidationFailure.validateJavaClassName(name, "attribute.type",
- attribute.getType());
- if (badType != null) {
- return badType;
- }
-
- return null;
+ return validateAttribute(attribute.getEntity().getName(), attribute.getName(), attribute.getType());
}
private ValidationFailure validateEmbeddedAttribute(ObjAttribute attribute) {
@@ -269,29 +228,7 @@ public abstract class GeneratorController extends CayenneController {
return badEmbeddedName;
}
- ValidationFailure emptyName = BeanValidationFailure.validateNotEmpty(name, "attribute.name", attr);
- if (emptyName != null) {
- return emptyName;
- }
-
- ValidationFailure badName = CodeValidationUtil.validateJavaIdentifier(name, "attribute.name", attr);
- if (badName != null) {
- return badName;
- }
-
- ValidationFailure emptyType = BeanValidationFailure.validateNotEmpty(name, "attribute.type",
- attribute.getType());
- if (emptyType != null) {
- return emptyType;
- }
-
- ValidationFailure badType = BeanValidationFailure.validateJavaClassName(name, "attribute.type",
- attribute.getType());
- if (badType != null) {
- return badType;
- }
-
- return null;
+ return validateAttribute(name, attr, attribute.getType());
}
private ValidationFailure validateRelationship(ObjRelationship relationship, boolean clientValidation) {
@@ -328,11 +265,8 @@ public abstract class GeneratorController extends CayenneController {
return emptyClass;
}
- ValidationFailure badClass = BeanValidationFailure.validateJavaClassName(name,
- "relationship.targetEntity.className", targetEntity.getClassName());
- if (badClass != null) {
- return badClass;
- }
+ return BeanValidationFailure.validateJavaClassName(name, "relationship.targetEntity.className",
+ targetEntity.getClassName());
}
}
@@ -342,7 +276,7 @@ public abstract class GeneratorController extends CayenneController {
/**
* Returns a predicate for default entity selection in a given mode.
*/
- public Predicate getDefaultClassFilter() {
+ public Predicate<Object> getDefaultClassFilter() {
return object -> {
if (object instanceof ObjEntity) {
return getParentController().getProblem(((ObjEntity) object).getName()) == null;