Passed
Push — master ( ae19cb...d3647d )
by Roeland
15:17 queued 10s
created
apps/settings/templates/settings/admin/sharing.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -27,17 +27,17 @@  discard block
 block discarded – undo
27 27
 ?>
28 28
 
29 29
 <div class="section" id="shareAPI">
30
-	<h2><?php p($l->t('Sharing'));?></h2>
30
+	<h2><?php p($l->t('Sharing')); ?></h2>
31 31
 	<a target="_blank" rel="noreferrer noopener" class="icon-info"
32
-	   title="<?php p($l->t('Open documentation'));?>"
32
+	   title="<?php p($l->t('Open documentation')); ?>"
33 33
 	   href="<?php p(link_to_docs('admin-sharing')); ?>"></a>
34
-        <p class="settings-hint"><?php p($l->t('As admin you can fine-tune the sharing behavior. Please see the documentation for more information.'));?></p>
34
+        <p class="settings-hint"><?php p($l->t('As admin you can fine-tune the sharing behavior. Please see the documentation for more information.')); ?></p>
35 35
 	<p id="enable">
36 36
 		<input type="checkbox" name="shareapi_enabled" id="shareAPIEnabled" class="checkbox"
37 37
 			   value="1" <?php if ($_['shareAPIEnabled'] === 'yes') {
38 38
 	print_unescaped('checked="checked"');
39 39
 } ?> />
40
-		<label for="shareAPIEnabled"><?php p($l->t('Allow apps to use the Share API'));?></label><br/>
40
+		<label for="shareAPIEnabled"><?php p($l->t('Allow apps to use the Share API')); ?></label><br/>
41 41
 	</p>
42 42
 
43 43
 	<p id="internalShareSettings" class="indent <?php if ($_['shareAPIEnabled'] === 'no') {
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 			   value="1" <?php if ($_['shareDefaultInternalExpireDateSet'] === 'yes') {
48 48
 	print_unescaped('checked="checked"');
49 49
 } ?> />
50
-		<label for="shareapiDefaultInternalExpireDate"><?php p($l->t('Set default expiration date for shares'));?></label><br/>
50
+		<label for="shareapiDefaultInternalExpireDate"><?php p($l->t('Set default expiration date for shares')); ?></label><br/>
51 51
 	</p>
52 52
 	<p id="setDefaultInternalExpireDate" class="double-indent <?php if ($_['shareDefaultInternalExpireDateSet'] === 'no' || $_['shareAPIEnabled'] === 'no') {
53 53
 	p('hidden');
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 			   value="1" <?php if ($_['shareInternalEnforceExpireDate'] === 'yes') {
61 61
 	print_unescaped('checked="checked"');
62 62
 } ?> />
63
-		<label for="shareapiInternalEnforceExpireDate"><?php p($l->t('Enforce expiration date'));?></label><br/>
63
+		<label for="shareapiInternalEnforceExpireDate"><?php p($l->t('Enforce expiration date')); ?></label><br/>
64 64
 	</p>
65 65
 
66 66
 	<p class="<?php if ($_['shareAPIEnabled'] === 'no') {
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 			   value="1" <?php if ($_['allowLinks'] === 'yes') {
71 71
 	print_unescaped('checked="checked"');
72 72
 } ?> />
73
-		<label for="allowLinks"><?php p($l->t('Allow users to share via link and emails'));?></label><br/>
73
+		<label for="allowLinks"><?php p($l->t('Allow users to share via link and emails')); ?></label><br/>
74 74
 	</p>
75 75
 
76 76
 	<p id="publicLinkSettings" class="indent <?php if ($_['allowLinks'] !== 'yes' || $_['shareAPIEnabled'] === 'no') {
@@ -80,23 +80,23 @@  discard block
 block discarded – undo
80 80
 			   value="1" <?php if ($_['allowPublicUpload'] == 'yes') {
81 81
 	print_unescaped('checked="checked"');
82 82
 } ?> />
83
-		<label for="allowPublicUpload"><?php p($l->t('Allow public uploads'));?></label><br/>
83
+		<label for="allowPublicUpload"><?php p($l->t('Allow public uploads')); ?></label><br/>
84 84
 		<input type="checkbox" name="shareapi_enable_link_password_by_default" id="enableLinkPasswordByDefault" class="checkbox"
85 85
 			   value="1" <?php if ($_['enableLinkPasswordByDefault'] === 'yes') {
86 86
 	print_unescaped('checked="checked"');
87 87
 } ?> />
