nextcloud / passman

lib/Db/CredentialMapper.php

Spacing +14 added lines, -14 removed lines

@@ -57,7 +57,7 @@ 
 		$sql = 'SELECT * FROM `*PREFIX*passman_credentials` ' .
 			'WHERE `user_id` = ? and vault_id = ? AND shared_key is NULL LIMIT 20';
 		$entities = $this->findEntities($sql, [$user_id, $vault_id]);
-		$count = count($entities)-1;
+		$count = count($entities) - 1;
 		$entities = array_splice($entities, rand(0, $count), 1);
 		return $entities;
 	}
@@ -67,7 +67,7 @@ 
 	 * @param $timestamp
 	 * @return Credential[]
 	 */
-	public function getExpiredCredentials($timestamp){
+	public function getExpiredCredentials($timestamp) {
 		$sql = 'SELECT * FROM `*PREFIX*passman_credentials` ' .
 			'WHERE `expire_time` > 0 AND `expire_time` < ?';
 		return $this->findEntities($sql, [$timestamp]);
@@ -80,16 +80,16 @@ 
      * @param null $user_id
      * @return Credential
      */
-	public function getCredentialById($credential_id, $user_id = null){
+	public function getCredentialById($credential_id, $user_id = null) {
 		$sql = 'SELECT * FROM `*PREFIX*passman_credentials` ' .
 			'WHERE `id` = ?';
         // If we want to check the owner, add it to the query
 		$params = [$credential_id];
-        if ($user_id !== null){
+        if ($user_id !== null) {
         	$sql .= ' and `user_id` = ? ';
 			array_push($params, $user_id);
 		}
-		return $this->findEntity($sql,$params);
+		return $this->findEntity($sql, $params);
 	}
 
 	/**
@@ -97,10 +97,10 @@ 
 	 * @param $credential_id
 	 * @return Credential
 	 */
-	public function getCredentialLabelById($credential_id){
+	public function getCredentialLabelById($credential_id) {
 		$sql = 'SELECT id, label FROM `*PREFIX*passman_credentials` ' .
 			'WHERE `id` = ? ';
-		return $this->findEntity($sql,[$credential_id]);
+		return $this->findEntity($sql, [$credential_id]);
 	}
 
 	/**
@@ -108,7 +108,7 @@ 
 	 * @param $raw_credential
 	 * @return Credential
 	 */
-	public function create($raw_credential){
+	public function create($raw_credential) {
 		$credential = new Credential();
 
 		$credential->setGuid($this->utils->GUID());
@@ -131,7 +131,7 @@ 
 		$credential->setCustomFields($raw_credential['custom_fields']);
 		$credential->setOtp($raw_credential['otp']);
 		$credential->setHidden($raw_credential['hidden']);
-		if(isset($raw_credential['shared_key'])) {
+		if (isset($raw_credential['shared_key'])) {
 			$credential->setSharedKey($raw_credential['shared_key']);
 		}
 		return parent::insert($credential);
@@ -142,7 +142,7 @@ 
 	 * @param $raw_credential array An array containing all the credential fields
 	 * @return Credential The updated credential
 	 */
-	public function updateCredential($raw_credential){
+	public function updateCredential($raw_credential) {
 		$original = $this->getCredentialByGUID($raw_credential['guid']);
 		$credential = new Credential();
 		$credential->setId($original->getId());
@@ -170,11 +170,11 @@ 
 		return parent::update($credential);
 	}
 
-	public function deleteCredential(Credential $credential){
+	public function deleteCredential(Credential $credential) {
 		return $this->delete($credential);
 	}
 
-	public function upd(Credential $credential){
+	public function upd(Credential $credential) {
 		$this->update($credential);
 	}
 
@@ -183,10 +183,10 @@ 
      * @param $credential_guid
      * @return Credential
      */
-	public function getCredentialByGUID($credential_guid, $user_id = null){
+	public function getCredentialByGUID($credential_guid, $user_id = null) {
 	    $q = 'SELECT * FROM `*PREFIX*passman_credentials` WHERE guid = ? ';
 		$params = [$credential_guid];
-		if ($user_id !== null){
+		if ($user_id !== null) {
 			$q .= ' and `user_id` = ? ';
 			array_push($params, $user_id);
 		}

controller/filecontroller.php

Spacing +8 added lines, -8 removed lines

@@ -23,7 +23,7 @@ 
 	public function __construct($AppName,
 								IRequest $request,
 								$UserId,
-								FileService $fileService){
+								FileService $fileService) {
 		parent::__construct($AppName, $request);
 		$this->userId = $UserId;
 		$this->fileService = $fileService;
@@ -59,20 +59,20 @@ 
 		return new JSONResponse($this->fileService->deleteFile($file_id, $this->userId));
 	}
 
-	public function updateFile($file_id, $file_data, $filename, $mimetype, $size){
-		try{
+	public function updateFile($file_id, $file_data, $filename, $mimetype, $size) {
+		try {
 			$file = $this->fileService->getFile($file_id, $this->userId);
-		} catch (\Exception $doesNotExistException){
+		} catch (\Exception $doesNotExistException) {
 
 		}
-		if($file){
-			if($file_data) {
+		if ($file) {
+			if ($file_data) {
 				$file->setFileData($file_data);
 			}
-			if($filename) {
+			if ($filename) {
 				$file->setFilename($filename);
 			}
-			if($filename || $file_data){
+			if ($filename || $file_data) {
 				new JSONResponse($this->fileService->updateFile($file));
 			}
 		}

templates/part.admin.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 		<h2><?php p($l->t('Passman Settings')); ?></h2>
 		<?php
 		if ($checkVersion) {
-			p($l->t('Github version:'). ' '. $githubVersion);
+			p($l->t('Github version:') . ' ' . $githubVersion);
 			print '<br />';
 		} ?>
 		Local version: <?php p($localVersion); ?><br/>

lib/Service/SettingsService.php

Spacing +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
 	 * @param $value mixed Value of the setting
 	 */
 
-	public function setUserSetting($key, $value){
+	public function setUserSetting($key, $value) {
 		return $this->config->setUserValue($this->userId, $this->appName, $key, $value);
 	}
 
@@ -107,7 +107,7 @@ 
 	 * @param $setting
 	 * @return bool
 	 */
-	public function isEnabled($setting){
+	public function isEnabled($setting) {
 		$value = intval($this->config->getAppValue('passman', $setting, false));
 		return ($value === 1);
 	}

lib/AppInfo/Application.php

Spacing +5 added lines, -5 removed lines

@@ -45,7 +45,7 @@ 
 use OCP\IL10N;
 use OCP\Util;
 class Application extends App {
-	public function __construct () {
+	public function __construct() {
 		parent::__construct('passman');
 		$container = $this->getContainer();
 		// Allow automatic DI for the View, until we migrated to Nodes API
@@ -59,7 +59,7 @@ 
 		/**
 		 * Middleware
 		 */
-		$container->registerService('ShareMiddleware', function($c){
+		$container->registerService('ShareMiddleware', function($c) {
 			return new ShareMiddleware($c->query('SettingsService'));
 		});
 		$container->registerMiddleware('ShareMiddleware');
@@ -89,7 +89,7 @@ 
 
 
 		/** Cron **/
-		$container->registerService('CronService', function ($c) {
+		$container->registerService('CronService', function($c) {
 			return new CronService(
 				$c->query('CredentialService'),
 				$c->query('Logger'),
@@ -100,7 +100,7 @@ 
 			);
 		});
 
-		$container->registerService('Db', function () {
+		$container->registerService('Db', function() {
 			return new Db();
 		});
 
@@ -132,7 +132,7 @@ 
 		$c = $this->getContainer();
 		/** @var \OCP\IServerContainer $server */
 		$server = $c->getServer();
-		$navigationEntry = function () use ($c, $server) {
+		$navigationEntry = function() use ($c, $server) {
 			return [
 				'id' => $c->getAppName(),
 				'order' => 10,

middleware/sharemiddleware.php

Spacing +5 added lines, -5 removed lines

@@ -12,23 +12,23 @@ 
 
 	private $settings;
 
-	public function __construct(SettingsService $config){
+	public function __construct(SettingsService $config) {
 		$this->settings = $config;
 	}
 
 
 	public function beforeController($controller, $methodName) {
-		if ($controller instanceof ShareController){
+		if ($controller instanceof ShareController) {
 			$http_response_code = \OCP\AppFramework\Http::STATUS_FORBIDDEN;
 			$result = 'FORBIDDEN';
 
-			if(in_array($methodName, array('updateSharedCredentialACL', 'getFile', 'getItemAcl'))){
+			if (in_array($methodName, array('updateSharedCredentialACL', 'getFile', 'getItemAcl'))) {
 				$sharing_enabled = ($this->settings->isEnabled('link_sharing_enabled') || $this->settings->isEnabled('user_sharing_enabled'));
 			} else {
 				$publicMethods = array('createPublicShare', 'getPublicCredentialData');
 				$setting = (in_array($methodName, $publicMethods)) ? 'link_sharing_enabled' : 'user_sharing_enabled';
 				$sharing_enabled = ($this->settings->isEnabled($setting));
-				if($methodName === 'getVaultItems' || $methodName === 'getPendingRequests'){
+				if ($methodName === 'getVaultItems' || $methodName === 'getPendingRequests') {
 					$http_response_code = Http::STATUS_OK;
 					$result = array();
 				}
@@ -39,7 +39,7 @@ 
 				$response = new JSONResponse($result);
 				http_response_code($http_response_code);
 				header('Passman-sharing: disabled');
-				header('Passman-method: ShareController.'. $methodName);
+				header('Passman-method: ShareController.' . $methodName);
 				die($response->render());
 			}
 		}