Passed
Push — scrutinizer-code-quality ( 09f5a1...c4c5fb )
by Adam
56:05 queued 14:08
created
soap/SoapRelationshipHelper.php 1 patch
Doc Comments   +8 added lines patch added patch discarded remove patch
@@ -146,6 +146,9 @@  discard block
 block discarded – undo
146 146
  *         error Mixed Array containing the SOAP errors if found, empty otherwise
147 147
  *
148 148
  */
149
+/**
150
+ * @param integer $show_deleted
151
+ */
149 152
 function retrieve_modified_relationships($module_name, $related_module, $relationship_query, $show_deleted, $offset, $max_results, $select_fields = array(), $relationship_name = ''){
150 153
 
151 154
     global  $beanList, $beanFiles, $dictionary, $current_user;
@@ -480,6 +483,11 @@  discard block
 block discarded – undo
480 483
 
481 484
 // Retrieves array of ids for records of $get_module linked to $from_module by $get_id
482 485
 // Example: to retrieve list of Contacts associated to Account X: $return = get_linked_records("Contacts", "Accounts", "X");
486
+/**
487
+ * @param string $get_module
488
+ * @param string $from_module
489
+ * @param string $get_id
490
+ */
483 491
 function get_linked_records($get_module, $from_module, $get_id) {
484 492
 	global $beanList, $beanFiles;
485 493
 
Please login to merge, or discard this patch.
soap/SoapSugarUsers.php 1 patch
Doc Comments   +10 added lines, -8 removed lines patch added patch discarded remove patch
@@ -168,8 +168,8 @@  discard block
 block discarded – undo
168 168
  * Check to see if the soap server and client are on the same machine.
169 169
  * We don't allow a server to sync to itself.
170 170
  *
171
- * @return true -- if the SOAP server and client are on the same machine
172
- * @return false -- if the SOAP server and client are not on the same machine.
171
+ * @return integer -- if the SOAP server and client are on the same machine
172
+ * @return integer -- if the SOAP server and client are not on the same machine.
173 173
  */
174 174
 function is_loopback(){
175 175
 	if(query_client_ip() == $_SERVER['SERVER_ADDR'])
@@ -181,8 +181,8 @@  discard block
 block discarded – undo
181 181
  * Validate the provided session information is correct and current.  Load the session.
182 182
  *
183 183
  * @param String $session_id -- The session ID that was returned by a call to login.
184
- * @return true -- If the session is valid and loaded.
185
- * @return false -- if the session is not valid.
184
+ * @return boolean -- If the session is valid and loaded.
185
+ * @return boolean -- if the session is not valid.
186 186
  */
187 187
 function validate_authenticated($session_id){
188 188
 	if(!empty($session_id)){
@@ -261,8 +261,8 @@  discard block
 block discarded – undo
261 261
  * Perform a seamless login.  This is used internally during the sync process.
262 262
  *
263 263
  * @param String $session -- Session ID returned by a previous call to login.
264
- * @return true -- if the session was authenticated
265
- * @return false -- if the session could not be authenticated
264
+ * @return integer -- if the session was authenticated
265
+ * @return integer -- if the session could not be authenticated
266 266
  */
267 267
 function seamless_login($session){
268 268
 		if(!validate_authenticated($session)){
@@ -464,7 +464,7 @@  discard block
 block discarded – undo
464 464
  *
465 465
  * @param String $session -- Session ID returned by a previous call to login.
466 466
  * @param String $module_name -- The name of the module to return records from.  This name should be the name the module was developed under (changing a tab name is studio does not affect the name that should be passed into this method)..
467
- * @param Array $ids -- An array of SugarBean IDs.
467
+ * @param string[] $ids -- An array of SugarBean IDs.
468 468
  * @param Array $select_fields -- A list of the fields to be included in the results. This optional parameter allows for only needed fields to be retrieved.
469 469
  * @return Array 'field_list' -- Var def information about the returned fields
470 470
  *               'entry_list' -- The records that were retrieved
@@ -680,7 +680,6 @@  discard block
 block discarded – undo
680 680
 /**
681 681
  * Retrieve an attachment from a note
682 682
  * @param String $session -- Session ID returned by a previous call to login.
683
- * @param Binary $note -- The flie contents of the attachment.
684 683
  * @return Array 'id' -- The ID of the new note or -1 on error
685 684
  *               'error' -- The SOAP error if any.
686 685
  *
@@ -2152,6 +2151,9 @@  discard block
 block discarded – undo
2152 2151
 }
2153 2152
 
2154 2153
 // INTERNAL FUNCTION NOT EXPOSED THROUGH API
2154
+/**
2155
+ * @param string $module_name
2156
+ */
2155 2157
 function handle_set_entries($module_name, $name_value_lists, $select_fields = FALSE) {
2156 2158
 	global $beanList, $beanFiles, $app_list_strings, $current_user;
2157 2159
 
Please login to merge, or discard this patch.
SugarSecurity.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -72,6 +72,9 @@
 block discarded – undo
72 72
 		}
73 73
 	}
74 74
 	
75
+	/**
76
+	 * @param string $contents
77
+	 */
75 78
 	function scanContents($contents){
76 79
 		return;	
77 80
 	}
Please login to merge, or discard this patch.
XTemplate/xtpl.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -278,6 +278,9 @@  discard block
 block discarded – undo
278 278
 /*
279 279
 	returns true if a block exists otherwise returns false.
280 280
 */
281
+/**
282
+ * @param string $bname
283
+ */
281 284
 function exists($bname){
282 285
 	return (!empty($this->parsed_blocks[$bname])) || (!empty($this->blocks[$bname]));
283 286
 }
@@ -325,6 +328,9 @@  discard block
 block discarded – undo
325 328
 	returns the parsed text for a block
326 329
 */
327 330
 
331
+/**
332
+ * @return string
333
+ */
328 334
 function text($bname) {
329 335
 
330 336
     if(!empty($this->parsed_blocks)){
@@ -446,6 +452,9 @@  discard block
 block discarded – undo
446 452
 */
447 453
 
448 454
 
455
+/**
456
+ * @param string $block
457
+ */
449 458
 function maketree($con,$block) {
450 459
 	$con2=explode($this->block_start_delim,$con);
451 460
 	$level=0;
Please login to merge, or discard this patch.
include/generic/SugarWidgets/SugarWidgetField.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -361,6 +361,9 @@
 block discarded – undo
361 361
 		return $this->recursiveVariableReplace($defs, $module, $var_values);
362 362
 	}
363 363
 
364
+	/**
365
+	 * @return string
366
+	 */
364 367
 	public function getModuleDir(){
365 368
 		return $this->module_dir;
366 369
 	}
Please login to merge, or discard this patch.
include/HTTP_WebDAV_Server/Tools/_parse_lockinfo.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -58,6 +58,7 @@
 block discarded – undo
58 58
 	 * constructor
59 59
 	 *
60 60
 	 * @access public
61
+	 * @param string $path
61 62
 	 */
62 63
 	function _parse_propfind($path) 
63 64
 	{
Please login to merge, or discard this patch.
include/Pear/Crypt_Blowfish/Blowfish.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
     /**
33 33
      * @param InputInterface $input
34 34
      * @param OutputInterface $output
35
-     * @return bool
35
+     * @return boolean|null
36 36
      */
37 37
     protected function execute(InputInterface $input, OutputInterface $output) {
38 38
         parent::_execute($input, $output);
Please login to merge, or discard this patch.
include/Smarty/internals/core.load_plugins.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -18,6 +18,9 @@
 block discarded – undo
18 18
 
19 19
  // $tpl_file, $cache_id, $compile_id, $results
20 20
 
21
+/**
22
+ * @param Smarty $smarty
23
+ */
21 24
 function smarty_core_write_cache_file($params, &$smarty)
22 25
 {
23 26
 
Please login to merge, or discard this patch.
include/Smarty/internals/core.load_resource_plugin.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -18,6 +18,9 @@
 block discarded – undo
18 18
 
19 19
  // $tpl_file, $cache_id, $compile_id, $results
20 20
 
21
+/**
22
+ * @param Smarty $smarty
23
+ */
21 24
 function smarty_core_write_cache_file($params, &$smarty)
22 25
 {
23 26
 
Please login to merge, or discard this patch.