Scrutinizer Auto-Fixes

This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com
LimeSurvey · Jan 20, 2017 · 783d0df · 783d0df
1 parent 64d7240
commit 783d0df
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 6 deletions.
diff --git a/application/controllers/RegisterController.php b/application/controllers/RegisterController.php
@@ -443,7 +443,7 @@ public function getExtraAttributeInfo($iSurveyId)
     /**
     * Get the date if survey is future
     * @param $iSurveyId
-    * @return localized date
+    * @return null|string date
     */
     public function getStartDate($iSurveyId){
         $aSurveyInfo=getSurveyInfo($iSurveyId,Yii::app()->language);

diff --git a/application/controllers/admin/tokens.php b/application/controllers/admin/tokens.php
@@ -649,10 +649,10 @@ public function addnew($iSurveyId)
 
     /**
      * Edit Tokens
-     * @param int $iSurveyID
+     * @param int $iSurveyId
      * @param int $iTokenId
      * @param boolean $ajax
-     * @return void
+     * @return false|null
      * @todo When is this function used without Ajax?
      */
     public function edit($iSurveyId, $iTokenId, $ajax = false)

diff --git a/application/helpers/expressions/em_core_helper.php b/application/helpers/expressions/em_core_helper.php
@@ -1949,6 +1949,7 @@ public function sProcessStringContainingExpressionsHelper($src, $questionNum, $s
 
     /**
      * If the equation contains refernece to this, expand to comma separated list if needed.
+     * @param string $src
      */
     function ExpandThisVar($src)
     {

diff --git a/application/helpers/remotecontrol/remotecontrol_handle.php b/application/helpers/remotecontrol/remotecontrol_handle.php
@@ -2155,7 +2155,7 @@ public function mail_registered_participants($sSessionKey, $iSurveyID, $override
     * @access public
     * @param string $sSessionKey Auth credentials
     * @param int $iSurveyID ID of the survey that participants belong
-    * @param array $aTokenIDs Ids of the participant to invite
+    * @param array $aTokenIds Ids of the participant to invite
     * @param bool $bEmail Send only pending invites (TRUE) or resend invites only (FALSE)
     * @return array Result of the action
     */

diff --git a/application/libraries/PluginManager/LimesurveyApi.php b/application/libraries/PluginManager/LimesurveyApi.php
@@ -318,7 +318,7 @@ public function getUserByName($username)
         /**
          * Get the user object for a given email
          *
-         * @param string $username
+         * @param string|null $email
          * @return User|null Returns the user, or null when not found
          */
         public function getUserByEmail($email)

diff --git a/application/models/Question.php b/application/models/Question.php
@@ -828,7 +828,7 @@ public function getOtherIcon()
 
     /**
      * Get an new title/code for a question
-     * @param integer|string $index base for question code (exemple : inde of question when survey import)
+     * @param integer $index base for question code (exemple : inde of question when survey import)
      * @return string|null : new title, null if impossible
      */
     public function getNewTitle($index=0)

diff --git a/application/models/SurveyLink.php b/application/models/SurveyLink.php
@@ -75,6 +75,10 @@ function getLinkInfo($participantid)
      *
      *
      * */
+
+    /**
+     * @param integer $iSurveyId
+     */
     function rebuildLinksFromTokenTable($iSurveyId)
     {
         $this->deleteLinksBySurvey($iSurveyId);