交换唯一属性值后,Grails验证失败

时间:2011-05-31 19:53:01

标签: grails grails-validation

交换唯一属性值后,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')
    }
}

到目前为止这个工作非常好。当我以语言交换的方式更新相同枚举值的两个枚举值转换时,我的问题出现了。例如,我有一个

  • 枚举值:“喜剧”

以及语言“意外”混淆的一些翻译

  • “喜剧”的翻译
    • 语言:德语,价值:“喜剧”
    • 语言:英语,价值“Komödie”

如果用户认识到他混淆了语言,他可能想要交换语言并再次保存枚举。这就是我的错误发生的地方,因为在交换语言后,枚举值转换唯一约束验证为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 ->

    }
}

这就是为什么我坚信必定存在一些非常重要的问题...如果您还有其他需要了解的地方,请告诉我。

感谢您的帮助

1 个答案:

答案 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。