88
-		<label for="enableLinkPasswordByDefault"><?php p($l->t('Always ask for a password'));?></label><br/>
88
+		<label for="enableLinkPasswordByDefault"><?php p($l->t('Always ask for a password')); ?></label><br/>
89 89
 		<input type="checkbox" name="shareapi_enforce_links_password" id="enforceLinkPassword" class="checkbox"
90 90
 			   value="1" <?php if ($_['enforceLinkPassword']) {
91 91
 	print_unescaped('checked="checked"');
92 92
 } ?> />
93
-		<label for="enforceLinkPassword"><?php p($l->t('Enforce password protection'));?></label><br/>
93
+		<label for="enforceLinkPassword"><?php p($l->t('Enforce password protection')); ?></label><br/>
94 94
 
95 95
 		<input type="checkbox" name="shareapi_default_expire_date" id="shareapiDefaultExpireDate" class="checkbox"
96 96
 			   value="1" <?php if ($_['shareDefaultExpireDateSet'] === 'yes') {
97 97
 	print_unescaped('checked="checked"');
98 98
 } ?> />
99
-		<label for="shareapiDefaultExpireDate"><?php p($l->t('Set default expiration date'));?></label><br/>
99
+		<label for="shareapiDefaultExpireDate"><?php p($l->t('Set default expiration date')); ?></label><br/>
100 100
 
101 101
 	</p>
102 102
 	<p id="setDefaultExpireDate" class="double-indent <?php if ($_['allowLinks'] !== 'yes' || $_['shareDefaultExpireDateSet'] === 'no' || $_['shareAPIEnabled'] === 'no') {
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 			   value="1" <?php if ($_['shareEnforceExpireDate'] === 'yes') {
111 111
 	print_unescaped('checked="checked"');
112 112
 } ?> />
113
-		<label for="shareapiEnforceExpireDate"><?php p($l->t('Enforce expiration date'));?></label><br/>
113
+		<label for="shareapiEnforceExpireDate"><?php p($l->t('Enforce expiration date')); ?></label><br/>
114 114
 	</p>
115 115
 	<p class="<?php if ($_['shareAPIEnabled'] === 'no') {
116 116
 	p('hidden');
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 			   value="1" <?php if ($_['allowResharing'] === 'yes') {
120 120
 	print_unescaped('checked="checked"');
121 121
 } ?> />
122
-		<label for="allowResharing"><?php p($l->t('Allow resharing'));?></label><br/>
122
+		<label for="allowResharing"><?php p($l->t('Allow resharing')); ?></label><br/>
123 123
 	</p>
124 124
 	<p class="<?php if ($_['shareAPIEnabled'] === 'no') {
125 125
 	p('hidden');
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 			   value="1" <?php if ($_['allowGroupSharing'] === 'yes') {
129 129
 	print_unescaped('checked="checked"');
130 130
 } ?> />
131
-		<label for="allowGroupSharing"><?php p($l->t('Allow sharing with groups'));?></label><br />
131
+		<label for="allowGroupSharing"><?php p($l->t('Allow sharing with groups')); ?></label><br />
132 132
 	</p>
133 133
 	<p class="<?php if ($_['shareAPIEnabled'] === 'no') {
134 134
 	p('hidden');
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 			   value="1" <?php if ($_['onlyShareWithGroupMembers']) {
138 138
 	print_unescaped('checked="checked"');
139 139
 } ?> />
140
-		<label for="onlyShareWithGroupMembers"><?php p($l->t('Restrict users to only share with users in their groups'));?></label><br/>
140
+		<label for="onlyShareWithGroupMembers"><?php p($l->t('Restrict users to only share with users in their groups')); ?></label><br/>
141 141
 	</p>
142 142
 	<p class="<?php if ($_['shareAPIEnabled'] === 'no') {
143 143
 	p('hidden');
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
 			   value="1" <?php if ($_['shareExcludeGroups']) {
147 147
 	print_unescaped('checked="checked"');
148 148
 } ?> />
149
-		<label for="shareapiExcludeGroups"><?php p($l->t('Exclude groups from sharing'));?></label><br/>
149
+		<label for="shareapiExcludeGroups"><?php p($l->t('Exclude groups from sharing')); ?></label><br/>
150 150
 	</p>
151 151
 	<p id="selectExcludedGroups" class="indent <?php if (!$_['shareExcludeGroups'] || $_['shareAPIEnabled'] === 'no') {
152 152
 	p('hidden');
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 			<?php if ($_['allowShareDialogUserEnumeration'] === 'yes') {
164 164
 	print_unescaped('checked="checked"');
165 165
 } ?> />
