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

暴露枚举值和inField没有提示语 #46

Open
ZhangChengLin opened this issue Sep 19, 2021 · 7 comments
Open

暴露枚举值和inField没有提示语 #46

ZhangChengLin opened this issue Sep 19, 2021 · 7 comments
Assignees

Comments

@ZhangChengLin
Copy link
Contributor

'enum' => '{attr} must in ({value0})',
'notIn' => '{attr} cannot in ({value0})',

这个提示语等于暴露了允许的值。


inField 验证器没有提示语

public function inFieldValidator($val, string $anotherField): bool
{
if ($anotherField && $dict = $this->getByPath($anotherField)) {
return Validators::in($val, $dict);
}
return false;
}

@inhere inhere self-assigned this Sep 19, 2021
@inhere
Copy link
Owner

inhere commented Sep 19, 2021

可以提个PR 调整下

@ZhangChengLin
Copy link
Contributor Author

可以提个公关调整下

稍后进行。
这项目为啥没有多少星标呢?

@ZhangChengLin
Copy link
Contributor Author

inField 验证器并不是没有提示语,而是走的默认提示语,汗颜!!!
而enum 和 notIn 是有单独的提示语的,是不是有时候需要把可选值暴露出来,不过默认应该是不暴露比较安全吧?

@inhere
Copy link
Owner

inhere commented Sep 21, 2021

可以提个PR调整下

稍后进行。
这项目为啥没有多少星标呢?

没怎么推过。项目很早就有了

@inhere
Copy link
Owner

inhere commented Sep 21, 2021

inField 验证器并不是没有提示语,而是走的默认提示语,汗颜!!!
而enum 和 notIn 是有单独的提示语的,是不是有时候需要把可选值暴露出来,不过默认应该是不暴露比较安全吧?

有些场景 确实需要这个提示。对API来说,不提示确实更安全点

@ZhangChengLin
Copy link
Contributor Author

inField 验证器并不是没有提示语,而是走的默认提示语,汗颜!!!
而enum 和 notIn 是有单独的提示语的,是不是有时候需要把可选值暴露出来,不过默认应该是不暴露比较安全吧?

有些场景 确实需要这个提示。对API来说,不提示确实更安全点

那是否可以增加个选项,用来控制走提示还是走统一的否定语。

@inhere
Copy link
Owner

inhere commented Sep 22, 2021

这个倒是可以

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

No branches or pull requests

3 participants