Note that hook_requirements is not really usable anyway in D5 (see issues about it), so it would be misleading to include this as a requirement when it can't be usefully implemented. ----- Original Message ----- From: "Nancy Wichmann" <nan_wich@bellsouth.net> To: "Development@Drupal. Org" <development@drupal.org> Sent: Monday, May 19, 2008 3:14 PM Subject: [development] Coding Standards / Hook_requirements
I've just taken another quick run through the Coding standards because of a module I looked at last night. The new module states right on its project page that it requires PHP 5 (as more and more modules are doing). Yet, in looking through the code, it does not include a hook_requirements. I propose adding to the Coding Standards:
"Any module that has requirements over and above base Drupal standards (e.g. PHP 5 vs. 4.3.5 for D6) must include a hook_requirements implementation to verify and/or warn users of this requirement."
Unfortunately, I don't think Coder would be able to check this.
Nancy