166
-		<label for="shareapi_allow_share_dialog_user_enumeration"><?php p($l->t('Allow username autocompletion in share dialog'));?></label><br />
166
+		<label for="shareapi_allow_share_dialog_user_enumeration"><?php p($l->t('Allow username autocompletion in share dialog')); ?></label><br />
167 167
 	</p>
168 168
 
169 169
 	<p id="shareapi_restrict_user_enumeration_to_group_setting" class="indent <?php if ($_['shareAPIEnabled'] === 'no' || $_['allowShareDialogUserEnumeration'] === 'no') {
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
 			<?php if ($_['restrictUserEnumerationToGroup'] === 'yes') {
174 174
 	print_unescaped('checked="checked"');
175 175
 } ?> />
176
-		<label for="shareapi_restrict_user_enumeration_to_group"><?php p($l->t('Allow username autocompletion to users within the same groups'));?></label><br />
176
+		<label for="shareapi_restrict_user_enumeration_to_group"><?php p($l->t('Allow username autocompletion to users within the same groups')); ?></label><br />
177 177
 	</p>
178 178
 
179 179
 	<p id="shareapi_restrict_user_enumeration_to_phone_setting" class="indent <?php if ($_['shareAPIEnabled'] === 'no' || $_['allowShareDialogUserEnumeration'] === 'no') {
@@ -183,12 +183,12 @@  discard block
 block discarded – undo
183 183
 			<?php if ($_['restrictUserEnumerationToPhone'] === 'yes') {
184 184
 	print_unescaped('checked="checked"');
185 185
 } ?> />
186
-		<label for="shareapi_restrict_user_enumeration_to_phone"><?php p($l->t('Allow username autocompletion to users based on phonebook matches'));?></label><br />
186
+		<label for="shareapi_restrict_user_enumeration_to_phone"><?php p($l->t('Allow username autocompletion to users based on phonebook matches')); ?></label><br />
187 187
 	</p>
188 188
 	<p id="shareapi_restrict_user_enumeration_combinewarning_setting" class="indent <?php if ($_['shareAPIEnabled'] === 'no' || $_['allowShareDialogUserEnumeration'] === 'no') {
189 189
 	p('hidden');
190 190
 }?>">
191
-		<em><?php p($l->t('If autocompletion "same group" and "phonebook matches" are enabled a match in either is enough to show the user.'));?></em><br />
191
+		<em><?php p($l->t('If autocompletion "same group" and "phonebook matches" are enabled a match in either is enough to show the user.')); ?></em><br />
192 192
 	</p>
193 193
 	<p id="shareapi_restrict_user_enumeration_full_match_setting" class="indent <?php if ($_['shareAPIEnabled'] === 'no') {
194 194
 	p('hidden');
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
 				<?php if ($_['restrictUserEnumerationFullMatch'] === 'yes') {
198 198
 	print_unescaped('checked="checked"');
199 199
 } ?> />
200
-		<label for="shareapi_restrict_user_enumeration_full_match"><?php p($l->t('Allow username autocompletion when entering the full name or email address (ignoring missing phonebook match and being in the same group)'));?></label><br />
200
+		<label for="shareapi_restrict_user_enumeration_full_match"><?php p($l->t('Allow username autocompletion when entering the full name or email address (ignoring missing phonebook match and being in the same group)')); ?></label><br />
201 201
 	</p>
202 202
 
203 203
 	<p>
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
 			<?php if ($_['publicShareDisclaimerText'] !== null) {
206 206
 	print_unescaped('checked="checked"');
207 207
 } ?> />
208
-		<label for="publicShareDisclaimer"><?php p($l->t('Show disclaimer text on the public link upload page (only shown when the file list is hidden)'));?></label>
208
+		<label for="publicShareDisclaimer"><?php p($l->t('Show disclaimer text on the public link upload page (only shown when the file list is hidden)')); ?></label>
209 209
 		<span id="publicShareDisclaimerStatus" class="msg" style="display:none"></span>
210 210
 		<br/>
211 211
 		<textarea placeholder="<?php p($l->t('This text will be shown on the public link upload page when the file list is hidden.')) ?>" id="publicShareDisclaimerText" <?php if ($_['publicShareDisclaimerText'] === null) {
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
 } ?>><?php p($_['publicShareDisclaimerText']) ?></textarea>
214 214
 	</p>
215 215
 
216
-	<h3><?php p($l->t('Default share permissions'));?></h3>
216
+	<h3><?php p($l->t('Default share permissions')); ?></h3>
217 217
 	<input type="hidden" name="shareapi_default_permissions" id="shareApiDefaultPermissions" class="checkbox"
