Dev Finetuning Scrutenizer

LimeSurvey · Nov 24, 2017 · 8bf84e4 · 8bf84e4
1 parent 8314bda
commit 8bf84e4
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/.scrutinizer.yml b/.scrutinizer.yml
@@ -7,7 +7,15 @@ build:
         tests: true
 filter:
     paths: [application/*]
-    excluded_paths: [application/helpers/adodb/*, application/libraries/admin/http/*, application/libraries/admin/pclzip/*, application/extensions/*, application/views/*, application/third_party/*]
+    excluded_paths: 
+        - 'application/helpers/adodb/*'
+        - 'application/libraries/admin/http/*'
+        - 'application/libraries/admin/pclzip/*'
+        - 'application/extensions/*'
+        - 'application/views/*'
+        - 'application/third_party/*'
+    dependency_paths:
+        - "framework/"    
 checks:
     javascript: true 
     php:

diff --git a/application/helpers/common_helper.php b/application/helpers/common_helper.php
@@ -868,7 +868,7 @@ function getUserList($outputformat = 'fullinfoarray')
 /**
 * Gets all survey infos in one big array including the language specific settings
 *
-* @param string $surveyid  The survey ID
+* @param integer $surveyid  The survey ID
 * @param string $languagecode The language code - if not given the base language of the particular survey is used
 * @return array|bool Returns array with survey info or false, if survey does not exist
 */

diff --git a/application/models/Response.php b/application/models/Response.php
@@ -72,7 +72,7 @@ public function getFilesAndSqga($sQID = 0)
             $files = array();
             foreach ($aQuestions as $question) {
                 $field = $question->sid.'X'.$question->gid.'X'.$question->qid;
-                 $data = json_decode(stripslashes($this->getAttribute($field)), true);
+                $data = json_decode(stripslashes($this->getAttribute($field)), true);
                 if (is_array($data)) {
                     $files[$field] = $data;
                 }