adamjakab / SuiteCRM

include/javascript/javascript.php

Doc Comments +20 added lines

@@ -71,12 +71,20 @@ 
 			}
 	}
 
+    /**
+     * @param string $dispField
+     * @param string $type
+     * @param string $required
+     */
     function addSpecialField($dispField, $realField, $type, $required, $prefix = '') {
        if (isset($this->sugarbean->field_name_map[$realField]['vname'])) {
     	$this->addFieldGeneric($dispField, 'date', $this->sugarbean->field_name_map[$realField]['vname'], $required, $prefix );
        }
     }
 
+	/**
+	 * @param string $required
+	 */
 	function addField($field,$required, $prefix='', $displayField='', $translate = false){
 		if ($field == "id") return;
         if(isset($this->sugarbean->field_name_map[$field]['vname'])){
@@ -184,6 +192,10 @@ 
 	}
 
     // Bug #47961 Generator of callback validator
+
+    /**
+     * @param string $prefix
+     */
     function addFieldCallback($field, $type, $displayName, $required, $prefix, $callback)
     {
         $this->script .= 'addToValidateCallback("'
@@ -230,6 +242,11 @@ 
                        . $this->stripEndColon(translate($displayName,$this->sugarbean->module_dir)) . "', '$compareTo', '$allowBlank' );\n";
 	}
 
+	/**
+	 * @param string $type
+	 * @param string $displayName
+	 * @param string $required
+	 */
 	function addToValidateBinaryDependency($field, $type, $displayName, $required, $prefix='',$compareTo){
 		$this->script .= "addToValidateBinaryDependency('".$this->formname."', '".$prefix.$field."', '".$type . "', {$this->getRequiredString($required)},'"
                        . $this->stripEndColon(translate($displayName,$this->sugarbean->module_dir)) . "', '$compareTo' );\n";
@@ -248,6 +265,9 @@ 
 		$this->script .= "addToValidateIsInArray('{$this->formname}', '{$name}', '{$type}', {$req}, '".$this->stripEndColon(translate($displayName,$this->sugarbean->module_dir))."', '{$arr}', '{$operator}');\n";
     }
 
+	/**
+	 * @param string $prefix
+	 */
 	function addAllFields($prefix,$skip_fields=null, $translate = false){
 		if (!isset($skip_fields))
 		{

include/javascript/jsAlerts.php

Doc Comments +4 added lines

@@ -59,6 +59,10 @@
 			$this->addAlert($app_strings['ERROR_JS_ALERT_SYSTEM_CLASS'], $app_strings['ERROR_JS_ALERT_TIMEOUT_TITLE'],'', $app_strings['ERROR_JS_ALERT_TIMEOUT_MSG_2'], (session_cache_expire()) * 60 , 'index.php');
 		}
 	}
+
+	/**
+	 * @param string $subtitle
+	 */
 	function addAlert($type, $name, $subtitle, $description, $countdown, $redirect='')
     {
 		$script = 'addAlert(' . json_encode($type) .',' . json_encode($name). ',' . json_encode($subtitle). ','. json_encode(str_replace(array("\r", "\n"), array('','<br>'),$description)) . ',' . $countdown . ','.json_encode($redirect).');' . "\n";

include/json_config.php

Doc Comments +3 added lines

@@ -53,6 +53,9 @@
 class json_config {
 	var $global_registry_var_name = 'GLOBAL_REGISTRY';
 
+	/**
+	 * @param string $module
+	 */
 	function get_static_json_server($configOnly = true, $getStrings = false, $module = null, $record = null, $scheduler = false) {
 		global $current_user;
 		$str = '';

include/ListView/ListView.php

Doc Comments +34 added lines, -7 removed lines

@@ -105,6 +105,9 @@ 
 }
 
 
+/**
+ * @param string $xTemplateSection
+ */
 function processListView($seed, $xTemplateSection, $html_varName)
 {
     global $sugar_config;
@@ -232,8 +235,8 @@ 
 /**
  * @return void
  * @param unknown $data
- * @param unknown $xTemplateSection
- * @param unknown $html_varName
+ * @param string $xtemplateSection
+ * @param string $html_varName
  * @desc INTERNAL FUNCTION handles the rows
  */
  function process_dynamic_listview_rows($data,$parent_data, $xtemplateSection, $html_varName, $subpanel_def)
@@ -610,6 +613,9 @@ 
  * All Rights Reserved.
  * Contributor(s): ______________________________________.
 */
+ /**
+  * @param string $value
+  */
  function setHeaderText($value) {
     $this->header_text = $value;
 }
@@ -728,6 +734,9 @@ 
  * All Rights Reserved.
  * Contributor(s): ______________________________________.
 */
+ /**
+  * @param string $where
+  */
  function setQuery($where, $limit, $orderBy, $varName, $allowOrderByOveride=true) {
     $this->query_where = $where;
     if($this->getSessionVariable("query", "where") != $where) {
@@ -834,6 +843,9 @@ 
  * All Rights Reserved.
  * Contributor(s): ______________________________________.
 */
+ /**
+  * @param XTemplate $newXTemplate
+  */
  function setXTemplate($newXTemplate) {
     $this->xTemplate = $newXTemplate;
 }
@@ -852,6 +864,9 @@ 
  * All Rights Reserved.
  * Contributor(s): ______________________________________.
 */
+ /**
+  * @param string $name
+  */
  function xTemplateAssign($name, $value) {
 
         if(!isset($this->xTemplate)) {
@@ -906,6 +921,9 @@ 
  * All Rights Reserved.
  * Contributor(s): ______________________________________.
 */
+ /**
+  * @param string $varName
+  */
  function getSessionVariable($localVarName,$varName) {
     //Set any variables pass in through request first
     if(isset($_REQUEST[$this->getSessionVariableName($localVarName, $varName)])) {
@@ -961,7 +979,7 @@ 
 
     /**
 
-    * @return void
+    * @return string
     * @param unknown $localVarName
     * @param unknown $varName
     * @desc INTERNAL FUNCTION returns the session/query variable name
@@ -977,7 +995,7 @@ 
 
     * @return void
     * @param unknown $seed
-    * @param unknown $xTemplateSection
+    * @param unknown $xtemplateSection
     * @param unknown $html_varName
     * @desc INTERNAL FUNCTION Handles List Views using seeds that extend SugarBean
         $XTemplateSection is the section in the XTemplate file that should be parsed usually main
@@ -1169,8 +1187,7 @@ 
     }
     /**
     * @return void
-    * @param unknown $data
-    * @param unknown $xTemplateSection
+    * @param unknown $xtemplateSection
     * @param unknown $html_varName
     * @desc INTERNAL FUNCTION process the List Navigation
     * Portions created by SugarCRM are Copyright (C) SugarCRM, Inc.
@@ -1550,7 +1567,7 @@ 
     /**
     * @return void
     * @param unknown $data
-    * @param unknown $xTemplateSection
+    * @param unknown $xtemplateSection
     * @param unknown $html_varName
     * @desc INTERNAL FUNCTION handles the rows
     * Portions created by SugarCRM are Copyright (C) SugarCRM, Inc.
@@ -2036,6 +2053,10 @@ 
      * All Rights Reserved.
      * Contributor(s): ______________________________________.
      */
+
+     /**
+      * @param string $varName
+      */
      function setLocalSessionVariable($localVarName,$varName, $value) {
         $_SESSION[$localVarName."_".$varName] = $value;
      }
@@ -2046,6 +2067,12 @@ 
      * All Rights Reserved.
      * Contributor(s): ______________________________________.
      */
+
+ /**
+  * @param string $varName
+  *
+  * @return string
+  */
  function getLocalSessionVariable($localVarName,$varName) {
     if(isset($_SESSION[$localVarName."_".$varName])) {
         return $_SESSION[$localVarName."_".$varName];

include/ListView/ListViewData.php

Doc Comments +7 added lines, -3 removed lines

@@ -162,7 +162,8 @@ 
 	/**
 	 * based off of a base name it sets base, offset, and order by variable names to retrieve them from requests and sessions
 	 *
-	 * @param unknown_type $baseName
+	 * @param string $baseName
+	 * @param string $where
 	 */
 	function setVariableName($baseName, $where, $listviewName = null){
         global $timedate;
@@ -180,6 +181,9 @@ 
         $_SESSION[strtoupper($baseName) . "_DETAIL_NAV_HISTORY"] = false;
 	}
 
+	/**
+	 * @param string $main_query
+	 */
 	function getTotalCount($main_query){
 		if(!empty($this->count_query)){
 		    $count_query = $this->count_query;
@@ -605,8 +609,8 @@ 
      * generates the additional details values
      *
      * @param unknown_type $fields
-     * @param unknown_type $adFunction
-     * @param unknown_type $editAccess
+     * @param string $adFunction
+     * @param boolean $editAccess
      * @return array string to attach to field
      */
     function getAdditionalDetails($fields, $adFunction, $editAccess)

include/ListView/ListViewDisplay.php

Doc Comments +4 added lines

@@ -75,6 +75,10 @@
 		$this->lvd = new ListViewData();
 		$this->searchColumns = array () ;
 	}
+
+	/**
+	 * @param string $moduleDir
+	 */
 	function shouldProcess($moduleDir){
 		$searching = false;
 		$sessionSearchQuery = "{$moduleDir}2_QUERY_QUERY";

include/Localization/Localization.php

Doc Comments +1 added lines

@@ -122,6 +122,7 @@
 	 * abstraction of precedence
 	 * @param string prefName Name of preference to retrieve based on overrides
 	 * @param object user User in focus, default null (current_user)
+	 * @param string $prefName
 	 * @return string pref Most significant preference
 	 */
 	function getPrecedentPreference($prefName, $user=null, $sugarConfigPrefName = '') {

include/MassUpdate.php

Doc Comments +6 added lines

@@ -1151,6 +1151,9 @@ 
 		return $html;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	function date_to_dateTime($field, $value) {
 		global $timedate;
 	    //Check if none was set
@@ -1180,6 +1183,9 @@ 
 	    return $value." ".$oldTime;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	function checkClearField($field, $value) {
 	    if ($value == 1 && strpos($field, '_flag')) {
 	        $fName = substr($field, -5);

include/MVC/Controller/SugarController.php

Doc Comments +4 added lines, -1 removed lines

@@ -244,6 +244,7 @@ 
 
 	/**
 	 * Generic load method to load mapping arrays.
+	 * @param string $var
 	 */
 	private function loadMapping($var, $merge = false){
 		$$var = sugar_cache_retrieve("CONTROLLER_". $var . "_".$this->module);
@@ -469,7 +470,8 @@ 
 	/**
 	 * Determine if a given function exists on the objects
 	 * @param function - the function to check
-	 * @return true if the method exists on the object, false otherwise
+	 * @param string $function
+	 * @return boolean if the method exists on the object, false otherwise
 	 */
 	protected function hasFunction($function){
 		return method_exists($this, $function);
@@ -777,6 +779,7 @@ 
 
 	/**
 	 * getActionFilename
+	 * @param string $action
 	 */
 	public static function getActionFilename($action) {
 	   if(isset(self::$action_case_file[$action])) {