nextcloud / passman

controller/pagecontroller.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
 
 	private $userId;
 
-	public function __construct($AppName, IRequest $request, $UserId){
+	public function __construct($AppName, IRequest $request, $UserId) {
 		parent::__construct($AppName, $request);
 		$this->userId = $UserId;
 	}
@@ -38,7 +38,7 @@ 
 	 */
 	public function index() {
 		$params = ['user' => $this->userId];
-		return new TemplateResponse('passman', 'main', $params);  // templates/main.php
+		return new TemplateResponse('passman', 'main', $params); // templates/main.php
 	}
 
 
@@ -46,7 +46,7 @@ 
 	 * @NoAdminRequired
 	 * @NoCSRFRequired
 	 */
-	public function bookmarklet($url='',$title='') {
+	public function bookmarklet($url = '', $title = '') {
 		$params = array('url' => $url, 'title' => $title);
 		return new TemplateResponse('passman', 'bookmarklet', $params);
 	}

lib/Db/DeleteVaultRequest.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
  */
 
 
-class DeleteVaultRequest extends Entity implements  \JsonSerializable{
+class DeleteVaultRequest extends Entity implements  \JsonSerializable {
 
 	use EntityJSONSerializer;
 

lib/Service/IconService.php

Spacing +1 added lines, -1 removed lines

@@ -181,7 +181,7 @@
 	}
 
 
-	private function parseLinkElement($htmlHead, $pageUrlInfo, $base_href){
+	private function parseLinkElement($htmlHead, $pageUrlInfo, $base_href) {
 		if (preg_match('#<\s*link[^>]*(rel=(["\'])[^>\2]*icon[^>\2]*\2)[^>]*>#i', $htmlHead, $matches)) {
 			$link_tag = $matches[0];
 			$this->debugInfo['link_tag'] = $link_tag;

lib/Settings/Admin.php

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 				}
 
 				if ($data !== false) {
-					$version = (string)$data->version;
+					$version = (string) $data->version;
 				} else {
 					libxml_clear_errors();
 				}

lib/Activity.php

Spacing +1 added lines, -1 removed lines

@@ -267,7 +267,7 @@
 			'apps' => array(self::FILTER_PASSMAN =>
 				array(
 					'id' => 'passman',
-					'name' => (string)$l->t('Passwords'),
+					'name' => (string) $l->t('Passwords'),
 					'url' => $this->URLGenerator->linkToRoute('activity.Activities.showList', ['filter' => self::FILTER_PASSMAN]),
 				),
 			),

lib/Notifier.php

Spacing +7 added lines, -7 removed lines

@@ -52,7 +52,7 @@ 
 			// Deal with known subjects
 			case 'credential_expired':
 				$notification->setParsedSubject(
-					(string)$l->t('Your credential "%s" expired, click here to update the credential.', $notification->getSubjectParameters())
+					(string) $l->t('Your credential "%s" expired, click here to update the credential.', $notification->getSubjectParameters())
 				);
 
 				// Deal with the actions for a known subject
@@ -60,13 +60,13 @@ 
 					switch ($action->getLabel()) {
 						case 'remind':
 							$action->setParsedLabel(
-								(string)$l->t('Remind me later')
+								(string) $l->t('Remind me later')
 							);
 							break;
 
 						case 'ignore':
 							$action->setParsedLabel(
-								(string)$l->t('Ignore')
+								(string) $l->t('Ignore')
 							);
 							break;
 					}
@@ -78,7 +78,7 @@ 
 
 			case 'credential_shared':
 				$notification->setParsedSubject(
-					(string)$l->t('%s shared "%s" with you. Click here to accept', $notification->getSubjectParameters())
+					(string) $l->t('%s shared "%s" with you. Click here to accept', $notification->getSubjectParameters())
 				);
 
 				// Deal with the actions for a known subject
@@ -86,7 +86,7 @@ 
 					switch ($action->getLabel()) {
 						case 'decline':
 							$action->setParsedLabel(
-								(string)$l->t('Decline')
+								(string) $l->t('Decline')
 							);
 							break;
 					}
@@ -97,13 +97,13 @@ 
 
 			case 'credential_share_denied':
 				$notification->setParsedSubject(
-					(string)$l->t('%s has declined your share request for "%s".', $notification->getSubjectParameters())
+					(string) $l->t('%s has declined your share request for "%s".', $notification->getSubjectParameters())
 				);
 				return $notification;
 
 			case 'credential_share_accepted':
 				$notification->setParsedSubject(
-					(string)$l->t('%s has accepted your share request for "%s".', $notification->getSubjectParameters())
+					(string) $l->t('%s has accepted your share request for "%s".', $notification->getSubjectParameters())
 				);
 				return $notification;
 			default:

lib/AppInfo/Application.php

Spacing +6 added lines, -6 removed lines

@@ -74,18 +74,18 @@ 
 
 		$context->registerSearchProvider(Provider::class);
 
-		$context->registerService(View::class, function () {
+		$context->registerService(View::class, function() {
 			return new View('');
 		}, false);
 
-		$context->registerService('isCLI', function () {
+		$context->registerService('isCLI', function() {
 			return \OC::$CLI;
 		});
 
 		$context->registerMiddleware(ShareMiddleware::class);
 		$context->registerMiddleware(APIMiddleware::class);
 
-		$context->registerService('ShareController', function (ContainerInterface $c) {
+		$context->registerService('ShareController', function(ContainerInterface $c) {
 			/** @var IUserManager $userManager */
 			$userManager = $c->get(IUserManager::class);
 			/** @var IGroupManager $groupManager */
@@ -111,7 +111,7 @@ 
 		});
 
 
-		$context->registerService('CronService', function (ContainerInterface $c) {
+		$context->registerService('CronService', function(ContainerInterface $c) {
 			return new CronService(
 				$c->get(CredentialService::class),
 				$c->get(LoggerInterface::class),
@@ -122,7 +122,7 @@ 
 			);
 		});
 
-		$context->registerService('Logger', function (ContainerInterface $c) {
+		$context->registerService('Logger', function(ContainerInterface $c) {
 			return $c->get(ServerContainer::class)->getLogger();
 		});
 	}
@@ -143,7 +143,7 @@ 
 		/** @var INavigationManager $navigationManager */
 		$navigationManager = $c->get(INavigationManager::class);
 
-		$navigationEntry = function () use ($c) {
+		$navigationEntry = function() use ($c) {
 			/** @var IURLGenerator $urlGenerator */
 			$urlGenerator = $c->get(IURLGenerator::class);
 			return [

lib/Utility/Utils.php

Spacing +6 added lines, -6 removed lines

@@ -58,7 +58,7 @@ 
 	 * @param IUserManager $userManager
 	 * @return string
 	 */
-	public static function getNameByUid(string $uid, IUserManager $userManager){
+	public static function getNameByUid(string $uid, IUserManager $userManager) {
 		$u = $userManager->get($uid);
 		return $u->getDisplayName();
 	}
@@ -68,14 +68,14 @@ 
 	 * @param array $results
 	 * @return array|mixed
 	 */
-	public static function getDirContents(string $dir, &$results = array()){
+	public static function getDirContents(string $dir, &$results = array()) {
 		$files = scandir($dir);
 
-		foreach($files as $value){
-			$path = realpath($dir.DIRECTORY_SEPARATOR.$value);
-			if(!is_dir($path)) {
+		foreach ($files as $value) {
+			$path = realpath($dir . DIRECTORY_SEPARATOR . $value);
+			if (!is_dir($path)) {
 				$results[] = $path;
-			} else if($value != "." && $value != "..") {
+			} else if ($value != "." && $value != "..") {
 				Utils::getDirContents($path, $results);
 				$results[] = $path;
 			}

controller/admincontroller.php

Spacing +13 added lines, -13 removed lines

@@ -80,7 +80,7 @@ 
 
 	public function moveCredentials($source_account, $destination_account) {
 		$succeed = false;
-		if ($source_account != $destination_account){
+		if ($source_account != $destination_account) {
 			$vaults = $this->vaultService->getByUser($source_account);
 			foreach ($vaults as $vault) {
 				$credentials = $this->credentialService->getCredentialsByVaultId($vault->getId(), $source_account);
@@ -118,10 +118,10 @@ 
 		return new JSONResponse(array('success' => $succeed));
 	}
 
-	public function listRequests(){
+	public function listRequests() {
 		$requests = $this->deleteVaultRequestService->getDeleteRequests();
 		$results = array();
-		foreach($requests as $request){
+		foreach ($requests as $request) {
 			$r = $request->jsonSerialize();
 			$r['displayName'] = Utils::getNameByUid($request->getRequestedBy(), $this->userManager);
 			array_push($results, $r);
@@ -129,28 +129,28 @@ 
 		return new JSONResponse($results);
 	}
 
-	public function acceptRequestDeletion($vault_guid, $requested_by){
+	public function acceptRequestDeletion($vault_guid, $requested_by) {
 		$req = $this->deleteVaultRequestService->getDeleteRequestForVault($vault_guid);
-		try{
+		try {
 			$vault = $this->vaultService->getByGuid($vault_guid, $requested_by);
-		} catch (\Exception $e){
+		} catch (\Exception $e) {
 			//Ignore
 		}
 
-		if(isset($vault)){
+		if (isset($vault)) {
 			$credentials = $this->credentialService->getCredentialsByVaultId($vault->getId(), $requested_by);
-			foreach($credentials as $credential){
+			foreach ($credentials as $credential) {
 				$revisions = $this->revisionService->getRevisions($credential->getId());
-				foreach($revisions as $revision){
+				foreach ($revisions as $revision) {
 					$this->revisionService->deleteRevision($revision['revision_id'], $requested_by);
 				}
-				if($credential instanceof Credential){
+				if ($credential instanceof Credential) {
 					$this->credentialService->deleteCredential($credential);
 				}
 			}
 			$this->vaultService->deleteVault($vault_guid, $requested_by);
 		}
-		if($req instanceof DeleteVaultRequest) {
+		if ($req instanceof DeleteVaultRequest) {
 			$this->deleteVaultRequestService->removeDeleteRequestForVault($req);
 		}
 
@@ -162,7 +162,7 @@ 
 	 */
 	public function requestDeletion($vault_guid, $reason) {
 		$req = $this->deleteVaultRequestService->getDeleteRequestForVault($vault_guid);
-		if($req){
+		if ($req) {
 			return new JSONResponse('Already exists');
 		}
 		$vault = $this->vaultService->getByGuid($vault_guid, $this->userId);
@@ -187,7 +187,7 @@ 
 		$result = false;
 		try {
 			$delete_request = $this->deleteVaultRequestService->getDeleteRequestForVault($vault_guid);
-		} catch (\Exception $exception){
+		} catch (\Exception $exception) {
 			// Ignore it
 		}