218 218
 		   value="<?php p($_['shareApiDefaultPermissions']) ?>" />
219 219
 	<p id="shareApiDefaultPermissionsSection" class="indent <?php if ($_['shareAPIEnabled'] === 'no') {
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
224 224
 				   class="noautosave checkbox" value="<?php p($perm['value']) ?>" <?php if (($_['shareApiDefaultPermissions'] & $perm['value']) !== 0) {
225 225
 	print_unescaped('checked="checked"');
226 226
 } ?> />
227
-			<label for="shareapi_default_permission_<?php p($perm['id']) ?>"><?php p($perm['label']);?></label>
227
+			<label for="shareapi_default_permission_<?php p($perm['id']) ?>"><?php p($perm['label']); ?></label>
228 228
 		<?php endforeach ?>
229 229
 	</p>
230 230
 </div>
Please login to merge, or discard this patch.
apps/sharebymail/lib/ShareByMailProvider.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -221,7 +221,7 @@  discard block
 block discarded – undo
221 221
 
222 222
 		$password = $passwordEvent->getPassword();
223 223
 		if ($password === null) {
224
-			$password = $this->secureRandom->generate(8, ISecureRandom::CHAR_LOWER . ISecureRandom::CHAR_UPPER . ISecureRandom::CHAR_DIGITS);
224
+			$password = $this->secureRandom->generate(8, ISecureRandom::CHAR_LOWER.ISecureRandom::CHAR_UPPER.ISecureRandom::CHAR_DIGITS);
225 225
 		}
226 226
 
227 227
 		return $password;
@@ -393,7 +393,7 @@  discard block
 block discarded – undo
393 393
 		$text = $this->l->t('%1$s shared »%2$s« with you.', [$initiatorDisplayName, $filename]);
394 394
 
395 395
 		$emailTemplate->addBodyText(
396
-			htmlspecialchars($text . ' ' . $this->l->t('Click the button below to open it.')),
396
+			htmlspecialchars($text.' '.$this->l->t('Click the button below to open it.')),
397 397
 			$text
398 398
 		);
