Passed
Push — master ( 4e9727...bdfd2d )
by Joas
15:17 queued 11s
created
core/Migrations/Version15000Date20181029084625.php 1 patch
Indentation   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -31,23 +31,23 @@
 block discarded – undo
31 31
 
32 32
 class Version15000Date20181029084625 extends SimpleMigrationStep {
33 33
 
34
-	/**
35
-	 * @param IOutput $output
36
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
37
-	 * @param array $options
38
-	 * @return null|ISchemaWrapper
39
-	 */
40
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
41
-		/** @var ISchemaWrapper $schema */
42
-		$schema = $schemaClosure();
34
+    /**
35
+     * @param IOutput $output
36
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
37
+     * @param array $options
38
+     * @return null|ISchemaWrapper
39
+     */
40
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
41
+        /** @var ISchemaWrapper $schema */
42
+        $schema = $schemaClosure();
43 43
 
44
-		$table = $schema->getTable('share');
45
-		$table->addColumn('label', 'string', [
46
-			'notnull' => false,
47
-			'length' => 255,
48
-		]);
44
+        $table = $schema->getTable('share');
45
+        $table->addColumn('label', 'string', [
46
+            'notnull' => false,
47
+            'length' => 255,
48
+        ]);
49 49
 
50
-		return $schema;
51
-	}
50
+        return $schema;
51
+    }
52 52
 
53 53
 }
Please login to merge, or discard this patch.
private/AppFramework/Middleware/Security/PasswordConfirmationMiddleware.php 1 patch
Indentation   +52 added lines, -52 removed lines patch added patch discarded remove patch
@@ -32,60 +32,60 @@
 block discarded – undo
32 32
 use OCP\User\Backend\IPasswordConfirmationBackend;
33 33
 
34 34
 class PasswordConfirmationMiddleware extends Middleware {
35
-	/** @var ControllerMethodReflector */
36
-	private $reflector;
37
-	/** @var ISession */
38
-	private $session;
39
-	/** @var IUserSession */
40
-	private $userSession;
41
-	/** @var ITimeFactory */
42
-	private $timeFactory;
43
-	/** @var array */
44
-	private $excludedUserBackEnds = ['user_saml' => true, 'user_globalsiteselector' => true];
35
+    /** @var ControllerMethodReflector */
36
+    private $reflector;
37
+    /** @var ISession */
38
+    private $session;
39
+    /** @var IUserSession */
40
+    private $userSession;
41
+    /** @var ITimeFactory */
42
+    private $timeFactory;
43
+    /** @var array */
44
+    private $excludedUserBackEnds = ['user_saml' => true, 'user_globalsiteselector' => true];
45 45
 
46
-	/**
47
-	 * PasswordConfirmationMiddleware constructor.
48
-	 *
49
-	 * @param ControllerMethodReflector $reflector
50
-	 * @param ISession $session
51
-	 * @param IUserSession $userSession
52
-	 * @param ITimeFactory $timeFactory
53
-	 */
54
-	public function __construct(ControllerMethodReflector $reflector,
55
-								ISession $session,
56
-								IUserSession $userSession,
57
-								ITimeFactory $timeFactory) {
58
-		$this->reflector = $reflector;
59
-		$this->session = $session;
60
-		$this->userSession = $userSession;
61
-		$this->timeFactory = $timeFactory;
62
-	}
46
+    /**
47
+     * PasswordConfirmationMiddleware constructor.
48
+     *
49
+     * @param ControllerMethodReflector $reflector
50
+     * @param ISession $session
51
+     * @param IUserSession $userSession
52
+     * @param ITimeFactory $timeFactory
53
+     */
54
+    public function __construct(ControllerMethodReflector $reflector,
55
+                                ISession $session,
56
+                                IUserSession $userSession,
57
+                                ITimeFactory $timeFactory) {
58
+        $this->reflector = $reflector;
59
+        $this->session = $session;
60
+        $this->userSession = $userSession;
61
+        $this->timeFactory = $timeFactory;
62
+    }
63 63
 
64
-	/**
65
-	 * @param Controller $controller
66
-	 * @param string $methodName
67
-	 * @throws NotConfirmedException
68
-	 */
69
-	public function beforeController($controller, $methodName) {
70
-		if ($this->reflector->hasAnnotation('PasswordConfirmationRequired')) {
71
-			$user = $this->userSession->getUser();
72
-			$backendClassName = '';
73
-			if ($user !== null) {
74
-				$backend = $user->getBackend();
75
-				if ($backend instanceof IPasswordConfirmationBackend) {
76
-					if (!$backend->canConfirmPassword($user->getUID())) {
77
-						return;
78
-					}
79
-				}
64
+    /**
65
+     * @param Controller $controller
66
+     * @param string $methodName
67
+     * @throws NotConfirmedException
68
+     */
69
+    public function beforeController($controller, $methodName) {
70
+        if ($this->reflector->hasAnnotation('PasswordConfirmationRequired')) {
71
+            $user = $this->userSession->getUser();
72
+            $backendClassName = '';
73
+            if ($user !== null) {
74
+                $backend = $user->getBackend();
75
+                if ($backend instanceof IPasswordConfirmationBackend) {
76
+                    if (!$backend->canConfirmPassword($user->getUID())) {
77
+                        return;
78
+                    }
79
+                }
80 80
 
81
-				$backendClassName = $user->getBackendClassName();
82
-			}
81
+                $backendClassName = $user->getBackendClassName();
82
+            }
83 83
 
84
-			$lastConfirm = (int) $this->session->get('last-password-confirm');
85
-			// we can't check the password against a SAML backend, so skip password confirmation in this case
86
-			if (!isset($this->excludedUserBackEnds[$backendClassName]) && $lastConfirm < ($this->timeFactory->getTime() - (30 * 60 + 15))) { // allow 15 seconds delay
87
-				throw new NotConfirmedException();
88
-			}
89
-		}
90
-	}
84
+            $lastConfirm = (int) $this->session->get('last-password-confirm');
85
+            // we can't check the password against a SAML backend, so skip password confirmation in this case
86
+            if (!isset($this->excludedUserBackEnds[$backendClassName]) && $lastConfirm < ($this->timeFactory->getTime() - (30 * 60 + 15))) { // allow 15 seconds delay
87
+                throw new NotConfirmedException();
88
+            }
89
+        }
90
+    }
91 91
 }
