Passed
Push — scrutinizer-code-quality ( 09f5a1...c4c5fb )
by Adam
56:05 queued 14:08
created
modules/iCals/iCal.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -191,7 +191,7 @@
 block discarded – undo
191 191
     * Creates the string for the user's events and todos between the given start
192 192
     * and end times
193 193
     *
194
-    * @param UserBean $user_bean the current UserBean
194
+    * @param User $user_bean the current UserBean
195 195
     * @param DateTime $start_date_time the start date to search from
196 196
     * @param DateTime $end_date_time the end date to search to
197 197
     * @param string $dtstamp the current timestamp
Please login to merge, or discard this patch.
modules/Import/ImportDuplicateCheck.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -133,7 +133,6 @@
 block discarded – undo
133 133
     /**
134 134
      * Checks to see if the given bean is a duplicate based off the given fields
135 135
      *
136
-     * @param  array $indexlist
137 136
      * @return bool true if this bean is a duplicate or false if it isn't
138 137
      */
139 138
     public function isADuplicateRecordByFields($fieldList)
Please login to merge, or discard this patch.
modules/Import/Importer.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -83,6 +83,9 @@  discard block
 block discarded – undo
83 83
     protected $sugarToExternalSourceFieldMap = array();
84 84
 
85 85
 
86
+    /**
87
+     * @param ImportFile $importSource
88
+     */
86 89
     public function __construct($importSource, $bean)
87 90
     {
88 91
         global $mod_strings, $sugar_config;
@@ -434,6 +437,9 @@  discard block
 block discarded – undo
434 437
     }
435 438
 
436 439
 
440
+    /**
441
+     * @param string $fieldTranslated
442
+     */
437 443
     protected function sanitizeFieldValueByType($rowValue, $fieldDef, $defaultRowValue, $focus, $fieldTranslated)
438 444
     {
439 445
         global $mod_strings, $app_list_strings;
@@ -527,6 +533,9 @@  discard block
 block discarded – undo
527 533
         }
528 534
     }
529 535
 
536
+    /**
537
+     * @param boolean $newRecord
538
+     */
530 539
     protected function saveImportBean($focus, $newRecord)
531 540
     {
532 541
         global $timedate, $current_user;
Please login to merge, or discard this patch.
modules/Import/ImportFieldSanitize.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -253,7 +253,7 @@
 block discarded – undo
253 253
      *
254 254
      * @param  $value  string
255 255
      * @param  $format string
256
-     * @return string sanitized and validated value on success, bool false on failure
256
+     * @return boolean sanitized and validated value on success, bool false on failure
257 257
      */
258 258
     public function isValidTimeDate(
259 259
         $value,
Please login to merge, or discard this patch.
modules/Import/maps/ImportMap.php 1 patch
Doc Comments   -8 removed lines patch added patch discarded remove patch
@@ -192,13 +192,6 @@  discard block
 block discarded – undo
192 192
     /**
193 193
      * Save
194 194
      *
195
-     * @param  string $owner_id
196
-     * @param  string $name
197
-     * @param  string $module
198
-     * @param  string $source
199
-     * @param  string $has_header
200
-     * @param  string $delimiter
201
-     * @param  string $enclosure
202 195
      * @return bool
203 196
      */
204 197
     public function save($check_notify = FALSE) {
@@ -362,7 +355,6 @@  discard block
 block discarded – undo
362 355
     /**
363 356
      * set and get field elements in request field to and from user preferences
364 357
      *
365
-     * @param  array $fields_array
366 358
      * @return array $obj_arr
367 359
      */
368 360
     public function set_get_import_wizard_fields($ForceValsArr = '')
Please login to merge, or discard this patch.
modules/Import/sources/ImportDataSource.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -107,7 +107,6 @@
 block discarded – undo
107 107
      * external field name and the rvalue equal to the actual value.  
108 108
      *
109 109
      * @abstract
110
-     * @param  int $startIndex
111 110
      * @param  int $maxResults
112 111
      * @return void
113 112
      */
Please login to merge, or discard this patch.
modules/Import/sources/ImportFile.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -393,6 +393,9 @@
 block discarded – undo
393 393
         return $this->_time_format;
394 394
     }
395 395
 
396
+    /**
397
+     * @param boolean $hasHeader
398
+     */
396 399
     public function setHeaderRow($hasHeader)
397 400
     {
398 401
         $this->_hasHeader = $hasHeader;
Please login to merge, or discard this patch.
modules/Import/views/ImportListView.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@
 block discarded – undo
84 84
      * Create a list view object that can display a data source which implements the Paginatable interface.
85 85
      *
86 86
      * @throws Exception
87
-     * @param  Paginatable $dataSource
87
+     * @param  ImportFile $dataSource
88 88
      * @param  array $params
89 89
      * @param string $tableIdentifier
90 90
      */
Please login to merge, or discard this patch.
modules/Import/views/view.confirm.php 1 patch
Doc Comments   +10 added lines patch added patch discarded remove patch
@@ -227,6 +227,9 @@  discard block
 block discarded – undo
227 227
         return get_select_options_with_id($GLOBALS['app_list_strings']['import_delimeter_options'], $selctedDelim);
228 228
     }
229 229
 
230
+    /**
231
+     * @param string $enclosure
232
+     */
230 233
     private function getEnclosureOptions($enclosure)
231 234
     {
232 235
         $results = array();
@@ -392,6 +395,9 @@  discard block
 block discarded – undo
392 395
         $this->ss->assign('TIMEZONEOPTIONS', TimeDate::getTimezoneList());
393 396
     }
394 397
 
398
+    /**
399
+     * @param ImportFile $importFile
400
+     */
395 401
     private function setImportFileCharacterSet($importFile, $field_map = array())
396 402
     {
397 403
         global $locale;
@@ -431,6 +437,9 @@  discard block
 block discarded – undo
431 437
         return $maxColumns;
432 438
     }
433 439
 
440
+    /**
441
+     * @param ImportFile $importFile
442
+     */
434 443
     public function getSampleSet($importFile)
435 444
     {
436 445
         $rows = array();
@@ -456,6 +465,7 @@  discard block
 block discarded – undo
456 465
 
457 466
     /**
458 467
      * Returns JS used in this view
468
+     * @param boolean $maxRecordsExceeded
459 469
      */
460 470
     private function _getJS($maxRecordsExceeded, $maxRecordsWarningMessg, $importMappingJS, $importFileMap)
461 471
     {
Please login to merge, or discard this patch.