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

Lack of Input validation #204

Open
cydh opened this issue Jul 16, 2018 · 4 comments
Open

Lack of Input validation #204

cydh opened this issue Jul 16, 2018 · 4 comments

Comments

@cydh
Copy link
Contributor

cydh commented Jul 16, 2018

I don't see any built-in or easy-to-use or function for input validation in flux.
What I found if there's input (POST or GET) passed by $this->params all you need to make self-validation.

How about https://github.com/Respect/Validation ?

As example,

Requirement

  • PHP 5.4+ or HHVM 3.3+ are required. (so README.md will be 5.3 to 5.4, I run this test on PHP 7.2 and in other localhost I have 5.4 so can't test for 5.3)
@hurtsky
Copy link
Contributor

hurtsky commented Jul 16, 2018

+1

@cydh
Copy link
Contributor Author

cydh commented Jul 19, 2018

Talk about respect/validation, it requires PHP 5.4 from its 1.1 version installed by composer. And it can used directly like this Flux with Respect Validation

@aleos89
Copy link
Contributor

aleos89 commented Jul 19, 2018

Seems like a very useful component to Flux.

@cydh
Copy link
Contributor Author

cydh commented Jul 19, 2018

and maybe a little workaround for Styling Error Message (<-- gist edited)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: New Items
Development

No branches or pull requests

4 participants