1 / 15

September 2011 Drupal@OSU Meetup

September 2011 Drupal@OSU Meetup. Agenda . Coder Module KMData Update from Jason Little Introductions & Open Discussion. What is Coder?. A Code Validation module for themes and Modules.

benny
Download Presentation

September 2011 Drupal@OSU Meetup

An Image/Link below is provided (as is) to download presentation Download Policy: Content on the Website is provided to you AS IS for your information and personal use and may not be sold / licensed / shared on other websites without getting consent from its author. Content is provided to you AS IS for your information and personal use only. Download presentation by click this link. While downloading, if for some reason you are not able to download a presentation, the publisher may have deleted the file from their server. During download, if you can't get a presentation, the file might be deleted by the publisher.

E N D

Presentation Transcript


  1. September 2011Drupal@OSU Meetup

  2. Agenda • Coder Module • KMData Update from Jason Little • Introductions & Open Discussion

  3. What is Coder? • A Code Validation module for themes and Modules. • From the module page:"This module utilizes regular expressions to review source code files for code that 1) needs to change due to Drupal API changes and 2) does not satisfy Drupal coding standards."

  4. What Else does coder check? • Doxygen compatible code & Drupal Coding Standards Compliance • Lightweight Security Warnings* • Upgrading between Drupal versions

  5. The Module Listing Page • Code Review link on module listing page.

  6. Coder Options • Conversion • Minor (Most)

  7. What to review • Module or tHEME Selection • Include or Exclude file types

  8. Results • Warnings • minor • normal • critical • List of warnings for each file with line numbers

  9. Basic Errors/Warnings • Comment should be read "Implements hook_foo()." • String concatenation should be formatted with a space separating the operators (dot .) and the surrounding terms • There should be no trailing spaces • Use an indent of 2 spaces, with no tabs

  10. Module Conversion • Info file should specify core=7.x • hook_perm() has been renamed to hook_permission(). (Drupal Docs) • hook_block has been split into a number of smaller hooks. (Drupal Docs) • Commits to the Git repository do not require the CVS $Id$ keyword in each file.

  11. Security Warnings* • Potential problem: when FAPI element '#type' is set to 'markup' (default), '#value' only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.

  12. ? QUESTIONS?

  13. What do you do with Themeing? • October Meetup - Planned Topic - Theming Basics • What are you doing that is out of the ordinary? • How many files are in your theme directory? • Do you use theme inheritance?

  14. KMDATA UPDATEJason Little

  15. Introductions &Open Discussion

More Related