nextcloud / passman

lib/Utility/PermissionEntity.php

Spacing +4 added lines, -4 removed lines

@@ -13,11 +13,11 @@
 use OCP\AppFramework\Db\Entity;
 
 class PermissionEntity extends Entity {
-	CONST READ  =   0b00000001;
-	CONST WRITE =   0b00000010;
-	CONST FILES =   0b00000100;
+	CONST READ  = 0b00000001;
+	CONST WRITE = 0b00000010;
+	CONST FILES = 0b00000100;
 	CONST HISTORY = 0b00001000;
-	CONST OWNER =   0b10000000;
+	CONST OWNER = 0b10000000;
 
 	/**
 	 * Checks wether a user matches one or more permissions at once

lib/Db/SharingACLMapper.php

Spacing +7 added lines, -7 removed lines

@@ -29,13 +29,13 @@ 
 	 * @param $item_guid
 	 * @return SharingACL[]
 	 */
-	public function getCredentialPermissions(IUser $userId, $item_guid){
-		$sql = "SELECT * FROM ". self::TABLE_NAME ." WHERE user_id = ? AND item_guid = ?";
+	public function getCredentialPermissions(IUser $userId, $item_guid) {
+		$sql = "SELECT * FROM " . self::TABLE_NAME . " WHERE user_id = ? AND item_guid = ?";
 
 		return $this->findEntities($sql, [$userId, $item_guid]);
 	}
 
-	public function createACLEntry(SharingACL $acl){
+	public function createACLEntry(SharingACL $acl) {
 		return $this->insert($acl);
 	}
 
@@ -46,7 +46,7 @@ 
 	 * @return SharingACL[]
 	 */
 	public function getVaultEntries($user_id, $vault_id) {
-		$q = "SELECT * FROM ". self::TABLE_NAME ." WHERE user_id = ? AND vault_guid = ?";
+		$q = "SELECT * FROM " . self::TABLE_NAME . " WHERE user_id = ? AND vault_guid = ?";
 		return $this->findEntities($q, [$user_id, $vault_id]);
 	}
 
@@ -59,7 +59,7 @@ 
 	public function getItemACL($user_id, $item_guid) {
 		$q = "SELECT * FROM " . self::TABLE_NAME . " WHERE item_guid = ? AND ";
 		$filter = [$item_guid];
-		if ($user_id === null){
+		if ($user_id === null) {
 			$q .= 'user_id is null';
 		}
 		else {
@@ -86,11 +86,11 @@ 
 	 * @return SharingACL[]
 	 */
 	public function getCredentialAclList($item_guid) {
-		$q = "SELECT * FROM ". self::TABLE_NAME ." WHERE item_guid = ?";
+		$q = "SELECT * FROM " . self::TABLE_NAME . " WHERE item_guid = ?";
 		return $this->findEntities($q, [$item_guid]);
 	}
 
-	public function deleteShareACL(SharingACL $ACL){
+	public function deleteShareACL(SharingACL $ACL) {
 		return $this->delete($ACL);
 	}
 }
\ No newline at end of file

Braces +1 added lines, -2 removed lines

@@ -61,8 +61,7 @@
 		$filter = [$item_guid];
 		if ($user_id === null){
 			$q .= 'user_id is null';
-		}
-		else {
+		} else {
 			$q .= 'user_id = ? ';
 			$filter[] = $user_id;
 		}

lib/Service/CredentialService.php

Indentation +13 added lines, -13 removed lines

@@ -81,17 +81,17 @@ 
 	}
 
 	public function getCredentialById($credential_id, $user_id) {
-        $credential = $this->credentialMapper->getCredentialById($credential_id);
-        if ($credential->getUserId() === $user_id) {
-            return $credential;
-        }
-        else {
-            $acl = $this->sharingACL->getItemACL($user_id, $credential->getGuid());
-            if ($acl->hasPermission(SharingACL::READ));
-            return $credential;
-        }
-
-        throw new DoesNotExistException("Did expect one result but found none when executing");
+		$credential = $this->credentialMapper->getCredentialById($credential_id);
+		if ($credential->getUserId() === $user_id) {
+			return $credential;
+		}
+		else {
+			$acl = $this->sharingACL->getItemACL($user_id, $credential->getGuid());
+			if ($acl->hasPermission(SharingACL::READ));
+			return $credential;
+		}
+
+		throw new DoesNotExistException("Did expect one result but found none when executing");
 	}
 
 	/**
@@ -102,6 +102,6 @@ 
 	}
 
 	public function getCredentialByGUID($credential_guid, $user_id = null){
-	    return $this->credentialMapper->getCredentialByGUID($credential_guid, $user_id);
-    }
+		return $this->credentialMapper->getCredentialByGUID($credential_guid, $user_id);
+	}
 }
\ No newline at end of file

Spacing +3 added lines, -3 removed lines

@@ -54,7 +54,7 @@ 
 	/**
 	 * @param Credential $credential
 	 */
-	public function deleteCredential($credential){
+	public function deleteCredential($credential) {
 		return $this->credentialMapper->deleteCredential($credential);
 	}
 
@@ -97,11 +97,11 @@ 
 	/**
 	 * @param integer $credential_id
 	 */
-	public function getCredentialLabelById($credential_id){
+	public function getCredentialLabelById($credential_id) {
 		return $this->credentialMapper->getCredentialLabelById($credential_id);
 	}
 
-	public function getCredentialByGUID($credential_guid, $user_id = null){
+	public function getCredentialByGUID($credential_guid, $user_id = null) {
 	    return $this->credentialMapper->getCredentialByGUID($credential_guid, $user_id);
     }
 }
\ No newline at end of file