Completed
Pull Request — master (#19)
by Lukas
08:58
created
apps/user_ldap/lib/Command/SetConfig.php 1 patch
Doc Comments   -2 removed lines patch added patch discarded remove patch
@@ -74,8 +74,6 @@
 block discarded – undo
74 74
 	/**
75 75
 	 * save the configuration value as provided
76 76
 	 * @param string $configID
77
-	 * @param string $configKey
78
-	 * @param string $configValue
79 77
 	 */
80 78
 	protected function setValue($configID, $key, $value) {
81 79
 		$configHolder = new Configuration($configID);
Please login to merge, or discard this patch.
apps/user_ldap/lib/Jobs/UpdateGroups.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -77,7 +77,7 @@
 block discarded – undo
77 77
 	}
78 78
 
79 79
 	/**
80
-	 * @return int
80
+	 * @return string
81 81
 	 */
82 82
 	static private function getRefreshInterval() {
83 83
 		//defaults to every hour
Please login to merge, or discard this patch.
apps/user_ldap/lib/Wizard.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -867,7 +867,7 @@  discard block
 block discarded – undo
867 867
 	 * @param integer $filterType int, for which use case the filter shall be created
868 868
 	 * can be any of self::LFILTER_USER_LIST, self::LFILTER_LOGIN or
869 869
 	 * self::LFILTER_GROUP_LIST
870
-	 * @return string|false string with the filter on success, false otherwise
870
+	 * @return string string with the filter on success, false otherwise
871 871
 	 * @throws \Exception
872 872
 	 */
873 873
 	private function composeLdapFilter($filterType) {
@@ -1103,7 +1103,7 @@  discard block
 block discarded – undo
1103 1103
 	}
1104 1104
 
1105 1105
 	/**
1106
-	 * @param array $reqs
1106
+	 * @param string[] $reqs
1107 1107
 	 * @return bool
1108 1108
 	 */
1109 1109
 	private function checkRequirements($reqs) {
Please login to merge, or discard this patch.
core/Command/Maintenance/Repair.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,6 @@
 block discarded – undo
32 32
 use Symfony\Component\Console\Input\InputInterface;
33 33
 use Symfony\Component\Console\Input\InputOption;
34 34
 use Symfony\Component\Console\Output\OutputInterface;
35
-use Symfony\Component\EventDispatcher\EventDispatcher;
36 35
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
37 36
 use Symfony\Component\EventDispatcher\GenericEvent;
38 37
 
Please login to merge, or discard this patch.
core/Command/Security/ListCertificates.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,6 @@
 block discarded – undo
28 28
 use Symfony\Component\Console\Command\Command;
29 29
 use Symfony\Component\Console\Helper\Table;
30 30
 use Symfony\Component\Console\Input\InputInterface;
31
-use Symfony\Component\Console\Input\InputOption;
32 31
 use Symfony\Component\Console\Output\OutputInterface;
33 32
 
34 33
 class ListCertificates extends Base {
Please login to merge, or discard this patch.
core/Controller/LostController.php 1 patch
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -146,6 +146,7 @@  discard block
 block discarded – undo
146 146
 	/**
147 147
 	 * @param string $userId
148 148
 	 * @param string $userId
149
+	 * @param string $token
149 150
 	 * @throws \Exception
150 151
 	 */
151 152
 	private function checkPasswordResetToken($token, $userId) {
@@ -167,7 +168,7 @@  discard block
 block discarded – undo
167 168
 	}
168 169
 
169 170
 	/**
170
-	 * @param $message
171
+	 * @param string $message
171 172
 	 * @param array $additional
172 173
 	 * @return array
173 174
 	 */
Please login to merge, or discard this patch.
core/Middleware/TwoFactorMiddleware.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -96,6 +96,10 @@
 block discarded – undo
96 96
 		// TODO: dont check/enforce 2FA if a auth token is used
97 97
 	}
98 98
 
99
+	/**
100
+	 * @param Controller $controller
101
+	 * @param string $methodName
102
+	 */
99 103
 	private function checkTwoFactor($controller, $methodName) {
100 104
 		// If two-factor auth is in progress disallow access to any controllers
101 105
 		// defined within "LoginController".
Please login to merge, or discard this patch.
lib/private/AllConfig.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -112,7 +112,7 @@
 block discarded – undo
112 112
 	 * Looks up a system wide defined value
113 113
 	 *
114 114
 	 * @param string $key the key of the value, under which it was saved
115
-	 * @param mixed $default the default value to be returned if the value isn't set
115
+	 * @param string $default the default value to be returned if the value isn't set
116 116
 	 * @return mixed the value or $default
117 117
 	 */
118 118
 	public function getSystemValue($key, $default = '') {
Please login to merge, or discard this patch.
lib/private/App/AppManager.php 1 patch
Doc Comments   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 	/**
118 118
 	 * List all installed apps
119 119
 	 *
120
-	 * @return string[]
120
+	 * @return integer[]
121 121
 	 */
122 122
 	public function getInstalledApps() {
123 123
 		return array_keys($this->getInstalledAppsValues());
@@ -274,7 +274,7 @@  discard block
 block discarded – undo
274 274
 	/**
275 275
 	 * Returns a list of apps that need upgrade
276 276
 	 *
277
-	 * @param array $version ownCloud version as array of version components
277
+	 * @param array $ocVersion ownCloud version as array of version components
278 278
 	 * @return array list of app info from apps that need an upgrade
279 279
 	 *
280 280
 	 * @internal
@@ -344,6 +344,9 @@  discard block
 block discarded – undo
344 344
 		return in_array($appId, $this->shippedApps);
345 345
 	}
346 346
 
347
+	/**
348
+	 * @param string $appId
349
+	 */
347 350
 	private function isAlwaysEnabled($appId) {
348 351
 		$alwaysEnabled = $this->getAlwaysEnabledApps();
349 352
 		return in_array($appId, $alwaysEnabled);
Please login to merge, or discard this patch.