交换唯一属性值后,Grails验证失败
嗨,我正在尝试创建一个界面,用户可以使用不同语言的翻译创建一些自定义枚举。例如,用户可以创建枚举“电影类型”。对于此枚举,可能存在枚举值“喜剧”,其中可能存在多个语言的一个或多个枚举值转换。
由于特定语言只能有一个翻译,因此我在枚举值转换域类中添加了一个唯一约束。这些是我的域类:
class Enumeration {
String label
List<EnumerationValue> enumerationValues = new ArrayList<EnumerationValue>()
static hasMany = [ enumerationValues: EnumerationValue ]
static constraints = {
label(nullable: false, blank: false)
enumerationValues(nullable: true)
}
}
class EnumerationValue {
String label
List<EnumerationValueTranslation> enumerationValueTranslations = new ArrayList<EnumerationValueTranslation>()
static belongsTo = [ enumeration: Enumeration ]
static hasMany = [ enumerationValueTranslations: EnumerationValueTranslation ]
static constraints = {
label(nullable: false, blank: false, unique: 'enumeration')
enumeration(nullable: false)
enumerationValueTranslations(nullable: false)
}
}
class EnumerationValueTranslation {
String value
Language language
static belongsTo = [ enumerationValue: EnumerationValue ]
static constraints = {
value(nullable: false, blank: false)
language(nullable: true, unique: 'enumerationValue')
enumerationValue(nullable: false)
/* unique constraint as mentioned in description text */
language(unique: 'enumerationValue')
}
}
到目前为止这个工作非常好。当我以语言交换的方式更新相同枚举值的两个枚举值转换时,我的问题出现了。例如,我有一个
以及语言“意外”混淆的一些翻译
如果用户认识到他混淆了语言,他可能想要交换语言并再次保存枚举。这就是我的错误发生的地方,因为在交换语言后,枚举值转换唯一约束验证为false。
为了调试这个,我只是尝试在处理params之前和之后打印出导致翻译的错误,所以:
Enumeration enumeration = Enumeration.get(params['id']);
println "before:"
enumeration.enumerationValues.each() { enumValue ->
enumValue.enumerationValueTranslations.each() { enumValueTr ->
println enumValueTr;
if(!enumValueTr.validate()) {
// print errors...
}
}
}
// swap languages:
// (this are the lines of codes that are actually executed, and cause the
// error. The actual processing of params looks different of course...)
// sets the language of "Comedy" to English
EnumerationValueTranslation.get(5).language = Language.get(1);
// sets the language of "Komödie" to German
EnumerationValueTranslation.get(6).language = Language.get(2);
println "after:"
enumeration.enumerationValues.each() { enumValue ->
enumValue.enumerationValueTranslations.each() { enumValueTr ->
println enumValueTr;
if(!enumValueTr.validate()) {
// print errors...
}
}
}
结果是:
before:
EnumerationValueTranslation(value: Fantasy, language: en_US, enumerationValue: Fantasy)
EnumerationValueTranslation(value: Phantasie, language: de_DE, enumerationValue: Fantasy)
EnumerationValueTranslation(value: Comedy, language: de_DE, enumerationValue: Comedy)
EnumerationValueTranslation(value: Komödie, language: en_US, enumerationValue: Comedy)
after:
EnumerationValueTranslation(value: Fantasy, language: en_US, enumerationValue: Fantasy)
EnumerationValueTranslation(value: Phantasie, language: de_DE, enumerationValue: Fantasy)
EnumerationValueTranslation(value: Comedy, language: en_US, enumerationValue: Comedy)
validation fails: Property [language] of class [Translation] with value [Language(code: en_US)] must be unique
EnumerationValueTranslation(value: Komödie, language: de_DE, enumerationValue: Comedy)
validation fails: Property [language] of class [Translation] with value [Language(code: de_DE)] must be unique
在这种状态下我要删除,或以任何方式保存(或以任何方式刷新) - 这只是改变对象后的结果。正如您所看到的,实际数据确实没有不一致,验证不应该失败。
我改变翻译的方式可能有错吗?我只是通过ID获取它们并简单地更新了语言 - 我在一个简约的例子中尝试了它并且它在那里工作... 它也可以工作,如果我只是创建所有枚举值和枚举值转换的深层副本并存储它(这意味着验证真的不应该失败),但我认为这真的不是应该的方式做...
另一个奇怪的事情是,如果我遍历数据,验证只会失败。如果我根本不接触数据,则不会发生错误,但数据也不会被保存,这意味着下面的行会导致验证被验证:
enumeration.enumerationValues.each() { ev ->
ev.enumerationValueTranslations.each() { evt ->
}
}
这就是为什么我坚信必定存在一些非常重要的问题...如果您还有其他需要了解的地方,请告诉我。
感谢您的帮助
答案 0 :(得分:3)
让我再试一次:))
我在看UniqueConstraint.processValidate()
,可以假设其逻辑不考虑交换案例。
特别是代码
boolean reject = false;
if (id != null) {
Object existing = results.get(0);
Object existingId = null;
try {
existingId = InvokerHelper.invokeMethod(existing, "ident", null);
}
catch (Exception e) {
// result is not a domain class
}
if (!id.equals(existingId)) {
reject = true;
}
}
else {
reject = true;
}
应迭代获得的results
并验证字段值 STILL 是否违反唯一性。在交换的情况下,应该从缓存中选择另一个实例并具有新的字段值。
所以我建议你创建一个UniqueConstraint
的自己的后代并使用它,除非有人要修补Grails。