Passed
Push — scrutinizer-code-quality ( eedb15...27193c )
by Adam
54:15 queued 15s
created
include/Localization/Localization.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -122,6 +122,7 @@
 block discarded – undo
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 = '') {
Please login to merge, or discard this patch.
include/MassUpdate.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -1151,6 +1151,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
include/MVC/Controller/SugarController.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -244,6 +244,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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])) {
Please login to merge, or discard this patch.
include/MVC/SugarApplication.php 1 patch
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -668,7 +668,6 @@  discard block
 block discarded – undo
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
 block discarded – undo
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,
Please login to merge, or discard this patch.
include/MVC/View/SugarView.php 1 patch
Doc Comments   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1039,6 +1039,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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;
Please login to merge, or discard this patch.
include/MVC/View/ViewFactory.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -53,6 +53,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 	 */
Please login to merge, or discard this patch.
include/MVC/View/views/view.list.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -272,6 +272,7 @@
 block discarded – undo
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")
Please login to merge, or discard this patch.
include/MVC/View/views/view.metadata.php 1 patch
Doc Comments   +7 added lines patch added patch discarded remove patch
@@ -47,6 +47,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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){
Please login to merge, or discard this patch.
include/nusoap/class.nusoap_base.php 1 patch
Doc Comments   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -356,7 +356,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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){
Please login to merge, or discard this patch.