Dev: Code inspection app/models fix param type & closing tags (#820)

Dev: Code inspection app/models fix param type Dev: Code inspection app/models redundant closing tags
LimeSurvey · Oct 9, 2017 · 703da1d · 703da1d
1 parent ea2aca5
commit 703da1d
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 7 deletions.
diff --git a/application/models/ParticipantAttributeName.php b/application/models/ParticipantAttributeName.php
@@ -421,20 +421,20 @@ function getAttributesValues($attribute_id = null)
     /**
      * this is a very specific function used to get the attributes that are
      * not present for the participant
-     * @param integer $attributeid
+     * @param array $attributeIds
      * @return array
      */
-    function getNotAddedAttributes($attributeid)
+    function getNotAddedAttributes($attributeIds)
     {
         $output = array();
         $notin=array();
-        foreach($attributeid as $row) {
+        foreach($attributeIds as $row) {
             $notin[] = $row;
         }
 
         $criteria = new CDbCriteria();
         $alias = $this->getTableAlias();
-        $criteria->addNotInCondition("$alias.attribute_id", $attributeid);
+        $criteria->addNotInCondition("$alias.attribute_id", $attributeIds);
         $records = ParticipantAttributeName::model()->with('participant_attribute_names_lang')->findAll($criteria);
         foreach($records as $row) { //Iterate through each attribute
             $thisname="";

diff --git a/application/models/SurveymenuEntryData.php b/application/models/SurveymenuEntryData.php
@@ -132,5 +132,3 @@ private function _getValueForLinkData($getDataPair)
     }
 
 }
-
-?>
diff --git a/application/models/behaviors/PluginEventBehavior.php b/application/models/behaviors/PluginEventBehavior.php
@@ -57,4 +57,3 @@ public function dispatchPluginModelEvent($sEventName, $criteria = null)
 			return App()->getPluginManager()->dispatchEvent($oPluginEvent);
 		}
 	}
-?>