Please login to merge, or discard this patch.
lib/public/User/Backend/IPasswordConfirmationBackend.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -29,8 +29,8 @@
 block discarded – undo
29 29
  */
30 30
 interface IPasswordConfirmationBackend {
31 31
 
32
-	/**
33
-	 * @since 15.0.0
34
-	 */
35
-	public function canConfirmPassword(string $uid): bool;
32
+    /**
33
+     * @since 15.0.0
34
+     */
35
+    public function canConfirmPassword(string $uid): bool;
36 36
 }
Please login to merge, or discard this patch.
apps/dav/lib/CalDAV/CachedSubscriptionObject.php 1 patch
Indentation   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -33,32 +33,32 @@
 block discarded – undo
33 33
  */
34 34
 class CachedSubscriptionObject extends \Sabre\CalDAV\CalendarObject {
35 35
 
36
-	/**
37
-	 * @inheritdoc
38
-	 */
39
-	public function get() {
40
-		// Pre-populating the 'calendardata' is optional, if we don't have it
41
-		// already we fetch it from the backend.
42
-		if (!isset($this->objectData['calendardata'])) {
43
-			$this->objectData = $this->caldavBackend->getCalendarObject($this->calendarInfo['id'], $this->objectData['uri'], CalDavBackend::CALENDAR_TYPE_SUBSCRIPTION);
44
-		}
36
+    /**
37
+     * @inheritdoc
38
+     */
39
+    public function get() {
40
+        // Pre-populating the 'calendardata' is optional, if we don't have it
41
+        // already we fetch it from the backend.
42
+        if (!isset($this->objectData['calendardata'])) {
43
+            $this->objectData = $this->caldavBackend->getCalendarObject($this->calendarInfo['id'], $this->objectData['uri'], CalDavBackend::CALENDAR_TYPE_SUBSCRIPTION);
44
+        }
45 45
 
46
-		return $this->objectData['calendardata'];
47
-	}
46
+        return $this->objectData['calendardata'];
47
+    }
48 48
 
49
-	/**
50
-	 * @param resource|string $calendarData
51
-	 * @return string|void
52
-	 * @throws MethodNotAllowed
53
-	 */
54
-	public function put($calendarData) {
55
-		throw new MethodNotAllowed('Creating objects in a cached subscription is not allowed');
56
-	}
49
+    /**
50
+     * @param resource|string $calendarData
51
+     * @return string|void
52
+     * @throws MethodNotAllowed
53
+     */
54
+    public function put($calendarData) {
55
+        throw new MethodNotAllowed('Creating objects in a cached subscription is not allowed');
56
+    }
57 57
 
58
-	/**
59
-	 * @throws MethodNotAllowed
60
-	 */
61
-	public function delete() {
62
-		throw new MethodNotAllowed('Deleting objects in a cached subscription is not allowed');
63
-	}
58
+    /**
59
+     * @throws MethodNotAllowed
60
+     */
61
+    public function delete() {
62
+        throw new MethodNotAllowed('Deleting objects in a cached subscription is not allowed');
63
+    }
64 64
 }
