Passed
Push — scrutinizer-code-quality ( eedb15...27193c )
by Adam
54:15 queued 15s
created
modules/ModuleBuilder/parsers/views/UndeployedSubpanelImplementation.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -61,6 +61,10 @@
 block discarded – undo
61 61
      * @param string moduleName     The name of the module to which this subpanel belongs
62 62
      * @param string packageName    If not empty, the name of the package to which this subpanel belongs
63 63
      */
64
+
65
+    /**
66
+     * @param string $packageName
67
+     */
64 68
     function __construct ($subpanelName , $moduleName , $packageName)
65 69
     {
66 70
         $this->_subpanelName = $subpanelName ;
Please login to merge, or discard this patch.
modules/ModuleBuilder/views/view.listview.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -177,6 +177,9 @@
 block discarded – undo
177 177
         return $ajax ;
178 178
     }
179 179
 
180
+    /**
181
+     * @param AbstractMetaDataParser|null $parser
182
+     */
180 183
     function constructSmarty ($parser)
181 184
     {
182 185
         global $mod_strings;
Please login to merge, or discard this patch.
modules/ModuleBuilder/views/view.modulefield.php 1 patch
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -364,8 +364,7 @@
 block discarded – undo
364 364
      * This function overrides fetchTemplate from SugarView.  For view.modulefield.php we go through the FieldViewer
365 365
      * class to fetch the display contents.
366 366
      *
367
-     * @param FieldViewer $mixed the FieldViewer instance
368
-     * @param string $template the file to fetch
367
+     * @param FieldViewer $fv
369 368
      * @return string contents from calling the fetch method on the FieldViewer Sugar_Smarty instance
370 369
      */
371 370
     protected function fetchTemplate($fv/*, $template*/)
Please login to merge, or discard this patch.
modules/ModuleBuilder/views/view.relationships.php 1 patch
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -156,8 +156,7 @@
 block discarded – undo
156 156
      * fetchTemplate
157 157
      * This function overrides fetchTemplate from SugarView.
158 158
      *
159
-     * @param FieldViewer $mixed the Sugar_Smarty instance
160
-     * @param string $template the file to fetch
159
+     * @param Sugar_Smarty $smarty
161 160
      * @return string contents from calling the fetch method on the Sugar_Smarty instance
162 161
      */
163 162
     protected function fetchTemplate($smarty/*, $template*/)
Please login to merge, or discard this patch.
modules/ModuleBuilder/views/view.resetmodule.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
     /**
133 133
      * Removes all custom fields created in studio
134 134
      * 
135
-     * @return html output record of the field deleted
135
+     * @return string output record of the field deleted
136 136
      */
137 137
     function removeCustomFields() 
138 138
     {    
@@ -165,7 +165,7 @@  discard block
 block discarded – undo
165 165
     /**
166 166
      * Removes the metadata files for all known studio layouts.
167 167
      * 
168
-     * @return html output record of the files deleted
168
+     * @return string output record of the files deleted
169 169
      */
170 170
     function removeCustomLayouts() 
171 171
     {
@@ -192,7 +192,7 @@  discard block
 block discarded – undo
192 192
     /**
193 193
      * Removes all custom relationships containing this module
194 194
      * 
195
-     * @return html output record of the files deleted
195
+     * @return string output record of the files deleted
196 196
      */
197 197
     function removeCustomRelationships() 
198 198
     {
Please login to merge, or discard this patch.
modules/ModuleBuilder/views/view.wizard.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -100,6 +100,9 @@  discard block
 block discarded – undo
100 100
 		echo $this->ajax->getJavascript () ;
101 101
 	}
102 102
 
103
+	/**
104
+	 * @param AjaxCompose $ajax
105
+	 */
103 106
 	function processStudio( 
104 107
 	    $ajax
105 108
 	    )
@@ -185,6 +188,9 @@  discard block
 block discarded – undo
185 188
 		}
186 189
 	}
187 190
 
191
+	/**
192
+	 * @param AjaxCompose $ajax
193
+	 */
188 194
 	function processMB ( 
189 195
 	    $ajax 
190 196
 	    )
Please login to merge, or discard this patch.
modules/MySettings/TabController.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -145,6 +145,9 @@
 block discarded – undo
145 145
 return true;
146 146
 }
147 147
 
148
+/**
149
+ * @param boolean $boolean
150
+ */
148 151
 function set_users_can_edit($boolean){
149 152
 	global $current_user;
150 153
 	if(is_admin($current_user)){
Please login to merge, or discard this patch.
modules/Notes/NoteSoap.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -144,6 +144,9 @@
 block discarded – undo
144 144
         return $return_id;
145 145
     }
146 146
 
147
+    /**
148
+     * @param string $filename
149
+     */
147 150
     function retrieveFile($id, $filename)
148 151
     {
149 152
     	if(empty($filename)){
Please login to merge, or discard this patch.
modules/Opportunities/SaveOverload.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -39,6 +39,9 @@
 block discarded – undo
39 39
  ********************************************************************************/
40 40
 
41 41
 
42
+/**
43
+ * @param Opportunity $focus
44
+ */
42 45
 function perform_save(&$focus){
43 46
 	//US DOLLAR
44 47
 	if(isset($focus->amount) && !number_empty($focus->amount)){
Please login to merge, or discard this patch.