Passed
Push — scrutinizer-code-quality ( 09f5a1...c4c5fb )
by Adam
56:05 queued 14:08
created
modules/SecurityGroups/SecurityGroup.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -168,6 +168,10 @@
 block discarded – undo
168 168
         return false;
169 169
     }
170 170
 
171
+    /**
172
+     * @param SugarBean $focus
173
+     * @param boolean $isUpdate
174
+     */
171 175
     static function inherit(&$focus,$isUpdate)
172 176
     {
173 177
         global $sugar_config;
Please login to merge, or discard this patch.
modules/Studio/DropDowns/DropDownHelper.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -54,6 +54,9 @@  discard block
 block discarded – undo
54 54
 
55 55
     }
56 56
 
57
+    /**
58
+     * @param string $filepath
59
+     */
57 60
     function scanForDropDowns($filepath, $module){
58 61
         $contents = file_get_contents($filepath);
59 62
         $matches = array();
@@ -73,6 +76,7 @@  discard block
 block discarded – undo
73 76
      *
74 77
      * @param string name
75 78
      * @param array dropdown
79
+     * @param string $name
76 80
      * @return array Filtered dropdown list
77 81
      */
78 82
     function filterDropDown($name,$dropdown)
Please login to merge, or discard this patch.
modules/Studio/parsers/StudioParser.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -434,6 +434,10 @@  discard block
 block discarded – undo
434 434
 		$_REQUEST['advanced'] = true;
435 435
 
436 436
 	}
437
+
438
+	/**
439
+	 * @param string $cache_file
440
+	 */
437 441
 	function cacheXTPL($file, $cache_file, $preview_file = false) {
438 442
 		global $beanList;
439 443
 		//now if we have a backup_file lets use that instead of the original
@@ -560,6 +564,8 @@  discard block
 block discarded – undo
560 564
 	 * add:2000
561 565
 	 * swap: 0 - 1999
562 566
 	 *
567
+	 * @param integer $slot_count
568
+	 * @param string $display_count
563 569
 	 */
564 570
 	function addSlotToForm($slot_count, $display_count){
565 571
 		$this->form .= "\n<input type='hidden' name='slot_$slot_count'  id='slot_$display_count' value='$slot_count'>";
@@ -599,6 +605,9 @@  discard block
 block discarded – undo
599 605
 		return $newView;
600 606
 	}
601 607
 
608
+	/**
609
+	 * @param string $file
610
+	 */
602 611
 	function parseOldestFile($file){
603 612
 	 ob_clean();
604 613
 		require_once('modules/Studio/SugarBackup.php');
Please login to merge, or discard this patch.
modules/SugarFeed/Dashlets/SugarFeedDashlet/SugarFeedDashlet.php 1 patch
Doc Comments   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -463,7 +463,7 @@  discard block
 block discarded – undo
463 463
 
464 464
 	/**
465 465
 	 *
466
-	 * @return the fully rendered dashlet
466
+	 * @return string fully rendered dashlet
467 467
 	 */
468 468
 	function display(){
469 469
 
@@ -499,7 +499,7 @@  discard block
 block discarded – undo
499 499
 
500 500
 	/**
501 501
 	 *
502
-	 * @return the title and the user post form
502
+	 * @return string title and the user post form
503 503
 	 * @param $text Object
504 504
 	 */
505 505
 	function getHeader($text='') {
@@ -579,6 +579,9 @@  discard block
 block discarded – undo
579 579
         }
580 580
     }
581 581
 
582
+    /**
583
+     * @param string $type
584
+     */
582 585
     function check_enabled($type){
583 586
         global $db;
584 587
         $query = "SELECT * FROM config where name = 'module_" .$type . "' and value =  1;";
Please login to merge, or discard this patch.
modules/SugarFeed/SugarFeed.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -220,7 +220,7 @@
 block discarded – undo
220 220
      * pushFeed2
221 221
      * This method is a wrapper to pushFeed
222 222
      *
223
-     * @param $text String value of the feed's description
223
+     * @param string $text String value of the feed's description
224 224
      * @param $bean The SugarBean that is triggering the feed
225 225
      * @param $link_type boolean value indicating whether or not feed is a link type
226 226
      * @param $link_url String value of the URL (for link types only)
Please login to merge, or discard this patch.
modules/Tasks/Task.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -236,6 +236,9 @@
 block discarded – undo
236 236
 	}
237 237
 
238 238
 
239
+    /**
240
+     * @param boolean $override_date_for_subpanel
241
+     */
239 242
     protected function formatStartAndDueDates(&$task_fields, $dbtime, $override_date_for_subpanel)
240 243
     {
241 244
         global $timedate;
Please login to merge, or discard this patch.
modules/Trackers/BreadCrumbStack.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@
 block discarded – undo
103 103
     * Returns true if the stack contains the specified item_id, false otherwise.
104 104
     * 
105 105
     * @param item_id the item id to search for
106
-    * @return id of the first item on the stack
106
+    * @return boolean of the first item on the stack
107 107
     */
108 108
    public function contains($item_id) {
109 109
    	  	if(!empty($this->stackMap)){
Please login to merge, or discard this patch.
modules/Trackers/monitor/Monitor.php 1 patch
Doc Comments   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -107,8 +107,8 @@  discard block
 block discarded – undo
107 107
     /**
108 108
      * setValue
109 109
      * Sets the value defined in the monitor's metrics for the given name
110
-     * @param $name String value of metric name
111
-     * @param $value Mixed value
110
+     * @param string $name String value of metric name
111
+     * @param string $value Mixed value
112 112
      * @throws Exception Thrown if metric name is not configured for monitor instance
113 113
      */
114 114
     public function setValue($name, $value) {
@@ -121,6 +121,9 @@  discard block
 block discarded – undo
121 121
         }
122 122
     }
123 123
 
124
+    /**
125
+     * @param string $name
126
+     */
124 127
     public function getValue($name){
125 128
     	return $this->$name;
126 129
     }
@@ -128,7 +131,7 @@  discard block
 block discarded – undo
128 131
     /**
129 132
      * getStores
130 133
      * Returns Array of store names defined for monitor instance
131
-     * @return Array of store names defined for monitor instance
134
+     * @return string of store names defined for monitor instance
132 135
      */
133 136
     function getStores() {
134 137
         return $this->stores;
Please login to merge, or discard this patch.
modules/Trackers/store/Store.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -51,6 +51,7 @@
 block discarded – undo
51 51
      * flush
52 52
      * This is the method implementations need to provide to store a monitor instance
53 53
      * @param $monitor An instance of Monitor
54
+     * @return void
54 55
      */
55 56
     public function flush($monitor);
56 57
 
Please login to merge, or discard this patch.