Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added filed level error message support. #259

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yankun01
Copy link

@yankun01 yankun01 commented Feb 3, 2016

fixed typo in error.

@yankun01
Copy link
Author

yankun01 commented Feb 3, 2016

@otakustay 两个简单修改。

@@ -266,6 +266,11 @@ define(

if (!result) {
var messageTemplate = checker.errorMessage;
// 当自定义错误信息存在时替换自定义错误信息
var fieldErrorMessage = fieldSchema[3];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Schema 本身并不感知 实体校验字段错误信息 这些概念。

实体验证后显示错误的模板国际化也应该在 checker 部分去扩展,直接放 Schema 上不太合适。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants