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

Client validation #9

Open
vityapro opened this issue Nov 7, 2016 · 7 comments
Open

Client validation #9

vityapro opened this issue Nov 7, 2016 · 7 comments

Comments

@vityapro
Copy link

vityapro commented Nov 7, 2016

If user use custom id for fild your client validation vil work inkorrect, can you fix it ?

@sdlins
Copy link
Owner

sdlins commented Nov 29, 2016

@vityapro I have been very busy nowadays, unfortunately. Could you show a real example? Or still better: could you make a PR? It would be very appreciated.

@vityapro
Copy link
Author

vityapro commented Nov 29, 2016

Here a small examle:
Developer use active form fild, and for sum rison hi set for this fild custom id

field($user, 'address')->textInput(['id' => 'custom_id'])?>

And in this case your function clientValidateAttribute() file: AtLeastValidator.php strin: 116 (client side validation) will work wron.

@sdlins
Copy link
Owner

sdlins commented Dec 2, 2016

@code-on-yii/core any help here (when you have some time) would be really great!

@vityapro
Copy link
Author

vityapro commented Dec 3, 2016

Ok, but I want to warn that the free time I have not much.

@sdlins
Copy link
Owner

sdlins commented Dec 3, 2016

No problem @vityapro. Just let me know if you start to code here to avoid duplicated efforts. Thanx.

@vityapro
Copy link
Author

vityapro commented Dec 3, 2016

Ok, but the repo @code-on-yii/core is private, or you meant repo : @code-on-yii/yii2-at-least-validator ?

@sdlins
Copy link
Owner

sdlins commented Dec 3, 2016

@vityapro core is not a repo, is the way used to mention all core code-on-yii members easily. 👍

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

2 participants