399 399
 		$emailTemplate->addBodyButton(
@@ -422,7 +422,7 @@  discard block
 block discarded – undo
422 422
 		$initiatorEmail = $initiatorUser->getEMailAddress();
423 423
 		if ($this->settingsManager->replyToInitiator() && $initiatorEmail !== null) {
424 424
 			$message->setReplyTo([$initiatorEmail => $initiatorDisplayName]);
425
-			$emailTemplate->addFooter($instanceName . ($this->defaults->getSlogan() !== '' ? ' - ' . $this->defaults->getSlogan() : ''));
425
+			$emailTemplate->addFooter($instanceName.($this->defaults->getSlogan() !== '' ? ' - '.$this->defaults->getSlogan() : ''));
426 426
 		} else {
427 427
 			$emailTemplate->addFooter();
428 428
 		}
@@ -486,7 +486,7 @@  discard block
 block discarded – undo
486 486
 		$message->setFrom([\OCP\Util::getDefaultEmailAddress($instanceName) => $senderName]);
487 487
 		if ($this->settingsManager->replyToInitiator() && $initiatorEmailAddress !== null) {
488 488
 			$message->setReplyTo([$initiatorEmailAddress => $initiatorDisplayName]);
489
-			$emailTemplate->addFooter($instanceName . ' - ' . $this->defaults->getSlogan());
489
+			$emailTemplate->addFooter($instanceName.' - '.$this->defaults->getSlogan());
490 490
 		} else {
491 491
 			$emailTemplate->addFooter();
492 492
 		}
@@ -546,7 +546,7 @@  discard block
 block discarded – undo
546 546
 		$message->setFrom([\OCP\Util::getDefaultEmailAddress($instanceName) => $senderName]);
547 547
 		if ($this->settingsManager->replyToInitiator() && $initiatorEmailAddress !== null) {
548 548
 			$message->setReplyTo([$initiatorEmailAddress => $initiatorDisplayName]);
549
-			$emailTemplate->addFooter($instanceName . ' - ' . $this->defaults->getSlogan());
549
+			$emailTemplate->addFooter($instanceName.' - '.$this->defaults->getSlogan());
550 550
 		} else {
551 551
 			$emailTemplate->addFooter();
552 552
 		}
@@ -683,7 +683,7 @@  discard block
 block discarded – undo
683 683
 			->setValue('password', $qb->createNamedParameter($password))
684 684
 			->setValue('password_by_talk', $qb->createNamedParameter($sendPasswordByTalk, IQueryBuilder::PARAM_BOOL))
685 685
 			->setValue('stime', $qb->createNamedParameter(time()))
686
-			->setValue('hide_download', $qb->createNamedParameter((int)$hideDownload, IQueryBuilder::PARAM_INT))
686
+			->setValue('hide_download', $qb->createNamedParameter((int) $hideDownload, IQueryBuilder::PARAM_INT))
687 687
 			->setValue('label', $qb->createNamedParameter($label));
688 688
 
689 689
 		if ($expirationTime !== null) {
@@ -731,7 +731,7 @@  discard block
 block discarded – undo
731 731
 			->set('password_by_talk', $qb->createNamedParameter($share->getSendPasswordByTalk(), IQueryBuilder::PARAM_BOOL))
732 732
 			->set('expiration', $qb->createNamedParameter($share->getExpirationDate(), IQueryBuilder::PARAM_DATE))
733 733
 			->set('note', $qb->createNamedParameter($share->getNote()))
734
-			->set('hide_download', $qb->createNamedParameter((int)$share->getHideDownload(), IQueryBuilder::PARAM_INT))
734
+			->set('hide_download', $qb->createNamedParameter((int) $share->getHideDownload(), IQueryBuilder::PARAM_INT))
735 735
 			->execute();
736 736
 
737 737
 		if ($originalShare->getNote() !== $share->getNote() && $share->getNote() !== '') {
@@ -977,22 +977,22 @@  discard block
 block discarded – undo
977 977
 	 */
978 978
 	protected function createShareObject($data) {
979 979
 		$share = new Share($this->rootFolder, $this->userManager);
980
-		$share->setId((int)$data['id'])
981
-			->setShareType((int)$data['share_type'])
982
-			->setPermissions((int)$data['permissions'])
980
+		$share->setId((int) $data['id'])
981
+			->setShareType((int) $data['share_type'])
982
+			->setPermissions((int) $data['permissions'])
983 983
 			->setTarget($data['file_target'])
984
-			->setMailSend((bool)$data['mail_send'])
984
+			->setMailSend((bool) $data['mail_send'])
985 985
 			->setNote($data['note'])
986 986
 			->setToken($data['token']);
987 987
 
988 988
 		$shareTime = new \DateTime();
989
-		$shareTime->setTimestamp((int)$data['stime']);
989
+		$shareTime->setTimestamp((int) $data['stime']);
990 990
 		$share->setShareTime($shareTime);
991 991
 		$share->setSharedWith($data['share_with']);
992 992
 		$share->setPassword($data['password']);
993 993
 		$share->setLabel($data['label']);
994
-		$share->setSendPasswordByTalk((bool)$data['password_by_talk']);
995
-		$share->setHideDownload((bool)$data['hide_download']);
994
+		$share->setSendPasswordByTalk((bool) $data['password_by_talk']);
995
+		$share->setHideDownload((bool) $data['hide_download']);
996 996
 
997 997
 		if ($data['uid_initiator'] !== null) {
998 998
 			$share->setShareOwner($data['uid_owner']);
@@ -1000,7 +1000,7 @@  discard block
 block discarded – undo
1000 1000
 		} else {
1001 1001
 			//OLD SHARE
1002 1002
 			$share->setSharedBy($data['uid_owner']);
1003
-			$path = $this->getNode($share->getSharedBy(), (int)$data['file_source']);
1003
+			$path = $this->getNode($share->getSharedBy(), (int) $data['file_source']);
1004 1004
 
1005 1005
 			$owner = $path->getOwner();
1006 1006
 			$share->setShareOwner($owner->getUID());
@@ -1013,7 +1013,7 @@  discard block
 block discarded – undo
1013 1013
 			}
1014 1014
 		}
1015 1015
 
1016
-		$share->setNodeId((int)$data['file_source']);
1016
+		$share->setNodeId((int) $data['file_source']);
1017 1017
 		$share->setNodeType($data['item_type']);
1018 1018
 
1019 1019
 		$share->setProviderId($this->identifier());
@@ -1130,7 +1130,7 @@  discard block
 block discarded – undo
1130 1130
 			);
1131 1131
 		}
1132 1132
 
1133
-		$qb->innerJoin('s', 'filecache' ,'f', $qb->expr()->eq('s.file_source', 'f.fileid'));
1133
+		$qb->innerJoin('s', 'filecache', 'f', $qb->expr()->eq('s.file_source', 'f.fileid'));
1134 1134
 		$qb->andWhere($qb->expr()->eq('f.parent', $qb->createNamedParameter($node->getId())));
1135 1135
 
1136 1136
 		$qb->orderBy('id');
Please login to merge, or discard this patch.