Failed Conditions
Push — master ( e8410d...a45585 )
by Marcos
09:36 queued 11s
created
controller/credentialcontroller.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -163,13 +163,13 @@
 block discarded – undo
163 163
 				$activity . '_self', array($label, $this->userId, $revision_created),
164 164
 				'', array(),
165 165
 				$link, $this->userId, Activity::TYPE_ITEM_ACTION);
166
-		} else if (($storedCredential->getDeleteTime() === 0) && (int)$delete_time > 0) {
166
+		} else if (($storedCredential->getDeleteTime() === 0) && (int) $delete_time > 0) {
167 167
 			$activity = 'item_deleted';
168 168
 			$this->activityService->add(
169 169
 				$activity . '_self', array($label, $this->userId),
170 170
 				'', array(),
171 171
 				$link, $this->userId, Activity::TYPE_ITEM_ACTION);
172
-		} else if (($storedCredential->getDeleteTime() > 0) && (int)$delete_time === 0) {
172
+		} else if (($storedCredential->getDeleteTime() > 0) && (int) $delete_time === 0) {
173 173
 			$activity = 'item_recovered';
174 174
 			$this->activityService->add(
175 175
 				$activity . '_self', array($label, $this->userId),
Please login to merge, or discard this patch.
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -36,13 +36,13 @@  discard block
 block discarded – undo
36 36
 	private $settings;
37 37
 
38 38
 	public function __construct($AppName,
39
-	                            IRequest $request,
40
-	                            $userId,
41
-	                            CredentialService $credentialService,
42
-	                            ActivityService $activityService,
43
-	                            CredentialRevisionService $credentialRevisionService,
44
-	                            ShareService $sharingService,
45
-	                            SettingsService $settings
39
+								IRequest $request,
40
+								$userId,
41
+								CredentialService $credentialService,
42
+								ActivityService $activityService,
43
+								CredentialRevisionService $credentialRevisionService,
44
+								ShareService $sharingService,
45
+								SettingsService $settings
46 46
 
47 47
 	) {
48 48
 		parent::__construct(
@@ -65,10 +65,10 @@  discard block
 block discarded – undo
65 65
 	 * @NoCSRFRequired
66 66
 	 */
67 67
 	public function createCredential($changed, $created,
68
-	                                 $credential_id, $custom_fields, $delete_time,
69
-	                                 $description, $email, $expire_time, $favicon, $files, $guid,
70
-	                                 $hidden, $icon, $label, $otp, $password, $renew_interval,
71
-	                                 $tags, $url, $username, $vault_id, $compromised) {
68
+									 $credential_id, $custom_fields, $delete_time,
69
+									 $description, $email, $expire_time, $favicon, $files, $guid,
70
+									 $hidden, $icon, $label, $otp, $password, $renew_interval,
71
+									 $tags, $url, $username, $vault_id, $compromised) {
72 72
 		$credential = array(
73 73
 			'credential_id' => $credential_id,
74 74
 			'guid' => $guid,
@@ -121,10 +121,10 @@  discard block
 block discarded – undo
121 121
 	 * @NoCSRFRequired
122 122
 	 */
123 123
 	public function updateCredential($changed, $created,
124
-	                                 $credential_id, $custom_fields, $delete_time, $credential_guid,
125
-	                                 $description, $email, $expire_time, $icon, $files, $guid,
126
-	                                 $hidden, $label, $otp, $password, $renew_interval,
127
-	                                 $tags, $url, $username, $vault_id, $revision_created, $shared_key, $acl, $unshare_action, $set_share_key, $skip_revision, $compromised) {
124
+									 $credential_id, $custom_fields, $delete_time, $credential_guid,
125
+									 $description, $email, $expire_time, $icon, $files, $guid,
126
+									 $hidden, $label, $otp, $password, $renew_interval,
127
+									 $tags, $url, $username, $vault_id, $revision_created, $shared_key, $acl, $unshare_action, $set_share_key, $skip_revision, $compromised) {
128 128
 
129 129
 
130 130
 		$storedCredential = $this->credentialService->getCredentialByGUID($credential_guid);
Please login to merge, or discard this patch.
controller/internalcontroller.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -27,12 +27,12 @@
 block discarded – undo
27 27
 	private $appManager;
28 28
 
29 29
 	public function __construct($AppName,
30
-	                            IRequest $request,
31
-	                            $UserId,
32
-	                            CredentialService $credentialService,
33
-	                            IConfig $config,
34
-	                            IManager $IManager,
35
-	                            IAppManager $appManager
30
+								IRequest $request,
31
+								$UserId,
32
+								CredentialService $credentialService,
33
+								IConfig $config,
34
+								IManager $IManager,
35
+								IAppManager $appManager
36 36
 	) {
37 37
 		parent::__construct(
38 38
 			$AppName,
Please login to merge, or discard this patch.
migration/serversideencryption.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@
 block discarded – undo
60 60
 	private $fileService;
61 61
 
62 62
 	public function __construct(EncryptService $encryptService, IDBConnection $db, LoggerInterface $logger, CredentialService $credentialService, CredentialRevisionService $revisionService,
63
-	                            FileService $fileService, IConfig $config) {
63
+								FileService $fileService, IConfig $config) {
64 64
 		$this->encryptService = $encryptService;
65 65
 		$this->db = $db;
66 66
 		$this->logger = $logger;
Please login to merge, or discard this patch.
lib/Settings/Admin.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@
 block discarded – undo
80 80
 				}
81 81
 
82 82
 				if ($data !== false) {
83
-					$version = (string)$data->version;
83
+					$version = (string) $data->version;
84 84
 				} else {
85 85
 					libxml_clear_errors();
86 86
 				}
Please login to merge, or discard this patch.
lib/Activity.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -267,7 +267,7 @@
 block discarded – undo
267 267
 			'apps' => array(self::FILTER_PASSMAN =>
268 268
 				array(
269 269
 					'id' => 'passman',
270
-					'name' => (string)$l->t('Passwords'),
270
+					'name' => (string) $l->t('Passwords'),
271 271
 					'url' => $this->URLGenerator->linkToRoute('activity.Activities.showList', ['filter' => self::FILTER_PASSMAN]),
272 272
 				),
273 273
 			),
Please login to merge, or discard this patch.
lib/Notifier.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 			// Deal with known subjects
53 53
 			case 'credential_expired':
54 54
 				$notification->setParsedSubject(
55
-					(string)$l->t('Your credential "%s" expired, click here to update the credential.', $notification->getSubjectParameters())
55
+					(string) $l->t('Your credential "%s" expired, click here to update the credential.', $notification->getSubjectParameters())
56 56
 				);
57 57
 
58 58
 				// Deal with the actions for a known subject
@@ -60,13 +60,13 @@  discard block
 block discarded – undo
60 60
 					switch ($action->getLabel()) {
61 61
 						case 'remind':
62 62
 							$action->setParsedLabel(
63
-								(string)$l->t('Remind me later')
63
+								(string) $l->t('Remind me later')
64 64
 							);
65 65
 							break;
66 66
 
67 67
 						case 'ignore':
68 68
 							$action->setParsedLabel(
69
-								(string)$l->t('Ignore')
69
+								(string) $l->t('Ignore')
70 70
 							);
71 71
 							break;
72 72
 					}
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 
79 79
 			case 'credential_shared':
80 80
 				$notification->setParsedSubject(
81
-					(string)$l->t('%s shared "%s" with you. Click here to accept', $notification->getSubjectParameters())
81
+					(string) $l->t('%s shared "%s" with you. Click here to accept', $notification->getSubjectParameters())
82 82
 				);
83 83
 
84 84
 				// Deal with the actions for a known subject
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 					switch ($action->getLabel()) {
87 87
 						case 'decline':
88 88
 							$action->setParsedLabel(
89
-								(string)$l->t('Decline')
89
+								(string) $l->t('Decline')
90 90
 							);
91 91
 							break;
92 92
 					}
@@ -97,13 +97,13 @@  discard block
 block discarded – undo
97 97
 
98 98
 			case 'credential_share_denied':
99 99
 				$notification->setParsedSubject(
100
-					(string)$l->t('%s has declined your share request for "%s".', $notification->getSubjectParameters())
100
+					(string) $l->t('%s has declined your share request for "%s".', $notification->getSubjectParameters())
101 101
 				);
102 102
 				return $notification;
103 103
 
104 104
 			case 'credential_share_accepted':
105 105
 				$notification->setParsedSubject(
106
-					(string)$l->t('%s has accepted your share request for "%s".', $notification->getSubjectParameters())
106
+					(string) $l->t('%s has accepted your share request for "%s".', $notification->getSubjectParameters())
107 107
 				);
108 108
 				return $notification;
109 109
 			default:
Please login to merge, or discard this patch.
lib/AppInfo/Application.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -74,18 +74,18 @@  discard block
 block discarded – undo
74 74
 
75 75
 		$context->registerSearchProvider(Provider::class);
76 76
 
77
-		$context->registerService(View::class, function () {
77
+		$context->registerService(View::class, function() {
78 78
 			return new View('');
79 79
 		}, false);
80 80
 
81
-		$context->registerService('isCLI', function () {
81
+		$context->registerService('isCLI', function() {
82 82
 			return \OC::$CLI;
83 83
 		});
84 84
 
85 85
 		$context->registerMiddleware(ShareMiddleware::class);
86 86
 		$context->registerMiddleware(APIMiddleware::class);
87 87
 
88
-		$context->registerService('ShareController', function (ContainerInterface $c) {
88
+		$context->registerService('ShareController', function(ContainerInterface $c) {
89 89
 			/** @var IUserManager $userManager */
90 90
 			$userManager = $c->get(IUserManager::class);
91 91
 			/** @var IGroupManager $groupManager */
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 		});
112 112
 
113 113
 
114
-		$context->registerService('CronService', function (ContainerInterface $c) {
114
+		$context->registerService('CronService', function(ContainerInterface $c) {
115 115
 			return new CronService(
116 116
 				$c->get(CredentialService::class),
117 117
 				$c->get(LoggerInterface::class),
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
 			);
123 123
 		});
124 124
 
125
-		$context->registerService('Logger', function (ContainerInterface $c) {
125
+		$context->registerService('Logger', function(ContainerInterface $c) {
126 126
 			return $c->get(ServerContainer::class)->getLogger();
127 127
 		});
128 128
 	}
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 		/** @var INavigationManager $navigationManager */
144 144
 		$navigationManager = $c->get(INavigationManager::class);
145 145
 
146
-		$navigationEntry = function () use ($c) {
146
+		$navigationEntry = function() use ($c) {
147 147
 			/** @var IURLGenerator $urlGenerator */
148 148
 			$urlGenerator = $c->get(IURLGenerator::class);
149 149
 			return [
Please login to merge, or discard this patch.
lib/Utility/Utils.php 2 patches
Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -26,10 +26,10 @@  discard block
 block discarded – undo
26 26
 use OCP\IUserManager;
27 27
 
28 28
 class Utils {
29
-    /**
30
-     * Gets the unix epoch UTC timestamp
31
-     * @return int
32
-     */
29
+	/**
30
+	 * Gets the unix epoch UTC timestamp
31
+	 * @return int
32
+	 */
33 33
 	public static function getTime() {
34 34
 		return (new \DateTime())->getTimestamp();
35 35
 	}
@@ -40,10 +40,10 @@  discard block
 block discarded – undo
40 40
 		return microtime(true);
41 41
 	}
42 42
 
43
-    /**
44
-     * Generates a Globally Unique ID
45
-     * @return string
46
-     */
43
+	/**
44
+	 * Generates a Globally Unique ID
45
+	 * @return string
46
+	 */
47 47
 	public static function GUID() {
48 48
 		if (function_exists('com_create_guid') === true)
49 49
 		{
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 	 * @param IUserManager $userManager
59 59
 	 * @return string
60 60
 	 */
61
-	public static function getNameByUid(string $uid, IUserManager $userManager){
61
+	public static function getNameByUid(string $uid, IUserManager $userManager) {
62 62
 		$u = $userManager->get($uid);
63 63
 		return $u->getDisplayName();
64 64
 	}
@@ -68,14 +68,14 @@  discard block
 block discarded – undo
68 68
 	 * @param array $results
69 69
 	 * @return array|mixed
70 70
 	 */
71
-	public static function getDirContents(string $dir, &$results = array()){
71
+	public static function getDirContents(string $dir, &$results = array()) {
72 72
 		$files = scandir($dir);
73 73
 
74
-		foreach($files as $value){
75
-			$path = realpath($dir.DIRECTORY_SEPARATOR.$value);
76
-			if(!is_dir($path)) {
74
+		foreach ($files as $value) {
75
+			$path = realpath($dir . DIRECTORY_SEPARATOR . $value);
76
+			if (!is_dir($path)) {
77 77
 				$results[] = $path;
78
-			} else if($value != "." && $value != "..") {
78
+			} else if ($value != "." && $value != "..") {
79 79
 				Utils::getDirContents($path, $results);
80 80
 				$results[] = $path;
81 81
 			}
Please login to merge, or discard this patch.
controller/sharecontroller.php 1 patch
Indentation   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -51,18 +51,18 @@
 block discarded – undo
51 51
 	private $offset = 0;
52 52
 
53 53
 	public function __construct($AppName,
54
-	                            IRequest $request,
55
-	                            $UserId,
56
-	                            IGroupManager $groupManager,
57
-	                            IUserManager $userManager,
58
-	                            ActivityService $activityService,
59
-	                            VaultService $vaultService,
60
-	                            ShareService $shareService,
61
-	                            CredentialService $credentialService,
62
-	                            NotificationService $notificationService,
63
-	                            FileService $fileService,
64
-	                            SettingsService $config,
65
-	                            IManager $IManager
54
+								IRequest $request,
55
+								$UserId,
56
+								IGroupManager $groupManager,
57
+								IUserManager $userManager,
58
+								ActivityService $activityService,
59
+								VaultService $vaultService,
60
+								ShareService $shareService,
61
+								CredentialService $credentialService,
62
+								NotificationService $notificationService,
63
+								FileService $fileService,
64
+								SettingsService $config,
65
+								IManager $IManager
66 66
 	) {
67 67
 		parent::__construct(
68 68
 			$AppName,
Please login to merge, or discard this patch.