Что не надо делать при разработке сайта на Drupal

Список худших практик в друпале:

1. Игнорирование coding standards. Самая распространённая ошибка как новичков, так и бывалых. Стандарты кодирования - первое что должен выучить друпал разработчик.

2. Транслит в машинных именах. Когда разработчик называет что-то в духе novosti, field_razdel, razmeri, где-то умирает котик.

3. Много отключённых контриб модулей. Не надо держать на продакшене неиспользуемые модули.

4. Много включённых ненужных контриб модулей. Как правило это говорит, что сайт собирался на коленке.

5. Отключённый watchdog / отключённый вывод ошибок / ошибки в watchdog / ошибки на экране. Неоднократно видел, как разработчики отключали вывод ошибок, чтобы не разбираться в их причинах.

6. Использование hook_form_alter(). Всегда пользуйтесь hook_form_FORM_ID_alter() и hook_form_BASE_FORM_ID_alter().

7. Русский текст в t(). Это бессмысленно и вредно.

8. Отсутствие PHPDoc комментариев к функциям и методам. Каждая функция и метод обязаны содержать комментарий.

9. Имена функций без префиксов модуля/темы. get_products() - плохо, modulename_get_products() - хорошо.

10. Длинные селекторы в css. #main-menu div.content ul.menu li a { ...; } - плохо, #main-menu a { ...; } - хорошо.

11. Использование тяжёлых базовых тем типа Bootstrap или Omega. Поначалу кажется, что они могут сэкономить время, но это ощущение обманчиво.

12. Сложная логика в шаблонах. Всю логику сложнее foreach/if нужно выносить в препроцессинг.

13. Множество переопределённых шаблонов. Их тяжело поддерживать.

14. Лишние файлы в корне. Все пользовательские файлы надо заливать в sites/default/files.

15. Помойка в sites/default/files. Все файлы должны быть рассортированы по подпапкам - sites/default/files/images/products, sites/default/files/inline и т.п.

16. Использование jQuery.ready(). В 99% нужно пользоваться Behaviors.

17. Использование темы для админки отличной от Seven. Seven - стандарт.

18. Сложная логика в коде, написанном через админку (phpfilter и т.п.). Максимум что там может быть - вызов своей функции.

19. Самописные модули в одной папке с контрибом. Все кастомные модули должны быть в своей папке.