Running the Coder module on uc_ajax_cart reveals thousands of violations of the Drupal Coding Standards. This patch fixes most of the 'normal' level problems reported by Coder. However, there are hundreds of 'minor' level problems remaining. Hopefully this patch takes you a long way towards making your code clean.
Note that the patch is over 1000 lines long, because almost every single line in several files needed to be changed.
Please, PLEASE familiarize yourself with the Drupal coding standards - your style is so far outside the norm for Drupal that your code is very hard to work with and understand. Even if you don't agree with some of the choices Drupal has made with these standards, they *are* the community standards, and all code in CVS is supposed to conform to these standards.
Comment | File | Size | Author |
---|---|---|---|
#2 | uc_ajax_cart.coder_.patch | 34.98 KB | TR |
Comments
Comment #1
TR CreditAttribution: TR commentedComment #2
TR CreditAttribution: TR commentedIt would help if I remembered to attach the patch ...
Comment #3
Erik Seifert CreditAttribution: Erik Seifert commentedThanks for your patch, but i wont fix this.
Because we have our own coding conventions. Our IDE's and all other software is we use, is configurated to our standards. Its too much time and afford for me to have 2 coding standards.
If you want to participate you are welcome.
Comment #4
japerryDrupal standards exist for the community to be able to contribute. This is an easy issue to fix and critical for community participation.
If you do not want to fix or accept this patch, I'll put a request in for this module to be forked and this version of the module to be removed.
Thanks TR for the patch.
Comment #5
Erik Seifert CreditAttribution: Erik Seifert commentedComment #6
Erik Seifert CreditAttribution: Erik Seifert commentedWill be fixed in 2.0-beta1