Passed
Push — scrutinizer-code-quality ( eedb15...27193c )
by Adam
54:15 queued 15s
created
modules/Calendar/CalendarGrid.php 1 patch
Doc Comments   -2 removed lines patch added patch discarded remove patch
@@ -343,7 +343,6 @@  discard block
 block discarded – undo
343 343
 	 * Get html of basic cell
344 344
 	 * @param integer $start timestamp
345 345
 	 * @param integer $height slot height
346
-	 * @param string $prefix prefix for id of slot used in shared view
347 346
 	 * @return string
348 347
 	 */
349 348
 	protected function get_basic_cell($start,$height = 80,$suffix = ""){
@@ -358,7 +357,6 @@  discard block
 block discarded – undo
358 357
 	/**
359 358
 	 * Get html of basic week grid
360 359
 	 * @param integer $start timestamp
361
-	 * @param string $prefix prefix for id of slot used in shared view
362 360
 	 * @return string
363 361
 	 */
364 362
 	protected function get_basic_row($start,$cols = 7,$suffix = ""){
Please login to merge, or discard this patch.
modules/Calls/Call.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -140,7 +140,7 @@
 block discarded – undo
140 140
 
141 141
 	/**
142 142
 	 * Disable edit if call is recurring and source is not Sugar. It should be edited only from Outlook.
143
-	 * @param $view string
143
+	 * @param string $view string
144 144
 	 * @param $is_owner bool
145 145
 	 */
146 146
 	function ACLAccess($view,$is_owner='not_set',$in_group='not_set'){
Please login to merge, or discard this patch.
modules/Calls/reschedule_history.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -69,6 +69,11 @@
 block discarded – undo
69 69
 
70 70
 }
71 71
 
72
+/**
73
+ * @param string $field
74
+ * @param string $value
75
+ * @param string $view
76
+ */
72 77
 function reschedule_count($focus, $field, $value, $view){
73 78
 
74 79
         $query = "SELECT COUNT(*) FROM calls_reschedule WHERE call_id='".$focus->id."'";
Please login to merge, or discard this patch.
modules/Campaigns/Campaign.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -399,7 +399,7 @@
 block discarded – undo
399 399
 	 * with a campaign.
400 400
      *
401 401
      * @param string $query Select query string
402
-     * @param array $param array of arguments
402
+     * @param array $params array of arguments
403 403
      * @return string count query
404 404
      *
405 405
 	 */
Please login to merge, or discard this patch.
modules/Campaigns/ProcessBouncedEmails.php 1 patch
Doc Comments   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 /**
101 101
  * Given an email address, mark it as invalid.
102 102
  *
103
- * @param $email_address
103
+ * @param Email $email_address
104 104
  */
105 105
 function markEmailAddressInvalid($email_address)
106 106
 {
@@ -118,6 +118,7 @@  discard block
 block discarded – undo
118 118
  * Get the existing campaign log entry by tracker key.
119 119
  * 
120 120
  * @param string Target Key
121
+ * @param string $identifier
121 122
  * @return array Campaign Log Row
122 123
  */
123 124
 function getExistingCampaignLogEntry($identifier)
@@ -136,6 +137,7 @@  discard block
 block discarded – undo
136 137
  * Scan the bounced email searching for a valid target identifier.
137 138
  * 
138 139
  * @param string Email Description
140
+ * @param string $email_description
139 141
  * @return array Results including matches and identifier
140 142
  */
141 143
 function checkBouncedEmailForIdentifier($email_description)
Please login to merge, or discard this patch.
modules/Campaigns/utils.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -285,6 +285,9 @@  discard block
 block discarded – undo
285 285
      * This method is deprecated
286 286
      * @deprecated 62_Joneses - June 24, 2011
287 287
      * @see campaign_log_lead_or_contact_entry()
288
+     * @param Prospect $parent_bean
289
+     * @param SugarBean $child_bean
290
+     * @param string $activity_type
288 291
      */
289 292
 function campaign_log_lead_entry($campaign_id, $parent_bean,$child_bean,$activity_type){
290 293
     campaign_log_lead_or_contact_entry($campaign_id, $parent_bean,$child_bean,$activity_type);
@@ -442,6 +445,7 @@  discard block
 block discarded – undo
442 445
 
443 446
 /**
444 447
  * same function as get_subscription_lists, but with the data separated in an associated array
448
+ * @param Contact $focus
445 449
  */
446 450
 function get_subscription_lists_keyed($focus) {
447 451
     $subs_arr = array();
Please login to merge, or discard this patch.
OpportunitiesByLeadSourceDashlet/OpportunitiesByLeadSourceDashlet.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -200,6 +200,9 @@
 block discarded – undo
200 200
 
201 201
     }
202 202
 
203
+    /**
204
+     * @param string $query
205
+     */
203 206
     function getChartData($query)
204 207
     {
205 208
         global $app_list_strings, $db;
Please login to merge, or discard this patch.
Charts/Dashlets/PipelineBySalesStageDashlet/PipelineBySalesStageDashlet.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -241,7 +241,7 @@
 block discarded – undo
241 241
      * awu: Bug 16794 - this function is a hack to get the correct sales stage order until
242 242
      * i can clean it up later
243 243
      *
244
-     * @param  $query string
244
+     * @param  string $query string
245 245
      * @return array
246 246
      */
247 247
     function getChartData(
Please login to merge, or discard this patch.
modules/Configurator/Configurator.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -182,6 +182,10 @@  discard block
 block discarded – undo
182 182
 		}
183 183
 		return $sugar_config;
184 184
 	}
185
+
186
+	/**
187
+	 * @param string $override
188
+	 */
185 189
 	function saveOverride($override) {
186 190
         require_once('install/install_utils.php');
187 191
 	    if ( !file_exists('config_override.php') ) {
@@ -230,6 +234,11 @@  discard block
 block discarded – undo
230 234
 		}
231 235
 	}
232 236
 
237
+	/**
238
+	 * @param string $path
239
+	 *
240
+	 * @return string
241
+	 */
233 242
 	function checkTempImage($path)
234 243
 	{
235 244
         if(!verify_uploaded_image($path)) {
Please login to merge, or discard this patch.