Please login to merge, or discard this patch.
lib/public/IUser.php 1 patch
Indentation   +174 added lines, -174 removed lines patch added patch discarded remove patch
@@ -36,178 +36,178 @@
 block discarded – undo
36 36
  */
37 37
 interface IUser {
38 38
 
39
-	/**
40
-	 * get the user id
41
-	 *
42
-	 * @return string
43
-	 * @since 8.0.0
44
-	 */
45
-	public function getUID();
46
-
47
-	/**
48
-	 * get the display name for the user, if no specific display name is set it will fallback to the user id
49
-	 *
50
-	 * @return string
51
-	 * @since 8.0.0
52
-	 */
53
-	public function getDisplayName();
54
-
55
-	/**
56
-	 * set the display name for the user
57
-	 *
58
-	 * @param string $displayName
59
-	 * @return bool
60
-	 * @since 8.0.0
61
-	 */
62
-	public function setDisplayName($displayName);
63
-
64
-	/**
65
-	 * returns the timestamp of the user's last login or 0 if the user did never
66
-	 * login
67
-	 *
68
-	 * @return int
69
-	 * @since 8.0.0
70
-	 */
71
-	public function getLastLogin();
72
-
73
-	/**
74
-	 * updates the timestamp of the most recent login of this user
75
-	 * @since 8.0.0
76
-	 */
77
-	public function updateLastLoginTimestamp();
78
-
79
-	/**
80
-	 * Delete the user
81
-	 *
82
-	 * @return bool
83
-	 * @since 8.0.0
84
-	 */
85
-	public function delete();
86
-
87
-	/**
88
-	 * Set the password of the user
89
-	 *
90
-	 * @param string $password
91
-	 * @param string $recoveryPassword for the encryption app to reset encryption keys
92
-	 * @return bool
93
-	 * @since 8.0.0
94
-	 */
95
-	public function setPassword($password, $recoveryPassword = null);
96
-
97
-	/**
98
-	 * get the users home folder to mount
99
-	 *
100
-	 * @return string
101
-	 * @since 8.0.0
102
-	 */
103
-	public function getHome();
104
-
105
-	/**
106
-	 * Get the name of the backend class the user is connected with
107
-	 *
108
-	 * @return string
109
-	 * @since 8.0.0
110
-	 */
111
-	public function getBackendClassName();
112
-
113
-	/**
114
-	 * Get the backend for the current user object
115
-	 *
116
-	 * @return UserInterface
117
-	 * @since 15.0.0
118
-	 */
119
-	public function getBackend();
120
-
121
-	/**
122
-	 * check if the backend allows the user to change his avatar on Personal page
123
-	 *
124
-	 * @return bool
125
-	 * @since 8.0.0
126
-	 */
127
-	public function canChangeAvatar();
128
-
129
-	/**
130
-	 * check if the backend supports changing passwords
131
-	 *
132
-	 * @return bool
133
-	 * @since 8.0.0
134
-	 */
135
-	public function canChangePassword();
136
-
137
-	/**
138
-	 * check if the backend supports changing display names
139
-	 *
140
-	 * @return bool
141
-	 * @since 8.0.0
142
-	 */
143
-	public function canChangeDisplayName();
144
-
145
-	/**
146
-	 * check if the user is enabled
147
-	 *
148
-	 * @return bool
149
-	 * @since 8.0.0
150
-	 */
151
-	public function isEnabled();
152
-
153
-	/**
154
-	 * set the enabled status for the user
155
-	 *
156
-	 * @param bool $enabled
157
-	 * @since 8.0.0
158
-	 */
159
-	public function setEnabled(bool $enabled = true);
160
-
161
-	/**
162
-	 * get the users email address
163
-	 *
164
-	 * @return string|null
165
-	 * @since 9.0.0
166
-	 */
167
-	public function getEMailAddress();
168
-
169
-	/**
170
-	 * get the avatar image if it exists
171
-	 *
172
-	 * @param int $size
173
-	 * @return IImage|null
174
-	 * @since 9.0.0
175
-	 */
176
-	public function getAvatarImage($size);
177
-
178
-	/**
179
-	 * get the federation cloud id
180
-	 *
181
-	 * @return string
182
-	 * @since 9.0.0
183
-	 */
184
-	public function getCloudId();
185
-
186
-	/**
187
-	 * set the email address of the user
188
-	 *
189
-	 * @param string|null $mailAddress
190
-	 * @return void
191
-	 * @since 9.0.0
192
-	 */
193
-	public function setEMailAddress($mailAddress);
194
-
195
-	/**
196
-	 * get the users' quota in human readable form. If a specific quota is not
197
-	 * set for the user, the default value is returned. If a default setting
198
-	 * was not set otherwise, it is return as 'none', i.e. quota is not limited.
199
-	 *
200
-	 * @return string
201
-	 * @since 9.0.0
202
-	 */
203
-	public function getQuota();
204
-
205
-	/**
206
-	 * set the users' quota
207
-	 *
208
-	 * @param string $quota
209
-	 * @return void
210
-	 * @since 9.0.0
211
-	 */
212
-	public function setQuota($quota);
39
+    /**
40
+     * get the user id
41
+     *
42
+     * @return string
43
+     * @since 8.0.0
44
+     */
45
+    public function getUID();
46
+
47
+    /**
48
+     * get the display name for the user, if no specific display name is set it will fallback to the user id
49
+     *
50
+     * @return string
51
+     * @since 8.0.0
52
+     */
53
+    public function getDisplayName();
54
+
55
+    /**
56
+     * set the display name for the user
57
+     *
58
+     * @param string $displayName
59
+     * @return bool
60
+     * @since 8.0.0
61
+     */
62
+    public function setDisplayName($displayName);
63
+
64
+    /**
65
+     * returns the timestamp of the user's last login or 0 if the user did never
66
+     * login
67
+     *
68
+     * @return int
69
+     * @since 8.0.0
70
+     */
71
+    public function getLastLogin();
72
+
73
+    /**
74
+     * updates the timestamp of the most recent login of this user
75
+     * @since 8.0.0
76
+     */
77
+    public function updateLastLoginTimestamp();
78
+
79
+    /**
80
+     * Delete the user
81
+     *
82
+     * @return bool
83
+     * @since 8.0.0
84
+     */
85
+    public function delete();
86
+
87
+    /**
88
+     * Set the password of the user
89
+     *
90
+     * @param string $password
91
+     * @param string $recoveryPassword for the encryption app to reset encryption keys
92
+     * @return bool
93
+     * @since 8.0.0
94
+     */
95
+    public function setPassword($password, $recoveryPassword = null);
96
+
97
+    /**
98
+     * get the users home folder to mount
99
+     *
100
+     * @return string
101
+     * @since 8.0.0
102
+     */
103
+    public function getHome();
104
+
105
+    /**
106
+     * Get the name of the backend class the user is connected with
107
+     *
108
+     * @return string
109
+     * @since 8.0.0
110
+     */
111
+    public function getBackendClassName();
112
+
113
+    /**
114
+     * Get the backend for the current user object
115
+     *
116
+     * @return UserInterface
117
+     * @since 15.0.0
118
+     */
119
+    public function getBackend();
120
+
121
+    /**
122
+     * check if the backend allows the user to change his avatar on Personal page
123
+     *
124
+     * @return bool
125
+     * @since 8.0.0
126
+     */
127
+    public function canChangeAvatar();
128
+
129
+    /**
130
+     * check if the backend supports changing passwords
131
+     *
132
+     * @return bool
133
+     * @since 8.0.0
134
+     */
135
+    public function canChangePassword();
136
+
137
+    /**
138
+     * check if the backend supports changing display names
139
+     *
140
+     * @return bool
141
+     * @since 8.0.0
142
+     */
143
+    public function canChangeDisplayName();
144
+
145
+    /**
146
+     * check if the user is enabled
147
+     *
148
+     * @return bool
149
+     * @since 8.0.0
150
+     */
151
+    public function isEnabled();
152
+
153
+    /**
154
+     * set the enabled status for the user
155
+     *
156
+     * @param bool $enabled
157
+     * @since 8.0.0
158
+     */
159
+    public function setEnabled(bool $enabled = true);
160
+
161
+    /**
162
+     * get the users email address
163
+     *
164
+     * @return string|null
165
+     * @since 9.0.0
166
+     */
167
+    public function getEMailAddress();
168
+
169
+    /**
170
+     * get the avatar image if it exists
171
+     *
172
+     * @param int $size
173
+     * @return IImage|null
174
+     * @since 9.0.0
175
+     */
176
+    public function getAvatarImage($size);
177
+
178
+    /**
179
+     * get the federation cloud id
180
+     *
181
+     * @return string
182
+     * @since 9.0.0
183
+     */
184
+    public function getCloudId();
185
+
186
+    /**
187
+     * set the email address of the user
188
+     *
189
+     * @param string|null $mailAddress
190
+     * @return void
191
+     * @since 9.0.0
192
+     */
193
+    public function setEMailAddress($mailAddress);
194
+
195
+    /**
196
+     * get the users' quota in human readable form. If a specific quota is not
197
+     * set for the user, the default value is returned. If a default setting
198
+     * was not set otherwise, it is return as 'none', i.e. quota is not limited.
199
+     *
200
+     * @return string
201
+     * @since 9.0.0
202
+     */
203
+    public function getQuota();
204
+
205
+    /**
206
+     * set the users' quota
207
+     *
208
+     * @param string $quota
209
+     * @return void
210
+     * @since 9.0.0
211
+     */
212
+    public function setQuota($quota);
213 213
 }
