Passed
Push — scrutinizer-code-quality ( 09f5a1...c4c5fb )
by Adam
56:05 queued 14:08
created
modules/Users/User.php 1 patch
Doc Comments   +4 added lines, -9 removed lines patch added patch discarded remove patch
@@ -269,7 +269,7 @@  discard block
 block discarded – undo
269 269
 	 *
270 270
 	 * @param string $name Name of the preference to set
271 271
 	 * @param string $value Value to set preference to
272
-	 * @param null $nosession For BC, ignored
272
+	 * @param integer $nosession For BC, ignored
273 273
 	 * @param string $category Name of the category to retrieve
274 274
 	 */
275 275
 	public function setPreference(
@@ -560,10 +560,6 @@  discard block
 block discarded – undo
560 560
 
561 561
 	/**
562 562
 	 * @deprecated
563
-	* @param string $user_name - Must be non null and at least 2 characters
564
-	* @param string $user_password - Must be non null and at least 1 character.
565
-	* @desc Take an unencrypted username and password and return the encrypted password
566
-	* @return string encrypted password for storage in DB and comparison against DB password.
567 563
 	*/
568 564
 	function encrypt_password($user_password)
569 565
 	{
@@ -598,8 +594,8 @@  discard block
 block discarded – undo
598 594
      *
599 595
      * @param string id ID of the User
600 596
      * @param bool encode encode the result
601
-     * @return object User bean
602
-     * @return null null if no User found
597
+     * @return SugarBean|null User bean
598
+     * @return SugarBean|null null if no User found
603 599
      */
604 600
 	function retrieve($id = -1, $encode = true, $deleted = true) {
605 601
 		$ret = parent::retrieve($id, $encode, $deleted);
@@ -715,7 +711,7 @@  discard block
 block discarded – undo
715 711
 
716 712
 	/**
717 713
 	 * Check that md5-encoded password matches existing hash
718
-	 * @param string $password MD5-encoded password
714
+	 * @param string $password_md5 MD5-encoded password
719 715
 	 * @param string $user_hash DB hash
720 716
 	 * @return bool Match or not?
721 717
 	 */
@@ -1627,7 +1623,6 @@  discard block
 block discarded – undo
1627 1623
     /**
1628 1624
      * Get user first day of week.
1629 1625
      *
1630
-     * @param [User] $user user object, current user if not specified
1631 1626
      * @return int : 0 = Sunday, 1 = Monday, etc...
1632 1627
      */
1633 1628
     public function get_first_day_of_week()
Please login to merge, or discard this patch.
include/Pear/XML_HTMLSax3/HTMLSax3/Decorators.php 1 patch
Doc Comments   +11 added lines patch added patch discarded remove patch
@@ -112,6 +112,8 @@  discard block
 block discarded – undo
112 112
     * @param object handler object being decorated
113 113
     * @param string original open handler method
114 114
     * @param string original close handler method
115
+    * @param string $orig_open_method
116
+    * @param string $orig_close_method
115 117
     * @access protected
116 118
     */
117 119
     function __construct(&$orig_obj, $orig_open_method, $orig_close_method) {
@@ -162,6 +164,7 @@  discard block
 block discarded – undo
162 164
     * Constructs XML_HTMLSax3_LineFeed
163 165
     * @param object handler object being decorated
164 166
     * @param string original handler method
167
+    * @param string $orig_method
165 168
     * @access protected
166 169
     */
167 170
     function XML_HTMLSax3_LineFeed(&$orig_obj, $orig_method) {
@@ -204,6 +207,7 @@  discard block
 block discarded – undo
204 207
     * Constructs XML_HTMLSax3_Tab
205 208
     * @param object handler object being decorated
206 209
     * @param string original handler method
210
+    * @param string $orig_method
207 211
     * @access protected
208 212
     */
209 213
     function __construct(&$orig_obj, $orig_method) {
@@ -247,6 +251,7 @@  discard block
 block discarded – undo
247 251
     * Constructs XML_HTMLSax3_Entities_Parsed
248 252
     * @param object handler object being decorated
249 253
     * @param string original handler method
254
+    * @param string $orig_method
250 255
     * @access protected
251 256
     */
252 257
     function __construct(&$orig_obj, $orig_method) {
@@ -271,6 +276,10 @@  discard block
 block discarded – undo
271 276
 * Compatibility with older PHP versions
272 277
 */
273 278
 if (version_compare(phpversion(), '4.3', '<') && !function_exists('html_entity_decode') ) {
279
+
280
+    /**
281
+     * @param integer $style
282
+     */
274 283
     function html_entity_decode($str, $style=ENT_NOQUOTES) {
275 284
         return strtr($str,
276 285
             array_flip(get_html_translation_table(HTML_ENTITIES,$style)));
@@ -299,6 +308,7 @@  discard block
 block discarded – undo
299 308
     * Constructs XML_HTMLSax3_Entities_Unparsed
300 309
     * @param object handler object being decorated
301 310
     * @param string original handler method
311
+    * @param string $orig_method
302 312
     * @access protected
303 313
     */
304 314
     function __construct(&$orig_obj, $orig_method) {
@@ -342,6 +352,7 @@  discard block
 block discarded – undo
342 352
     * Constructs XML_HTMLSax3_Entities_Unparsed
343 353
     * @param object handler object being decorated
344 354
     * @param string original handler method
355
+    * @param string $orig_method
345 356
     * @access protected
346 357
     */
347 358
     function __construct(&$orig_obj, $orig_method) {
Please login to merge, or discard this patch.
include/Popups/PopupSmarty.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -123,6 +123,7 @@
 block discarded – undo
123 123
      * @param file file Template file to use
124 124
      * @param data array from ListViewData
125 125
      * @param html_var string the corresponding html var in xtpl per row
126
+     * @param string $htmlVar
126 127
      *
127 128
      */
128 129
 	function process($file, $data, $htmlVar) {
Please login to merge, or discard this patch.
include/utils/db_utils.php 1 patch
Doc Comments   +2 added lines patch added patch discarded remove patch
@@ -42,6 +42,8 @@
 block discarded – undo
42 42
 
43 43
 /**
44 44
  * @deprecated use DBManager::convert() instead.
45
+ * @param string $string
46
+ * @param string $type
45 47
  */
46 48
 function db_convert($string, $type, $additional_parameters=array(),$additional_parameters_oracle_only=array())
47 49
 	{
Please login to merge, or discard this patch.
install/UserDemoData.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -180,6 +180,7 @@
 block discarded – undo
180 180
 	
181 181
 	/**
182 182
 	 * 
183
+	 * @param string $name
183 184
 	 */
184 185
 	function _quick_create($name)
185 186
 	{
Please login to merge, or discard this patch.
modules/ModuleBuilder/MB/MBLanguage.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -67,6 +67,9 @@  discard block
 block discarded – undo
67 67
 			$this->generateAppStrings();
68 68
 		}
69 69
 
70
+		/**
71
+		 * @param string $file
72
+		 */
70 73
 		function loadStrings($file)
71 74
         {
72 75
             $module = strtoupper($this->name);
@@ -90,6 +93,9 @@  discard block
 block discarded – undo
90 93
 			}
91 94
 		}
92 95
 
96
+	    /**
97
+	     * @param string $file
98
+	     */
93 99
 	    function loadAppListStrings($file){
94 100
             if(!file_exists($file))return;
95 101
 			//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/MBVardefs.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -68,6 +68,10 @@
 block discarded – undo
68 68
     }
69 69
 
70 70
 
71
+	/**
72
+	 * @param boolean $by_group
73
+	 * @param string $file
74
+	 */
71 75
 	function loadTemplate($by_group, $template, $file){
72 76
 		$module = $this->name;
73 77
 		$table_name = $this->name;
Please login to merge, or discard this patch.