coder 5.x-2.7

douggreen - February 19, 2008 - 12:21
Official release from CVS tag: DRUPAL-5--2-7
Download: coder-5.x-2.7.tar.gz
Size: 45.98 KB
md5_file hash: a7194e8b5fece01bd05daf40616fa41b
First released: February 19, 2008 - 12:21
Last updated: February 19, 2008 - 12:25

Per the conversation on the devel thread, I'm creating new latest releases of several modules.

Changes since DRUPAL-5--2-6:

  • fix coding style violation involving quoted strings
  • only display link to review help if it exists
  • correct name of sql review, and add it to the defaults
  • #161215 by sun - fix class inheritance
  • backport 6.x rules
  • #173260 - fix url/l checks to include $vars as arguments
  • 173472 reported by snpower - catch drupal_retrieve_form api changes
  • add list as an exception to the function rule
  • add security check for php vars inside SQL
  • slightly change SQL injection warning and add description help
  • the coder cache is not longer "experimental" so remove this word from the settings form
  • #177246 - fix api.drupal.org version links
  • #180066 by snpower - add check for node/add
  • doh - this wasn't ready to commit
  • #184106 by shaal - fix syntax error
  • #180226 - added rule to check for unquoted array references
  • some minor coding style fixes and also a fix for the rule warning text being
    matched by the rule itself.
  • #189823 - comment settings are now per node type
  • #189805 - node_access_rebuild() should not be called from within
    hook_enable/disable functions in node access modules in D6
  • #189774 - admin/logs has been renamed to admin/reports
  • #189794 - The 'administer access control' permission has been renamed to
    'administer permissions' and the path 'admin/user/access' has changed to
    'admin/user/permissions'.
  • #189786 - Support for form property #DANGEROUS_SKIP_CHECK has been removed in
    D6.
  • #189767 - The locale_refresh_cache() function is now deprecated in D6,
    cache_clear_all() should be used instead.
  • #189763 - db_next_id() is now deprecated in D6 and db_last_insert_id() should be
    used instead.
  • #189756 - menu_set_location() is deprecated in D6 and menu_set_item() or
    drupal_set_breadcrumb() should be used instead.
  • taxonomy_get_vocabulary() changed to taxonomy_vocabulary_load() in D6
  • $row argument removed from db_result() method in D6
  • Module authors who use the watchdog() function with $type = 'debug' are strongly
    encouraged to replace that with $severity WATCHDOG_DEBUG for consistency.
  • Many of the book module API functions have changed -
    http://drupal.org/node/114774#book_module
  • #178629 - added in checks for incorrect @see usage
  • links for 2 deprecated functions were pointing at the api docs for D6 instead of
    D5
  • #177246 - Correct api.drupal.org links
  • #195054 - incorrect error message for one of the 5.x to 6.x API changes.
  • add rule to check for Submit
  • add rule to look for hook_nodeapi submit
  • #180235 by snpower
  • add #not check to exclude array() when looking for args by comma
  • added in fix for db_result() api change - it incorrectly flagged valid
    db_result() calls that contained db_query()
  • changes for valid uses of 'comment_form_location'
  • Changed db_result() warning to use #warning_callback.
    Fixed a typo for confirm_submit warning and changed hook_help warning to also
    match on the word "function" before it.
  • #173508 - fixed issue where there was no checking on install file when a schema
    file was present, etc.
  • #189744 - check for drupal_mail() 5.x to 6.x api changes
  • restoring some changes lost in the last check-in, doh!
  • #170548 - restrict camelCase check to variables and function declarations
  • #170548 - restrict camelCase check to variables and function declarations
  • #173508 - replace basename with filename in drupal_system_listing
  • add #never flag so we can prevent matches of <?php on lines that don't indent properly
  • fix #never 'continue'
  • only catch quotes around a string literal array index with lower case literals -- because define constants are ok, and define constants should be upper case
  • #203380 - add catch to list of control statements
  • theme modules and themes in 3 rows for easier viewing/checking
  • fix the calculation of the column totals in theme_coder_checkboxes
  • #207830 by grugnog2 - Fix $_POST hack in form submission
  • #149826 by ezyang/sun - add coder format windows installer
  • Fixed PHPdoc of coder_order_processors().
  • #210828 - add if ($may_cache) to hook_menu
  • #212195 by greggles - use warning that links to more help
  • remove trailing spaces
  • #200880 - added php function documentation based on ezyang's work for the 6.x
    version.
  • new includes/tests files taken from 6.x version. The only change is the
    addition of php documentation from ezyang's work in #200880.
  • #212737 by snpower - Warn about trailing spaces and missing newlines
  • remove some extra spaces, change the trailing spaces error to severity=minor
  • #212737 - only check for trailing spaces or tabs, not newlines and carriage returns
  • #219417 by csevb10 - fix error counts
  • #221432 by sun - fix token url replacement in one of the rule messages
  • #222405 by webchick - harsher warning for t() in watchdog in 6.x check
  • make sure all references to functions in warning messages have () after them, as in !t()
  • #222599 by webchick/ChrisKennedy - in menu upgrade help provide a more specific link to menu documentation
 
 

Drupal is a registered trademark of Dries Buytaert.