[D6] Authored Blog

Automated report:

It appears you are working in the "HEAD" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.
Review of the HEAD branch:

The drush command 'coder-review no-empty minor comment i18n security [error]
sql style sites/all/modules/pareview_temp/test_candidate' could not
be found.
• README.txt is missing, see the guidelines for in-project documentation.

This automated report was generated with PAReview.sh, your friendly project application review script. Go and review some other project applications, so we can get back to yours sooner.

A rough look:

• Please remove ;$id$ tags, they are deprecated.
• You did not use doxygen the right way (eg. in parameter descriptions), see http://drupal.org/node/1354.
• Don't use space between index's (dunno if it's forbidden but nobody actually does it) $form_state ['values'] ['blog'] ['blog_content_type'] => $form_state ['values']['blog']['blog_content_type']
• Introduce hooks like "Implements hook_help().", also http://drupal.org/node/1354

I'm shure here's more to do, but you'r on the right track! :)

Review of the master branch:

• Run coder to check your style, some issues were found (please check the Drupal coding standards):
  

  Severity minor, Drupal Commenting Standards, Internationalization, Drupal Security Checks, Drupal SQL Standards, Drupal Coding Standards
  
  sites/all/modules/pareview_temp/test_candidate/authored_blog.module:
   +2: [minor] There should be no trailing spaces
   +9: [minor] There should be no trailing spaces
   +19: [minor] There should be no trailing spaces
   +30: [minor] There should be no trailing spaces
   +58: [minor] There should be no trailing spaces
   +59: [minor] There should be no trailing spaces
   +60: [minor] There should be no trailing spaces
   +64: [minor] There should be no trailing spaces
   +104: [minor] There should be no trailing spaces
   +118: [minor] There should be no trailing spaces
   +121: [minor] There should be no trailing spaces
   +124: [minor] There should be no trailing spaces
   +125: [minor] There should be no trailing spaces
   +146: [minor] There should be no trailing spaces
   +169: [minor] There should be no trailing spaces
   +172: [minor] There should be no trailing spaces
   +196: [minor] There should be no trailing spaces
   +275: [minor] Use an indent of 2 spaces, with no tabs
   +290: [minor] There should be no trailing spaces
   +291: [minor] There should be no trailing spaces
   +404: [minor] There should be no trailing spaces
   +525: [minor] There should be no trailing spaces
  
  Status Messages:
   Coder found 1 projects, 1 files, 22 minor warnings, 0 warnings were flagged to be ignored
  

• README.txt is missing, see the guidelines for in-project documentation.
• @file doc block is missing in the module file, see http://drupal.org/node/1354#files .
• Comments: there should be a space after "//", see http://drupal.org/node/1354#inline
  

  authored_blog.module:292:    //there to avoid AHAH+Validation action-rewrite.
  

• ./authored_blog.module: all functions should be prefixed with your module/theme name to avoid name clashes. See http://drupal.org/node/318#naming
  

  function get_user_refernce($type) {
  function blogs_user_reference_callback() {
  function _get_blogs($uid) {
  function _get_users_list() {
  function blogs_select_lists() {
  

• ./authored_blog.module: The description on the line after the @param/@return documentation is either missing or not formatted correctly. See http://drupal.org/node/1354#functions
  

  61- *  $form_state
  65- *  Array with form attributes
  142- *  $from
  

• Bad line endings were found, always use unix style terminators. See http://drupal.org/coding-standards#indenting
  

  ./authored_blog.info:    ASCII text, with CRLF line terminators
  authored_blog.info
  authored_blog.install
  

• Do not use t() in hook_schema(), this will only generate overhead for translators.
  

        'description' => t('The base table for nodes.'),
              'description' => t('The primary identifier for a node.'),
  

• All text files should end in a single newline (\n). See http://drupal.org/node/318#indenting
  

  ./authored_blog.js ./authored_blog.install ./authored_blog.info
  

This automated report was generated with PAReview.sh, your friendly project application review script. Go and review some other project applications, so we can get back to yours sooner.

Duplicate project application located at http://drupal.org/node/1339350