Reply to comment

 

Bravo!

I sometimes feel like I'm stepping out-of-line when pointing out some of the things you mentioned here to seasoned module developers, so I rarely do it. After all, they've been developing longer than I. They've been involved with Drupal longer than I. They should know better than I what is correct and acceptable and what is not. Am I being the "snot-nosed, know-it-all kid" (in experience, not age) when I point out the Drupal coding guidelines, or the discussions on the development mail list where these decisions are made? Don't these people already know all this stuff and where to find them if they need a refresher? I know I frequently need reminding, and constantly return to the source to verify or disprove what I think is correct.

Might I also add that following the guidelines not only helps with quality, but it also helps with readability for those times when "patch?" is uttered? If I feel like I am going to have to spend a lot more time with reading through the code in order to make a change or three (because of the lack of coding style or following the guidelines), I am less likely to go through the effort, especially if I feel like my effort won't be accepted or maintained.

I have heard it said that working in contrib is a way to be involved with Drupal without having to comply with the coding guidelines. For me, however, it's the starting point of what my code should look like and strive to achieve.

Reply

The content of this field is kept private and will not be shown publicly.
  • Web page addresses and e-mail addresses turn into links automatically.
  • Allowed HTML tags: <a> <em> <strong> <cite> <code> <blockquote> <ul> <ol> <li>
  • Lines and paragraphs break automatically.