Passed
Push — scrutinizer-code-quality ( 09f5a1...c4c5fb )
by Adam
56:05 queued 14:08
created
modules/AOW_WorkFlow/aow_utils.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -608,6 +608,9 @@  discard block
 block discarded – undo
608 608
 
609 609
 }
610 610
 
611
+/**
612
+ * @param string $list_id
613
+ */
611 614
 function getDropdownList($list_id, $selected_value) {
612 615
     global $app_list_strings;
613 616
     $option = '';
@@ -634,6 +637,9 @@  discard block
 block discarded – undo
634 637
     return array_shift($countsKeys);
635 638
 }
636 639
 
640
+/**
641
+ * @param string $id
642
+ */
637 643
 function getRoundRobinUser($users, $id) {
638 644
 
639 645
     $file = create_cache_directory('modules/AOW_WorkFlow/Users/') . $id . 'lastUser.cache.php';
@@ -659,6 +665,9 @@  discard block
 block discarded – undo
659 665
    return $users[0];
660 666
 }
661 667
 
668
+/**
669
+ * @param string $id
670
+ */
662 671
 function setLastUser($user_id, $id) {
663 672
 
664 673
     $_SESSION['lastuser'][$id] = $user_id;
Please login to merge, or discard this patch.
modules/AOW_WorkFlow/AOW_WorkFlow.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -188,6 +188,9 @@
 block discarded – undo
188 188
         return null;
189 189
     }
190 190
 
191
+    /**
192
+     * @param string $type
193
+     */
191 194
     function build_flow_query_join($name, SugarBean $module, $type, $query = array()){
192 195
 
193 196
         if(!isset($query['join'][$name])){
Please login to merge, or discard this patch.
modules/Calendar/CalendarActivity.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -130,6 +130,9 @@  discard block
 block discarded – undo
130 130
         return self::getOccursWhereClauseGeneral($table_name, $rel_table, $start_ts_obj, $end_ts_obj, $field_name, array('self', 'until'));
131 131
     }
132 132
 
133
+	/**
134
+	 * @param User $user_focus
135
+	 */
133 136
 	function get_freebusy_activities($user_focus, $start_date_time, $end_date_time){
134 137
 		$act_list = array();
135 138
 		$vcal_focus = new vCal();
@@ -258,7 +261,7 @@  discard block
 block discarded – undo
258 261
      * @param SugarDateTime $start_ts_obj start date
259 262
      * @param SugarDateTime $end_ts_obj end date
260 263
      * @param string $field_name date field in table
261
-     * @param array $callback callback function to generete specific SQL query-part
264
+     * @param string[] $callback callback function to generete specific SQL query-part
262 265
      * @return string
263 266
      */
264 267
     protected static function getOccursWhereClauseGeneral($table_name, $rel_table, $start_ts_obj, $end_ts_obj, $field_name, $callback)
Please login to merge, or discard this patch.
modules/Calendar/CalendarDisplay.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -204,6 +204,7 @@
 block discarded – undo
204 204
 
205 205
 	/**
206 206
 	 * load settings popup template
207
+	 * @param Sugar_Smarty $ss
207 208
 	 */
208 209
 	protected function load_settings_template(&$ss){
209 210
 
Please login to merge, or discard this patch.
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.