Pursuant Posted May 4, 2011 Share Posted May 4, 2011 Prestateam, I love you guys. This product is cool, its great, and its free. But the DAMN CODE is awful. Absolutely awful. Figuring out how to code modules around your constantly changing formats is all I do. I spend 80% of my time reverse engineering this script in an attempt to get around the HORRIBLE lack of documentation. The new changes from 1.3 to 1.4 are frightening. FrontController is extended by every other controller, but it has functions called from other scripts (if its a base, why should you EVER call FrontController::anything()????).There isn't even Computer Generated documentation for this script. There is nothing. NOTHING. I'm very upset with the direction this script has taken. It has an active record pattern that still uses 30 line long SQL queries. It is the most procedural code I've ever seen that still uses classes. If a class is a model for a database, it shouldn't contain static functions. That is all that Prestashop uses.I have considered branching off of prestashop because of how its being managed and coded. It is impossible to code in. I wish I had someone to talk to about this because I have so much to say. But, alas, only a quiet forum awaits me.I can hope that by Prestashop 2.0 it is a COMPLETE rewrite. If it is not, there is no hope in this project.Thats all.--Kevin Klika Link to comment Share on other sites More sharing options...
Thomas Didierjean Posted May 5, 2011 Share Posted May 5, 2011 We are currently refactoring big chunks of the code base, but as you can imagine it's not easy to do while maintaining as much backwards compatibility as possible. That said, PrestaShop 1.5 code base will be a clear improvement and we even intend to provide documentation! Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now