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

toSailsAttribute bug -- > bad result for views #16

Open
Islam-Baraka opened this issue Feb 14, 2018 · 1 comment
Open

toSailsAttribute bug -- > bad result for views #16

Islam-Baraka opened this issue Feb 14, 2018 · 1 comment

Comments

@Islam-Baraka
Copy link
Contributor

Hello here are many bugs I found in the view creating process from the mysql compiler
1- the ID field shouldn't be exported to any view
2- the return array view_content have many bugs
a. the indexOf searching method will return many issues like searching for time resolve to text and then datetime will be set also to text not datetime as time is part of datetime string
b. boolean in MySql will be set automatically to tinyint<1> or int<1> so perhaps we need to check for this value/length to return checkbox
c. not really sure as i'm writing this while i'm discovering more issues
3- the view for create/edit didn't prepare for any element like textarea

i will work on a patch for this issues

@juliandavidmr
Copy link
Owner

juliandavidmr commented Feb 14, 2018

@Islam-Baraka Hello, thank you so much for being part of the open source community. Sails Inverse Model is a project that requires more features, including many improvements, as you may have noticed, there are many bugs because the tests performed were run on only a few computers, and relatively small databases along with few data types. The structure of the controllers and views is only a basic schema that does not cover security aspects, SIM only takes care of generating a simple view that has a structure that the user must adjust to their needs, such as security, etc. Any suggestion, issue, pull request... will be welcome.

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

No branches or pull requests

2 participants