Completed
Pull Request — master (#5280)
by Maxence
16:09
created
apps/theming/lib/ThemingDefaults.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -58,7 +58,6 @@
 block discarded – undo
58 58
 	 * @param IConfig $config
59 59
 	 * @param IL10N $l
60 60
 	 * @param IURLGenerator $urlGenerator
61
-	 * @param \OC_Defaults $defaults
62 61
 	 * @param IAppData $appData
63 62
 	 * @param ICacheFactory $cacheFactory
64 63
 	 * @param Util $util
Please login to merge, or discard this patch.
apps/theming/lib/Util.php 1 patch
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 	/**
72 72
 	 * get color for on-page elements:
73 73
 	 * theme color by default, grey if theme color is to bright
74
-	 * @param $color
74
+	 * @param string $color
75 75
 	 * @return string
76 76
 	 */
77 77
 	public function elementColor($color) {
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 
115 115
 	/**
116 116
 	 * @param $app string app name
117
-	 * @return string|ISimpleFile path to app icon / file of logo
117
+	 * @return string path to app icon / file of logo
118 118
 	 */
119 119
 	public function getAppIcon($app) {
120 120
 		$app = str_replace(array('\0', '/', '\\', '..'), '', $app);
@@ -190,8 +190,8 @@  discard block
 block discarded – undo
190 190
 	/**
191 191
 	 * replace default color with a custom one
192 192
 	 *
193
-	 * @param $svg string content of a svg file
194
-	 * @param $color string color to match
193
+	 * @param string $svg string content of a svg file
194
+	 * @param string $color string color to match
195 195
 	 * @return string
196 196
 	 */
197 197
 	public function colorizeSvg($svg, $color) {
Please login to merge, or discard this patch.
lib/private/legacy/template/functions.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -42,7 +42,6 @@  discard block
 block discarded – undo
42 42
  * Prints a <script> tag with nonce and defer depending on config
43 43
  * @param string $src the source URL, ignored when empty
44 44
  * @param string $script_content the inline script content, ignored when empty
45
- * @param bool $defer_flag deferred loading or not
46 45
 */
47 46
 function emit_script_tag($src, $script_content) {
48 47
 	$defer_str=' defer';
@@ -76,7 +75,7 @@  discard block
 block discarded – undo
76 75
 /**
77 76
  * Prints an unsanitized string - usage of this function may result into XSS.
78 77
  * Consider using p() instead.
79
- * @param string|array $string the string which will be printed as it is
78
+ * @param string $string the string which will be printed as it is
80 79
  */
81 80
 function print_unescaped($string) {
82 81
 	print($string);
@@ -219,7 +218,7 @@  discard block
 block discarded – undo
219 218
  * make preview_icon available as a simple function
220 219
  * Returns the path to the preview of the image.
221 220
  * @param string $path path of file
222
- * @return link to the preview
221
+ * @return string to the preview
223 222
  */
224 223
 function preview_icon( $path ) {
225 224
 	return \OC::$server->getURLGenerator()->linkToRoute('core.Preview.getPreview', ['x' => 32, 'y' => 32, 'file' => $path]);
@@ -227,6 +226,7 @@  discard block
 block discarded – undo
227 226
 
228 227
 /**
229 228
  * @param string $path
229
+ * @param string $token
230 230
  */
231 231
 function publicPreview_icon ( $path, $token ) {
232 232
 	return \OC::$server->getURLGenerator()->linkToRoute('files_sharing.PublicPreview.getPreview', ['x' => 32, 'y' => 32, 'file' => $path, 't' => $token]);
Please login to merge, or discard this patch.
apps/user_ldap/lib/Access.php 1 patch
Doc Comments   +8 added lines, -6 removed lines patch added patch discarded remove patch
@@ -491,7 +491,7 @@  discard block
 block discarded – undo
491 491
 
492 492
 	/**
493 493
 	 * returns the internal Nextcloud name for the given LDAP DN of the user, false on DN outside of search DN or failure
494
-	 * @param string $dn the dn of the user object
494
+	 * @param string $fdn the dn of the user object
495 495
 	 * @param string $ldapName optional, the display name of the object
496 496
 	 * @return string|false with with the name to use in Nextcloud
497 497
 	 */
@@ -508,7 +508,7 @@  discard block
 block discarded – undo
508 508
 
509 509
 	/**
510 510
 	 * returns an internal Nextcloud name for the given LDAP DN, false on DN outside of search DN
511
-	 * @param string $dn the dn of the user object
511
+	 * @param string $fdn the dn of the user object
512 512
 	 * @param string $ldapName optional, the display name of the object
513 513
 	 * @param bool $isUser optional, whether it is a user object (otherwise group assumed)
514 514
 	 * @return string|false with with the name to use in Nextcloud
@@ -771,7 +771,7 @@  discard block
 block discarded – undo
771 771
 	 * the login filter.
772 772
 	 *
773 773
 	 * @param string $loginName
774
-	 * @param array $attributes optional, list of attributes to read
774
+	 * @param string[] $attributes optional, list of attributes to read
775 775
 	 * @return array
776 776
 	 */
777 777
 	public function fetchUsersByLoginName($loginName, $attributes = array('dn')) {
@@ -844,7 +844,7 @@  discard block
 block discarded – undo
844 844
 
845 845
 	/**
846 846
 	 * @param string $filter
847
-	 * @param string|string[] $attr
847
+	 * @param string[] $attr
848 848
 	 * @param int $limit
849 849
 	 * @param int $offset
850 850
 	 * @return array
@@ -892,7 +892,7 @@  discard block
 block discarded – undo
892 892
 
893 893
 	/**
894 894
 	 * @param string $filter
895
-	 * @param string|string[] $attr
895
+	 * @param string[] $attr
896 896
 	 * @param int $limit
897 897
 	 * @param int $offset
898 898
 	 * @return false|int
@@ -989,6 +989,7 @@  discard block
 block discarded – undo
989 989
 	 * retrieved. Results will according to the order in the array.
990 990
 	 * @param int $limit optional, maximum results to be counted
991 991
 	 * @param int $offset optional, a starting point
992
+	 * @param string $filter
992 993
 	 * @return array|false array with the search result as first value and pagedSearchOK as
993 994
 	 * second | false if not successful
994 995
 	 * @throws \OC\ServerNotAvailableException
@@ -1240,7 +1241,7 @@  discard block
 block discarded – undo
1240 1241
 
1241 1242
 	/**
1242 1243
 	 * @param string $name
1243
-	 * @return bool|mixed|string
1244
+	 * @return string
1244 1245
 	 */
1245 1246
 	public function sanitizeUsername($name) {
1246 1247
 		if($this->connection->ldapIgnoreNamingRules) {
@@ -1264,6 +1265,7 @@  discard block
 block discarded – undo
1264 1265
 	* escapes (user provided) parts for LDAP filter
1265 1266
 	* @param string $input, the provided value
1266 1267
 	* @param bool $allowAsterisk whether in * at the beginning should be preserved
1268
+	* @param string $input
1267 1269
 	* @return string the escaped string
1268 1270
 	*/
1269 1271
 	public function escapeFilterPart($input, $allowAsterisk = false) {
Please login to merge, or discard this patch.