@@ -122,6 +122,7 @@ |
||
122 | 122 | * abstraction of precedence |
123 | 123 | * @param string prefName Name of preference to retrieve based on overrides |
124 | 124 | * @param object user User in focus, default null (current_user) |
125 | + * @param string $prefName |
|
125 | 126 | * @return string pref Most significant preference |
126 | 127 | */ |
127 | 128 | function getPrecedentPreference($prefName, $user=null, $sugarConfigPrefName = '') { |
@@ -1151,6 +1151,9 @@ discard block |
||
1151 | 1151 | return $html; |
1152 | 1152 | } |
1153 | 1153 | |
1154 | + /** |
|
1155 | + * @param string $value |
|
1156 | + */ |
|
1154 | 1157 | function date_to_dateTime($field, $value) { |
1155 | 1158 | global $timedate; |
1156 | 1159 | //Check if none was set |
@@ -1180,6 +1183,9 @@ discard block |
||
1180 | 1183 | return $value." ".$oldTime; |
1181 | 1184 | } |
1182 | 1185 | |
1186 | + /** |
|
1187 | + * @param string $value |
|
1188 | + */ |
|
1183 | 1189 | function checkClearField($field, $value) { |
1184 | 1190 | if ($value == 1 && strpos($field, '_flag')) { |
1185 | 1191 | $fName = substr($field, -5); |
@@ -244,6 +244,7 @@ discard block |
||
244 | 244 | |
245 | 245 | /** |
246 | 246 | * Generic load method to load mapping arrays. |
247 | + * @param string $var |
|
247 | 248 | */ |
248 | 249 | private function loadMapping($var, $merge = false){ |
249 | 250 | $$var = sugar_cache_retrieve("CONTROLLER_". $var . "_".$this->module); |
@@ -469,7 +470,8 @@ discard block |
||
469 | 470 | /** |
470 | 471 | * Determine if a given function exists on the objects |
471 | 472 | * @param function - the function to check |
472 | - * @return true if the method exists on the object, false otherwise |
|
473 | + * @param string $function |
|
474 | + * @return boolean if the method exists on the object, false otherwise |
|
473 | 475 | */ |
474 | 476 | protected function hasFunction($function){ |
475 | 477 | return method_exists($this, $function); |
@@ -777,6 +779,7 @@ discard block |
||
777 | 779 | |
778 | 780 | /** |
779 | 781 | * getActionFilename |
782 | + * @param string $action |
|
780 | 783 | */ |
781 | 784 | public static function getActionFilename($action) { |
782 | 785 | if(isset(self::$action_case_file[$action])) { |
@@ -668,7 +668,6 @@ discard block |
||
668 | 668 | * Redirect to another URL |
669 | 669 | * |
670 | 670 | * @access public |
671 | - * @param string $url The URL to redirect to |
|
672 | 671 | */ |
673 | 672 | public static function appendErrorMessage($error_message) |
674 | 673 | { |
@@ -692,6 +691,8 @@ discard block |
||
692 | 691 | /** |
693 | 692 | * Wrapper for the PHP setcookie() function, to handle cases where headers have |
694 | 693 | * already been sent |
694 | + * @param string $name |
|
695 | + * @param string $domain |
|
695 | 696 | */ |
696 | 697 | public static function setCookie( |
697 | 698 | $name, |
@@ -1039,6 +1039,7 @@ discard block |
||
1039 | 1039 | * |
1040 | 1040 | * @param string option - the option that we want to know the valye of |
1041 | 1041 | * @param bool default - what the default value should be if we do not find the option |
1042 | + * @param string $option |
|
1042 | 1043 | * |
1043 | 1044 | * @return bool - the value of the option |
1044 | 1045 | */ |
@@ -1273,7 +1274,7 @@ discard block |
||
1273 | 1274 | * Return the "breadcrumbs" to display at the top of the page |
1274 | 1275 | * |
1275 | 1276 | * @param bool $show_help optional, true if we show the help links |
1276 | - * @return HTML string containing breadcrumb title |
|
1277 | + * @return string string containing breadcrumb title |
|
1277 | 1278 | */ |
1278 | 1279 | public function getModuleTitle( |
1279 | 1280 | $show_help = true |
@@ -1446,6 +1447,9 @@ discard block |
||
1446 | 1447 | } |
1447 | 1448 | } |
1448 | 1449 | |
1450 | + /** |
|
1451 | + * @param string $module |
|
1452 | + */ |
|
1449 | 1453 | protected function getModuleTitleIconPath($module) |
1450 | 1454 | { |
1451 | 1455 | $iconPath = ""; |
@@ -1496,7 +1500,7 @@ discard block |
||
1496 | 1500 | /** |
1497 | 1501 | * Fetch config values to be put into an array for JavaScript |
1498 | 1502 | * |
1499 | - * @return array |
|
1503 | + * @return string[] |
|
1500 | 1504 | */ |
1501 | 1505 | protected function getSugarConfigJS(){ |
1502 | 1506 | global $sugar_config; |
@@ -53,6 +53,7 @@ discard block |
||
53 | 53 | /** |
54 | 54 | * load the correct view |
55 | 55 | * @param string $type View Type |
56 | + * @param string $module |
|
56 | 57 | * @return valid view |
57 | 58 | */ |
58 | 59 | static function loadView($type = 'default', $module, $bean = null, $view_object_map = array(), $target_module=''){ |
@@ -98,6 +99,7 @@ discard block |
||
98 | 99 | |
99 | 100 | /** |
100 | 101 | * Load the view_<view>_config.php file which holds options used by the view. |
102 | + * @param string $type |
|
101 | 103 | */ |
102 | 104 | static function _loadConfig(&$view, $type){ |
103 | 105 | $view_config_custom = array(); |
@@ -199,6 +201,8 @@ discard block |
||
199 | 201 | * This is a private function which just helps the getView function generate the |
200 | 202 | * proper view object |
201 | 203 | * |
204 | + * @param string $file |
|
205 | + * @param string $type |
|
202 | 206 | * @return a valid SugarView |
203 | 207 | */ |
204 | 208 | static function _buildFromFile($file, &$bean, $view_object_map, $type, $module){ |
@@ -242,6 +246,7 @@ discard block |
||
242 | 246 | * @param object bean = the bean to pass to the view |
243 | 247 | * @param array view_object_map - the array which holds obejcts to pass between the |
244 | 248 | * controller and the view. |
249 | + * @param string $class |
|
245 | 250 | * |
246 | 251 | * @return SugarView |
247 | 252 | */ |
@@ -272,6 +272,7 @@ |
||
272 | 272 | |
273 | 273 | /** |
274 | 274 | * |
275 | + * @param string $module |
|
275 | 276 | * @return SearchForm |
276 | 277 | */ |
277 | 278 | protected function getSearchForm2($seed, $module, $action = "index") |
@@ -47,6 +47,9 @@ discard block |
||
47 | 47 | |
48 | 48 | |
49 | 49 | |
50 | + /** |
|
51 | + * @param string $name |
|
52 | + */ |
|
50 | 53 | function displayCheckBoxes($name,$values, $selected =array(), $attr=''){ |
51 | 54 | echo "<div $attr style='overflow:auto;float:left;width:200px;height:200px' >"; |
52 | 55 | foreach($values as $value){ |
@@ -56,6 +59,10 @@ discard block |
||
56 | 59 | echo "</div>"; |
57 | 60 | } |
58 | 61 | |
62 | + /** |
|
63 | + * @param string $name |
|
64 | + * @param string[] $values |
|
65 | + */ |
|
59 | 66 | function displaySelect($name,$values, $selected ='', $attr=''){ |
60 | 67 | echo "<select name='$name' $attr>"; |
61 | 68 | foreach($values as $value){ |
@@ -356,7 +356,7 @@ discard block |
||
356 | 356 | /** |
357 | 357 | * gets the current debug data for this instance |
358 | 358 | * |
359 | - * @return debug data |
|
359 | + * @return string data |
|
360 | 360 | * @access public |
361 | 361 | */ |
362 | 362 | function &getDebug() { |
@@ -369,7 +369,7 @@ discard block |
||
369 | 369 | * gets the current debug data for this instance as an XML comment |
370 | 370 | * this may change the contents of the debug data |
371 | 371 | * |
372 | - * @return debug data as an XML comment |
|
372 | + * @return string data as an XML comment |
|
373 | 373 | * @access public |
374 | 374 | */ |
375 | 375 | function &getDebugAsXMLComment() { |
@@ -402,7 +402,7 @@ discard block |
||
402 | 402 | /** |
403 | 403 | * returns error string if present |
404 | 404 | * |
405 | - * @return mixed error string or false |
|
405 | + * @return string|false error string or false |
|
406 | 406 | * @access public |
407 | 407 | */ |
408 | 408 | function getError(){ |
@@ -415,7 +415,7 @@ discard block |
||
415 | 415 | /** |
416 | 416 | * sets error string |
417 | 417 | * |
418 | - * @return boolean $string error string |
|
418 | + * @return boolean|null $string error string |
|
419 | 419 | * @access private |
420 | 420 | */ |
421 | 421 | function setError($str){ |
@@ -851,7 +851,7 @@ discard block |
||
851 | 851 | * returns false, if not prefixed |
852 | 852 | * |
853 | 853 | * @param string $str The prefixed string |
854 | - * @return mixed The prefix or false if there is no prefix |
|
854 | + * @return string|false The prefix or false if there is no prefix |
|
855 | 855 | * @access public |
856 | 856 | */ |
857 | 857 | function getPrefix($str){ |
@@ -882,7 +882,7 @@ discard block |
||
882 | 882 | * or false if no prefixes registered for the given namespace |
883 | 883 | * |
884 | 884 | * @param string $ns The namespace |
885 | - * @return mixed The prefix, false if the namespace has no prefixes |
|
885 | + * @return string The prefix, false if the namespace has no prefixes |
|
886 | 886 | * @access public |
887 | 887 | */ |
888 | 888 | function getPrefixFromNamespace($ns) { |
@@ -948,7 +948,7 @@ discard block |
||
948 | 948 | * |
949 | 949 | * @param int $timestamp Unix time stamp |
950 | 950 | * @param boolean $utc Whether the time stamp is UTC or local |
951 | -* @return mixed ISO 8601 date string or false |
|
951 | +* @return string|false ISO 8601 date string or false |
|
952 | 952 | * @access public |
953 | 953 | */ |
954 | 954 | function timestamp_to_iso8601($timestamp,$utc=true){ |