Please login to merge, or discard this patch.
apps/dav/lib/Migration/Version1008Date20181105112049.php 1 patch
Indentation   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -32,18 +32,18 @@
 block discarded – undo
32 32
 
33 33
 class Version1008Date20181105112049 extends SimpleMigrationStep {
34 34
 
35
-	/**
36
-	 * @param IOutput $output
37
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
38
-	 * @param array $options
39
-	 * @return null|ISchemaWrapper
40
-	 */
41
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
42
-		/** @var ISchemaWrapper $schema */
43
-		$schema = $schemaClosure();
44
-		$table = $schema->getTable('calendarsubscriptions');
45
-		$table->dropColumn('source_copy');
35
+    /**
36
+     * @param IOutput $output
37
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
38
+     * @param array $options
39
+     * @return null|ISchemaWrapper
40
+     */
41
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
42
+        /** @var ISchemaWrapper $schema */
43
+        $schema = $schemaClosure();
44
+        $table = $schema->getTable('calendarsubscriptions');
45
+        $table->dropColumn('source_copy');
46 46
 
47
-		return $schema;
48
-	}
47
+        return $schema;
48
+    }
49 49
 }
Please login to merge, or discard this patch.
apps/dav/lib/Migration/Version1008Date20181105104833.php 1 patch
Indentation   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -32,18 +32,18 @@
 block discarded – undo
