http://git.drupal.org/project/github_projects.git 7.x-1.x-dev

Git errors:

  • The following git branches do not match the release branch pattern, you should remove/rename them. See https://www.drupal.org/node/1015226

    * 7.x-1.x-dev
    remotes/origin/7.x-1.x-dev

Review of the 7.x-1.x-dev branch (commit d6f4142):

  • Remove the LICENSE, drupal.org packaging will add a LICENSE.txt file automatically.
  • Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
  • DrupalPractice has found some issues with your code, but could be false positives.

    FILE: ...eviewsh/pareview_temp/src/Classes/GithubProjectsConnection.class.php
    --------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    --------------------------------------------------------------------------
    13 | WARNING | Class name must be prefixed with the project name
    | | "GithubProjects"
    --------------------------------------------------------------------------

    FILE: .../pareview_temp/src/Classes/GithubProjectsRemoteSelectQuery.class.php
    --------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    --------------------------------------------------------------------------
    353 | WARNING | Unused variable $readme.
    --------------------------------------------------------------------------

    Time: 88ms; Memory: 6Mb

  • No automated test cases were found, did you consider writing Simpletests or PHPUnit tests? This is not a requirement but encouraged for professional software development.

This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.

FILE: ...oot/repos/pareviewsh/pareview_temp/src/Theme/github_projects.tpl.php
--------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------
1 | ERROR | [x] Missing file doc comment
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------

FILE: ...eviewsh/pareview_temp/src/Classes/GithubProjectsConnection.class.php
--------------------------------------------------------------------------
FOUND 9 ERRORS AFFECTING 6 LINES
--------------------------------------------------------------------------
13 | ERROR | Class name must begin with a capital letter
13 | ERROR | Class name must use UpperCamel naming without underscores
29 | ERROR | Public method name
| | "clients_connection_our_rest::remote_entity_load" is not
| | in lowerCamel format
51 | ERROR | Public method name
| | "clients_connection_our_rest::remote_entity_save" is not
| | in lowerCamel format
51 | ERROR | Type hint "array" missing for $remote_properties
68 | ERROR | Public method name
| | "clients_connection_our_rest::entity_property_type_map" is
| | not in lowerCamel format
123 | ERROR | Type hint "array" missing for $method_params
167 | ERROR | Type hint "array" missing for $header_options
167 | ERROR | Type hint "array" missing for $data
--------------------------------------------------------------------------

FILE: .../pareview_temp/src/Classes/GithubProjectsRemoteSelectQuery.class.php
--------------------------------------------------------------------------
FOUND 5 ERRORS AND 1 WARNING AFFECTING 6 LINES
--------------------------------------------------------------------------
23 | ERROR | Class property $retrieve_multiple should use lowerCamel
| | naming without underscores
37 | ERROR | Class property $from_date should use lowerCamel naming
| | without underscores
44 | ERROR | Class property $to_date should use lowerCamel naming
| | without underscores
51 | ERROR | Class property $user_id should use lowerCamel naming
| | without underscores
59 | WARNING | Possible useless method overriding detected
400 | ERROR | Public method name
| | "GithubProjectsRemoteSelectQuery::buildFromEFQ" is not
| | in lowerCamel format
--------------------------------------------------------------------------

Time: 182ms; Memory: 8Mb