Passed
Push — scrutinizer-code-quality ( eedb15...27193c )
by Adam
54:15 queued 15s
created
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/MBLanguage.php 1 patch
Doc Comments   +11 added lines patch added patch discarded remove patch
@@ -40,6 +40,11 @@  discard block
 block discarded – undo
40 40
 class MBLanguage{
41 41
 		var $iTemplates = array();
42 42
 		var $templates = array();
43
+
44
+		/**
45
+		 * @param string $name
46
+		 * @param string $path
47
+		 */
43 48
 		function MBLanguage( $name, $path, $label, $key_name){
44 49
 			$this->path = $path;
45 50
 			$this->name = $name;
@@ -52,6 +57,9 @@  discard block
 block discarded – undo
52 57
 			$this->generateAppStrings();
53 58
 		}
54 59
 		
60
+		/**
61
+		 * @param string $file
62
+		 */
55 63
 		function loadStrings($file)
56 64
         {
57 65
             $module = strtoupper($this->name);
@@ -75,6 +83,9 @@  discard block
 block discarded – undo
75 83
 			}
76 84
 		}
77 85
 		
86
+	    /**
87
+	     * @param string $file
88
+	     */
78 89
 	    function loadAppListStrings($file){
79 90
             if(!file_exists($file))return;
80 91
 			//we may not need this when loading in the app strings, but there is no harm
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.
modules/ModuleBuilder/MB/MBPackage.php 1 patch
Doc Comments   +15 added lines, -1 removed lines patch added patch discarded remove patch
@@ -184,6 +184,9 @@  discard block
 block discarded – undo
184 184
 */
185 185
 }
186 186
     
187
+/**
188
+ * @param string $path
189
+ */
187 190
 function buildInstall($path){
188 191
     $installdefs = array ('id' => $this->name,
189 192
         'beans'=>array(),
@@ -371,6 +374,10 @@  discard block
 block discarded – undo
371 374
     
372 375
     
373 376
         //creation of the installdefs[] array for the manifest when exporting customizations
377
+
378
+    /**
379
+     * @param string $path
380
+     */
374 381
     function customBuildInstall($modules, $path, $extensions = array()){
375 382
         $columns=$this->getColumnsName();
376 383
         $installdefs = array ('id' => $this->name, 'relationships' => array());
@@ -629,6 +636,10 @@  discard block
 block discarded – undo
629 636
             }
630 637
         }
631 638
     }
639
+
640
+    /**
641
+     * @param string $path
642
+     */
632 643
     private function copyCustomDropdownValuesForModules($modules, $path)
633 644
     {
634 645
         if(file_exists("custom/include/language")){
@@ -862,7 +873,7 @@  discard block
 block discarded – undo
862 873
      * Returns a set of field defs for fields that will exist when this package is deployed
863 874
      * based on the relationships in all of its modules.
864 875
      * 
865
-     * @param $moduleName (module must be from whithin this package)
876
+     * @param string $moduleName (module must be from whithin this package)
866 877
      * @return array Field defs
867 878
      */
868 879
     function getRelationshipsForModule($moduleName) {
@@ -885,6 +896,9 @@  discard block
 block discarded – undo
885 896
     
886 897
 
887 898
     
899
+    /**
900
+     * @param boolean $for_export
901
+     */
888 902
     function exportProjectInstall($package, $for_export){
889 903
         $pre = $for_export ? MB_EXPORTPREPEND : "";
890 904
         $installdefs = array ('id' => $pre . $this->name);
Please login to merge, or discard this patch.
modules/ModuleBuilder/MB/MBPackageTree.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -52,6 +52,9 @@
 block discarded – undo
52 52
 		return 'Packages';
53 53
 	}
54 54
 	
55
+	/**
56
+	 * @param Tree $parent
57
+	 */
55 58
 	function populateTree($nodes, &$parent){
56 59
 		foreach($nodes as $node){
57 60
 			if(empty($node['label']))$node['label'] = $node['name'];
Please login to merge, or discard this patch.
modules/ModuleBuilder/MB/MBVardefs.php 1 patch
Doc Comments   +8 added lines patch added patch discarded remove patch
@@ -46,6 +46,10 @@  discard block
 block discarded – undo
46 46
 	var $name = '';
47 47
 	var $errors = array();
48 48
 
49
+	/**
50
+	 * @param string $path
51
+	 * @param string $key_name
52
+	 */
49 53
 	function MBVardefs($name, $path, $key_name){
50 54
 		$this->path = $path;
51 55
 		$this->name = $name;
@@ -53,6 +57,10 @@  discard block
 block discarded – undo
53 57
 		$this->load();
54 58
 	}
55 59
 
60
+	/**
61
+	 * @param boolean $by_group
62
+	 * @param string $file
63
+	 */
56 64
 	function loadTemplate($by_group, $template, $file){
57 65
 		$module = $this->name;
58 66
 		$table_name = $this->name;
Please login to merge, or discard this patch.