32 32
 
33 33
 class Version1008Date20181105104833 extends SimpleMigrationStep {
34 34
 
35
-	/**
36
-	 * @param IOutput $output
37
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
38
-	 * @param array $options
39
-	 * @return null|ISchemaWrapper
40
-	 */
41
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
42
-		/** @var ISchemaWrapper $schema */
43
-		$schema = $schemaClosure();
44
-		$table = $schema->getTable('calendarsubscriptions');
45
-		$table->dropColumn('source');
35
+    /**
36
+     * @param IOutput $output
37
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
38
+     * @param array $options
39
+     * @return null|ISchemaWrapper
40
+     */
41
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
42
+        /** @var ISchemaWrapper $schema */
43
+        $schema = $schemaClosure();
44
+        $table = $schema->getTable('calendarsubscriptions');
45
+        $table->dropColumn('source');
46 46
 
47
-		return $schema;
48
-	}
47
+        return $schema;
48
+    }
49 49
 }
Please login to merge, or discard this patch.
lib/public/Security/ICrypto.php 1 patch
Indentation   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -37,31 +37,31 @@
 block discarded – undo
37 37
  */
38 38
 interface ICrypto {
39 39
 
40
-	/**
41
-	 * @param string $message The message to authenticate
42
-	 * @param string $password Password to use (defaults to `secret` in config.php)
43
-	 * @return string Calculated HMAC
44
-	 * @since 8.0.0
45
-	 */
46
-	public function calculateHMAC(string $message, string $password = ''): string;
40
+    /**
41
+     * @param string $message The message to authenticate
42
+     * @param string $password Password to use (defaults to `secret` in config.php)
43
+     * @return string Calculated HMAC
44
+     * @since 8.0.0
45
+     */
46
+    public function calculateHMAC(string $message, string $password = ''): string;
47 47
 
48
-	/**
49
-	 * Encrypts a value and adds an HMAC (Encrypt-Then-MAC)
50
-	 * @param string $plaintext
51
-	 * @param string $password Password to encrypt, if not specified the secret from config.php will be taken
52
-	 * @return string Authenticated ciphertext
53
-	 * @since 8.0.0
54
-	 */
55
-	public function encrypt(string $plaintext, string $password = ''): string;
48
+    /**
49
+     * Encrypts a value and adds an HMAC (Encrypt-Then-MAC)
50
+     * @param string $plaintext
51
+     * @param string $password Password to encrypt, if not specified the secret from config.php will be taken
52
+     * @return string Authenticated ciphertext
53
+     * @since 8.0.0
54
+     */
55
+    public function encrypt(string $plaintext, string $password = ''): string;
56 56
 
57
-	/**
58
-	 * Decrypts a value and verifies the HMAC (Encrypt-Then-Mac)
59
-	 * @param string $authenticatedCiphertext
60
-	 * @param string $password Password to encrypt, if not specified the secret from config.php will be taken
61
-	 * @return string plaintext
62
-	 * @throws \Exception If the HMAC does not match
63
-	 * @throws \Exception If the decryption failed
64
-	 * @since 8.0.0
65
-	 */
66
-	public function decrypt(string $authenticatedCiphertext, string $password = ''): string;
57
+    /**
58
+     * Decrypts a value and verifies the HMAC (Encrypt-Then-Mac)
59
+     * @param string $authenticatedCiphertext
60
+     * @param string $password Password to encrypt, if not specified the secret from config.php will be taken
61
+     * @return string plaintext
62
+     * @throws \Exception If the HMAC does not match
63
+     * @throws \Exception If the decryption failed
64
+     * @since 8.0.0
65
+     */
66
+    public function decrypt(string $authenticatedCiphertext, string $password = ''): string;
67 67
 }
