I would say these brief bulleted points could serve as the basis for a very interesting discussion not as the end all and be all. I wonder why people seem to be so very touchy / take offense without getting into a discussion of the ramifications of these very general guidelines. They are very helpful to me as an intermediate drupalero. <br>
Tony<br><br><div class="gmail_quote">On Thu, Sep 20, 2012 at 9:18 AM, Steve Kessler <span dir="ltr"><<a href="mailto:skessler@denverdataman.com" target="_blank">skessler@denverdataman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I would agree with Patrick overall. This is a good blog post for someone but not *commandments* for the communities. <div><br></div><div>I was going to let someone else say this but it is also based somewhat on opinion. There are best practices and coding guidelines but not commandments. </div>
<div class="HOEnZb"><div class="h5">
<div><br><div class="gmail_quote">On Thu, Sep 20, 2012 at 10:00 AM, Patrick Avella <span dir="ltr"><<a href="mailto:me@patrickavella.com" target="_blank">me@patrickavella.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm not a fan of this list, and if someone tried to push that on me<br>
here I'd be visibly upset. Some of the bullet points are good, maybe<br>
even great but:<br>
<br>
<br>
2. No PHP nodes.<br>
3. No PHP blocks. (A sample block module is available.)<br>
If I want to use PHP to insert a value into a link, I will, and see no<br>
problem with that. This is left up to the developer, not all PHP code<br>
needs to be in a module. PHP filter is part of core. When used<br>
responsibly there is no reason to not use it if it fits<br>
budget/time/needs.<br>
<br>
4. Minimize PHP in Views.<br>
Depending on the scope of the project, this can't always be enforced.<br>
Sometimes it's a lost more efficient to shove a little bit of PHP in a<br>
view than to have to recreate the view (and lose its support) from<br>
scratch in a module.<br>
<br>
6. Use Features for content types and views; and for other things<br>
that lend themselves thereto. Commit these to Git repository.<br>
I don't understand the love for features. It only supports "some"<br>
important modules, and needs a host of other modules (UUID, strongarm)<br>
before it starts to become actually useful. Bloat, bloat, bloat.<br>
<br>
9. CSS is your friend, use it before programmatic or theme styling<br>
as much as possible.<br>
No one is going to not tell me to not use the theme system. If I need<br>
to rearrange elements or make theme level changes, I will do so, and<br>
will not be told otherwise.<br>
<br>
14. Links, including menus, should use relative URLs.<br>
This makes no sense. All menu links in drupal are absolute (they start<br>
at the domain root). It's a guideline where I work to never use<br>
relative URLs for files because they are not easily moved to new<br>
locations.<br>
<br>
But I knew I wouldn't like this list as soon as I saw the word<br>
"commandments". These guidelines sound like a lot of personal<br>
preference, and not a lot of realistic "in the fox hole" programming.<br>
<br>
Thanks,<br>
Patrick Avella<br>
<div><div><br>
<br>
<br>
<br>
On Thu, Sep 20, 2012 at 11:35 AM, Steve Kessler<br>
<<a href="mailto:skessler@denverdataman.com" target="_blank">skessler@denverdataman.com</a>> wrote:<br>
> Changing . htaccess is required in many cases. That is not hacking core.<br>
><br>
> I like to comment out core when editing . htaccess and then add my changes<br>
> with a comment about what they are for.<br>
><br>
> -Steve<br>
><br>
><br>
> On Thu, Sep 20, 2012 at 9:29 AM, Franz Iberl <<a href="mailto:f.iberl@amazonas-box.de" target="_blank">f.iberl@amazonas-box.de</a>><br>
> wrote:<br>
>><br>
>> Am 20.09.12 17:17, schrieb Walt Daniels:<br>
>> > Unfortunately hacking core, in particular .htaccess, is not optional on<br>
>> > some servers.<br>
>><br>
>> Adapting .htaccess I do not consider as hacking core.<br>
>><br>
>> Servus<br>
>> Franz<br>
>> --<br>
>> [ Drupal support list | <a href="http://lists.drupal.org/" target="_blank">http://lists.drupal.org/</a> ]<br>
><br>
><br>
><br>
><br>
> --<br>
> Steve Kessler<br>
> Owner and Lead Consultant<br>
> Denver DataMan, LLC<br>
> <a href="tel:303-587-4428" value="+13035874428" target="_blank">303-587-4428</a><br>
><br>
><br>
> --<br>
> [ Drupal support list | <a href="http://lists.drupal.org/" target="_blank">http://lists.drupal.org/</a> ]<br>
--<br>
[ Drupal support list | <a href="http://lists.drupal.org/" target="_blank">http://lists.drupal.org/</a> ]<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Kessler<div>Owner and Lead Consultant </div><div>Denver DataMan, LLC </div><div><a href="tel:303-587-4428" value="+13035874428" target="_blank">303-587-4428</a></div>
<br>
</div>
</div></div><br>--<br>
[ Drupal support list | <a href="http://lists.drupal.org/" target="_blank">http://lists.drupal.org/</a> ]<br></blockquote></div><br><br clear="all"><br>-- <br><p><b style="font-family:'Lucida Handwriting',cursive;font-size:13px;background-color:rgb(255,255,255)"><i>Anthony Stefan Maciejowski</i></b></p>
<p></p><br><p></p><br>