Passed
Push — scrutinizer-code-quality ( 09f5a1...c4c5fb )
by Adam
56:05 queued 14:08
created
modules/InboundEmail/AOPInboundEmail.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 
29 29
     /**
30 30
      * Replaces embedded image links with links to the appropriate note in the CRM.
31
-     * @param $string
31
+     * @param string $string
32 32
      * @param $noteIds A whitelist of note ids to replace
33 33
      * @return mixed
34 34
      */
@@ -53,6 +53,9 @@  discard block
 block discarded – undo
53 53
     }
54 54
 
55 55
 
56
+    /**
57
+     * @param Email $email
58
+     */
56 59
     function handleCreateCase($email, $userId) {
57 60
         global $current_user, $mod_strings, $current_language;
58 61
         $mod_strings = return_module_language($current_language, "Emails");
Please login to merge, or discard this patch.
modules/jjwg_Maps/controller.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -446,7 +446,7 @@
 block discarded – undo
446 446
      * 
447 447
      * Export rows of data as a CSV file
448 448
      * @param unknown_type $rows
449
-     * @param unknown_type $filename
449
+     * @param string $filename
450 450
      */
451 451
     private function do_list_csv_output($rows, $filename) {
452 452
 
Please login to merge, or discard this patch.
modules/Leads/Lead.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -476,6 +476,10 @@
 block discarded – undo
476 476
 	}
477 477
 
478 478
 //carrys forward custom lead fields to contacts, accounts, opportunities during Lead Conversion
479
+	/**
480
+	 * @param string $form
481
+	 * @param Opportunity $tempBean
482
+	 */
479 483
 	function convertCustomFieldsForm(&$form, &$tempBean, &$prefix) {
480 484
 
481 485
 		global $mod_strings, $app_list_strings, $app_strings, $lbl_required_symbol;
Please login to merge, or discard this patch.
modules/Leads/LeadFormBase.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@
 block discarded – undo
53 53
  * @see checkForDuplicates (method), ContactFormBase.php, LeadFormBase.php, ProspectFormBase.php
54 54
  * @param $focus sugarbean
55 55
  * @param $prefix String value of prefix that may be present in $_POST variables
56
- * @return SQL String of the query that should be used for the initial duplicate lookup check
56
+ * @return string String of the query that should be used for the initial duplicate lookup check
57 57
  */
58 58
 public function getDuplicateQuery($focus, $prefix='')
59 59
 {
Please login to merge, or discard this patch.
modules/Meetings/Meeting.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 
129 129
 	/**
130 130
 	 * Disable edit if meeting is recurring and source is not Sugar. It should be edited only from Outlook.
131
-	 * @param $view string
131
+	 * @param string $view string
132 132
 	 * @param $is_owner bool
133 133
 	 */
134 134
 	function ACLAccess($view,$is_owner='not_set',$in_group='not_set'){
@@ -656,6 +656,9 @@  discard block
 block discarded – undo
656 656
 	}
657 657
 
658 658
 
659
+	/**
660
+	 * @param string $status
661
+	 */
659 662
 	function set_accept_status(&$user,$status)
660 663
 	{
661 664
 		if($user->object_name == 'User')
Please login to merge, or discard this patch.
modules/MergeRecords/MergeRecord.php 1 patch
Doc Comments   +7 added lines patch added patch discarded remove patch
@@ -115,6 +115,10 @@  discard block
 block discarded – undo
115 115
     }
116 116
 
117 117
     // Bug 22994, when the search key words are in other module, there needs to be another merge_bean.
118
+
119
+    /**
120
+     * @param string $merge_module
121
+     */
118 122
     function load_merge_bean2($merge_module, $load_module_strings = false, $merge_id = '') {
119 123
         global $moduleList;
120 124
         global $beanList;
@@ -246,6 +250,9 @@  discard block
 block discarded – undo
246 250
     	return $query;
247 251
     }
248 252
     
253
+    /**
254
+     * @param string $search_type
255
+     */
249 256
     function release_name_query($search_type, $value) {
250 257
     	$this->load_merge_bean2('Releases');
251 258
     	if($search_type=='like') {
Please login to merge, or discard this patch.
modules/ModuleBuilder/controller.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -669,6 +669,11 @@
 block discarded – undo
669 669
         }
670 670
     }
671 671
 
672
+    /**
673
+     * @param boolean $label
674
+     * @param boolean $labelvalue
675
+     * @param boolean $modulename
676
+     */
672 677
     function DeleteLabel($language, $label, $labelvalue, $modulename, $basepath = null, $forRelationshipLabel = false)
673 678
     {
674 679
         // remove the label
Please login to merge, or discard this patch.
modules/ModuleBuilder/MB/AjaxCompose.php 1 patch
Doc Comments   +7 added lines patch added patch discarded remove patch
@@ -40,6 +40,10 @@  discard block
 block discarded – undo
40 40
 class AjaxCompose{
41 41
 	var $sections = array();
42 42
 	var $crumbs = array('Home'=>'ModuleBuilder.main("Home")',/* 'Assistant'=>'Assistant.mbAssistant.xy=Array("650, 40"); Assistant.mbAssistant.show();'*/);
43
+
44
+	/**
45
+	 * @param string $name
46
+	 */
43 47
 	function addSection($name, $title, $content, $action='activate'){
44 48
 		$crumb = '';
45 49
 		if($name == 'center'){
@@ -58,6 +62,9 @@  discard block
 block discarded – undo
58 62
 		return $json->encode($this->sections);
59 63
 	}
60 64
 	
65
+	/**
66
+	 * @param string $action
67
+	 */
61 68
 	function addCrumb($name, $action){
62 69
 		$this->crumbs[$name] = $action;
63 70
 	}
Please login to merge, or discard this patch.
modules/ModuleBuilder/MB/MBModule.php 1 patch
Doc Comments   +25 added lines patch added patch discarded remove patch
@@ -59,6 +59,11 @@  discard block
 block discarded – undo
59 59
 
60 60
     public $config_md5 = null ;
61 61
 
62
+    /**
63
+     * @param string $path
64
+     * @param string $package
65
+     * @param string $package_key
66
+     */
62 67
     function __construct ($name , $path , $package , $package_key)
63 68
     {
64 69
         global $mod_strings;
@@ -225,6 +230,9 @@  discard block
 block discarded – undo
225 230
         }
226 231
     }
227 232
 
233
+    /**
234
+     * @param string $key
235
+     */
228 236
     function deleteLabel ($language = 'en_us' , $key)
229 237
     {
230 238
    		foreach ($this->mblanguage->strings as $lang => $values) {
@@ -335,6 +343,10 @@  discard block
 block discarded – undo
335 343
         }
336 344
     }
337 345
 
346
+    /**
347
+     * @param string $from
348
+     * @param string $to
349
+     */
338 350
     function copyCustomFiles ( $from , $to )
339 351
     {
340 352
     	$d = dir ( $from ) ;
@@ -360,6 +372,10 @@  discard block
 block discarded – undo
360 372
         }
361 373
     }
362 374
 
375
+    /**
376
+     * @param string $from
377
+     * @param string $to
378
+     */
363 379
     function copyMetaRecursive ($from , $to , $overwrite = false)
364 380
     {
365 381
         if (! file_exists ( $from ))
@@ -431,6 +447,9 @@  discard block
 block discarded – undo
431 447
         }
432 448
     }
433 449
 
450
+    /**
451
+     * @param string $path
452
+     */
434 453
     function createClasses ($path)
435 454
     {
436 455
         $class = array ( ) ;
@@ -497,6 +516,9 @@  discard block
 block discarded – undo
497 516
         }
498 517
     }
499 518
 
519
+    /**
520
+     * @param string $path
521
+     */
500 522
     function createMenu ($path)
501 523
     {
502 524
         $smarty = new Sugar_Smarty ( ) ;
@@ -631,6 +653,9 @@  discard block
 block discarded – undo
631 653
         return $renamed ;
632 654
     }
633 655
 
656
+	/**
657
+	 * @param string $new_dir
658
+	 */
634 659
 	function renameLanguageFiles ($new_dir , $duplicate = false)
635 660
     {
636 661
 
Please login to merge, or discard this patch.