Merge pull request #217 from localheinz/feature/phpmd

Enhancement: Run phpmd on Travis

.travis.yml

@@ -9,6 +9,7 @@ install:
 
 script:
   - ./vendor/bin/phpcs --standard=./phpcs.xml -np --report=summary .
+  - ./vendor/bin/phpmd . text codesize,unusedcode --exclude data,puphpet,vendor
   - ./vendor/bin/phpunit --configuration phpunit.xml
 
 notifications:

composer.json

@@ -17,6 +17,7 @@
         "socalnick/scn-social-auth": "1.14.1"
     },
     "require-dev": {
+        "phpmd/phpmd": "~2.1",
         "phpunit/phpunit": "4.0.*",
         "bjyoungblood/BjyProfiler": "1.1.0",
         "squizlabs/php_codesniffer": "~2.0.0@RC",

module/User/src/User/Module.php

@@ -8,6 +8,12 @@
 
 class Module extends AbstractModule
 {
+    /**
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
+     *
+     * @param ModuleManager $moduleManager
+     * @param ApplicationInterface $app
+     */
     public function bootstrap(ModuleManager $moduleManager, ApplicationInterface $app)
     {
         $em = $app->getEventManager()->getSharedManager();

module/User/src/User/View/Helper/UserRepositories.php

@@ -29,13 +29,9 @@ class UserRepositories extends AbstractHelper implements ServiceLocatorAwareInte
     protected $events;
 
     /**
-     * __invoke
-     *
-     * @access public
-     * @param array $options array of options
-     * @return string
+     * @return array
      */
-    public function __invoke($options = array())
+    public function __invoke()
     {
         $sl = $this->getServiceLocator();
 

module/ZfModule/src/ZfModule/Mapper/Module.php

@@ -77,6 +77,9 @@ public function findByLike($query, $limit = null, $orderBy = null, $sort = 'ASC'
         return $entity;
     }
 
+    /**
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
+g     */
     public function findByOwner($owner, $limit = null, $orderBy = null, $sort = 'ASC')
     {
         $select = $this->getSelect();

module/ZfModule/src/ZfModule/View/Helper/NewModule.php

@@ -22,13 +22,9 @@ public function __construct(Mapper\Module $moduleMapper)
     }
 
     /**
-     * __invoke
-     *
-     * @access public
-     * @param array $options array of options
-     * @return array Array of modules
+     * @return string
      */
-    public function __invoke($options = null)
+    public function __invoke()
     {
         $modules = $this->moduleMapper->findAll(10, 'created_at', 'DESC');