nextcloud / passman

lib/Db/CredentialMapper.php

Doc Comments +6 added lines

@@ -33,12 +33,18 @@
 		return $this->findEntities($sql, [$user_id, $vault_id]);
 	}
 
+	/**
+	 * @param integer $vault_id
+	 */
 	public function getRandomCredentialByVaultId($vault_id, $user_id) {
 		$sql = 'SELECT * FROM `*PREFIX*passman_credentials` ' .
 			'WHERE `user_id` = ? and vault_id = ? AND shared_key is NULL ORDER BY RAND() LIMIT 1';
 		return $this->findEntities($sql, [$user_id, $vault_id]);
 	}
 
+	/**
+	 * @param integer $timestamp
+	 */
 	public function getExpiredCredentials($timestamp) {
 		$sql = 'SELECT * FROM `*PREFIX*passman_credentials` ' .
 			'WHERE `expire_time` > 0 AND `expire_time` < ?';

controller/credentialcontroller.php

Braces +3 added lines, -6 removed lines

@@ -255,21 +255,18 @@
 	public function getRevision($credential_guid) {
 		try {
 			$credential = $this->credentialService->getCredentialByGUID($credential_guid);
-		}
-		catch (DoesNotExistException $ex) {
+		} catch (DoesNotExistException $ex) {
 			return new NotFoundJSONResponse();
 		}
 
 		// If the request was made by the owner of the credential
 		if ($this->userId === $credential->getUserId()) {
 			$result = $this->credentialRevisionService->getRevisions($credential->getId(), $this->userId);
-		}
-		else {
+		} else {
 			$acl = $this->sharingService->getACL($this->userId, $credential_guid);
 			if ($acl->hasPermission(SharingACL::HISTORY)) {
 				$result = $this->credentialRevisionService->getRevisions($credential->getId());
-			}
-			else {
+			} else {
 				return new NotFoundJSONResponse();
 			}
 		}