Dev-Int / glsr

src/AppBundle/Controller/AbstractController.php

Doc Comments +3 added lines

@@ -195,6 +195,9 @@
         }
     }
 
+    /**
+     * @param string $entityName
+     */
     private function testReturnParam($entity, $entityName) {
         if ($entityName === 'company' || $entityName === 'settings' || $entityName === 'tva') {
             $param = array('id' => $entity->getId());

app/check.php

Doc Comments +3 added lines

@@ -95,6 +95,9 @@
     echo PHP_EOL;
 }
 
+/**
+ * @param string $message
+ */
 function echo_style($style, $message)
 {
     // ANSI color codes

app/SymfonyRequirements.php

Doc Comments +2 added lines, -2 removed lines

@@ -223,7 +223,7 @@ 
      * Adds a mandatory requirement in form of a php.ini configuration.
      *
      * @param string        $cfgName           The configuration name used for ini_get()
-     * @param bool|callback $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
+     * @param boolean|string $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
      *                                         or a callback function receiving the configuration value as parameter to determine the fulfillment of the requirement
      * @param bool          $approveCfgAbsence If true the Requirement will be fulfilled even if the configuration option does not exist, i.e. ini_get() returns false.
      *                                         This is helpful for abandoned configs in later PHP versions or configs of an optional extension, like Suhosin.
@@ -241,7 +241,7 @@ 
      * Adds an optional recommendation in form of a php.ini configuration.
      *
      * @param string        $cfgName           The configuration name used for ini_get()
-     * @param bool|callback $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
+     * @param string|false $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
      *                                         or a callback function receiving the configuration value as parameter to determine the fulfillment of the requirement
      * @param bool          $approveCfgAbsence If true the Requirement will be fulfilled even if the configuration option does not exist, i.e. ini_get() returns false.
      *                                         This is helpful for abandoned configs in later PHP versions or configs of an optional extension, like Suhosin.

src/AppBundle/Controller/ArticleController.php

Doc Comments +1 added lines, -1 removed lines

@@ -189,7 +189,7 @@
      *
      * @param \Symfony\Component\HttpFoundation\Request $request  Form request
      * @param \AppBundle\Entity\Supplier                $supplier Supplier to desactivate
-     * @return array
+     * @return \Symfony\Component\HttpFoundation\RedirectResponse
      */
     public function changeAction(Request $request, Supplier $supplier)
     {