Please login to merge, or discard this patch.
apps/dav/lib/Migration/Version1008Date20181114084440.php 1 patch
Indentation   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -33,27 +33,27 @@
 block discarded – undo
33 33
 class Version1008Date20181114084440 extends SimpleMigrationStep {
34 34
 
35 35
 
36
-	/**
37
-	 * @param IOutput $output
38
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
39
-	 * @param array $options
40
-	 * @return null|ISchemaWrapper
41
-	 */
42
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
43
-		/** @var ISchemaWrapper $schema */
44
-		$schema = $schemaClosure();
45
-
46
-		if ($schema->hasTable('calendarchanges')) {
47
-			$calendarChangesTable = $schema->getTable('calendarchanges');
48
-			if ($calendarChangesTable->hasIndex('calendarid_calendartype_synctoken')) {
49
-				$calendarChangesTable->dropIndex('calendarid_calendartype_synctoken');
50
-			}
51
-
52
-			if (!$calendarChangesTable->hasIndex('calid_type_synctoken')) {
53
-				$calendarChangesTable->addIndex(['calendarid', 'calendartype', 'synctoken'], 'calid_type_synctoken');
54
-			}
55
-		}
56
-
57
-		return $schema;
58
-	}
36
+    /**
37
+     * @param IOutput $output
38
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
39
+     * @param array $options
40
+     * @return null|ISchemaWrapper
41
+     */
42
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
43
+        /** @var ISchemaWrapper $schema */
44
+        $schema = $schemaClosure();
45
+
46
+        if ($schema->hasTable('calendarchanges')) {
47
+            $calendarChangesTable = $schema->getTable('calendarchanges');
48
+            if ($calendarChangesTable->hasIndex('calendarid_calendartype_synctoken')) {
49
+                $calendarChangesTable->dropIndex('calendarid_calendartype_synctoken');
50
+            }
51
+
52
+            if (!$calendarChangesTable->hasIndex('calid_type_synctoken')) {
53
+                $calendarChangesTable->addIndex(['calendarid', 'calendartype', 'synctoken'], 'calid_type_synctoken');
54
+            }
55
+        }
56
+
57
+        return $schema;
58
+    }
59 59
 }
Please login to merge, or discard this patch.