Jerome Nadaud Posted April 1, 2015 Share Posted April 1, 2015 Hi, We are discussing the possibility of changing PrestaShop development norm to PSR2+. This would affect developers of the Community (in a good way, we think), so if you do contribute to PrestaShop's ecosystem with PR's, modules, themes, etc... your vote is important! Thank you. Jérôme. 4 Link to comment Share on other sites More sharing options...
Xavier Borderie Posted April 1, 2015 Share Posted April 1, 2015 To make sure that everyone understands what this means: PSR-2 is a coding style guide standard published by the PHP Framework Interoperability Group (or PHP-FIG), an informal group of representatives from major PHP projects, which offers a place to "talk about the commonalities between our projects and find ways we can work together" (see the FAQ). The group issues a series of standards (PSRs, or PHP Standard Recommendation) on how to best build a PHP project. PSR-2 establishes a coding style guide. It builds upon PSR-1's "Basic Coding standard", which PrestaShop follows closely (but not entirely) thanks to its Coding Standard. Using PSR-2 (and thus PSR-1) means that we will slightly change our Coding Standard to follow PHP-FIG's norm. The code behavior itself will not change much: mostly, only the way the code is written will change, and since it's already close to those standards, the impact should be minimal. Still, choosing the follow PSR-2 will imply some code reformatting here and there, and will make sure that our open-source code will be more likely to be used by other PSR-2-using projects! If you think this is a bad idea, please comment! 5 Link to comment Share on other sites More sharing options...
hpar Posted April 3, 2015 Share Posted April 3, 2015 I don't think it's fondamentally a bad idea to change coding style. But coding style isn't the problem when you start reading Prestashop code. There is other areas where spending time will improve developer experience. 5 Link to comment Share on other sites More sharing options...
Xavier Borderie Posted April 3, 2015 Share Posted April 3, 2015 But coding style isn't the problem when you start reading Prestashop code. There is other areas where spending time will improve developer experience. Choosing to follow PSR-2 from now will certainly not detract our developers from the time they spend impriving the code and adding features (see the Core Weekly for a proof of that). If anything, it will push them to review code that hasn't changed for a long time because it had no notable issue, and will thus give fresh pair of eyes on those lines of code. So in the end, we might even see more bugs fixed while doing this Link to comment Share on other sites More sharing options...
hpar Posted April 3, 2015 Share Posted April 3, 2015 (edited) If anything, it will push them to review code that hasn't changed for a long time because it had no notable issue, and will thus give fresh pair of eyes on those lines of code. So in the end, we might even see more bugs fixed while doing this There is no need to understand code when you review coding style, so I doubt there will be much improvement / refactoring. But if the objective is : and will make sure that our open-source code will be more likely to be used by other PSR-2-using projects! then coding style is not very relevant. Prestashop (even parts of it) is hard to interact with / integrate in other projects. Because it's monolithic, slow, hard to debug, not documented, and some parts are poorly designed. Note: I have nothing against PSR-2, it will add some consistency in code, and it will be very usefull the day prestashop will include other components from the community (PSR-3 Loggers, PSR-4 autoloading). I just see it as a "Nice to Have". Edited April 3, 2015 by hpar (see edit history) 4 Link to comment Share on other sites More sharing options...
PhpMadman Posted April 7, 2015 Share Posted April 7, 2015 If I understood it correctly. All ifs / functions etc must end with }. Meaning all one line ifs should be changed to using braces { } That would re-factor alot of files... But i'm all for this. 1 Link to comment Share on other sites More sharing options...
gytske Posted April 10, 2015 Share Posted April 10, 2015 I'd like to see that. Standards = cleaner code! Also, on a similar topic: https://github.com/airbnb/javascript This is unofficial, but widely accepted. JS is a little bit messy at the moment 2 Link to comment Share on other sites More sharing options...
olea Posted April 15, 2015 Share Posted April 15, 2015 Please cancel your own coding standard and use a common one. If anything, it will push them to review code that hasn't changed for a long time because it had no notable issue, and will thus give fresh pair of eyes on those lines of code. So in the end, we might even see more bugs fixed while doing this Take care that refactoring the code may fix some bugs but also introduce new ones Have you all the test cases to check you don't have regressions ? 1 Link to comment Share on other sites More sharing options...
wayann Posted April 20, 2015 Share Posted April 20, 2015 +1 for PSR-2 1 Link to comment Share on other sites More sharing options...
Raphaël Malié Posted May 2, 2015 Share Posted May 2, 2015 It's always a good idea to improve coding standards on an open source project, but it's far to be a priority right now IMO. + what about modules validation ? 1 Link to comment Share on other sites More sharing options...
tuk66 Posted May 5, 2015 Share Posted May 5, 2015 This would affect developers of the Community (in a good way, we think), so if you do contribute to PrestaShop's ecosystem with PR's, modules, themes, etc... your vote is important! . In general I am in, but this also means that you will force to rewrite all modules for the second time, because of the great Validator. Nice!!!! At least some competing developers will leave. 1 Link to comment Share on other sites More sharing options...
Recommended Posts