nextcloud / passman

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

lib/Service/CredentialService.php

Spacing +5 added lines, -5 removed lines

@@ -47,7 +47,7 @@ 
 		return $this->credentialMapper->upd($credential);
 	}
 
-	public function deleteCredential($credential){
+	public function deleteCredential($credential) {
 		return $this->credentialMapper->deleteCredential($credential);
 	}
 
@@ -64,9 +64,9 @@ 
 		return $this->credentialMapper->getExpiredCredentials($timestamp);
 	}
 
-	public function getCredentialById($credential_id, $user_id){
+	public function getCredentialById($credential_id, $user_id) {
         $credential = $this->credentialMapper->getCredentialById($credential_id);
-        if ($credential->getUserId() === $user_id){
+        if ($credential->getUserId() === $user_id) {
             return $credential;
         }
         else {
@@ -77,11 +77,11 @@ 
 
         throw new DoesNotExistException("Did expect one result but found none when executing");
 	}
-	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

lib/Service/CronService.php

Spacing +5 added lines, -5 removed lines

@@ -37,7 +37,7 @@ 
 	public function expireCredentials() {
 		$this->logger->info('Passman cron test', array('app' => 'passman'));
 		$expired_credentials = $this->credentialService->getExpiredCredentials($this->utils->getTime());
-		foreach($expired_credentials as $credential){
+		foreach ($expired_credentials as $credential) {
 			$link = ''; // @TODO create direct link to credential
 
 			$sql = 'SELECT count(*) as rows from `*PREFIX*notifications` WHERE `subject`= \'credential_expired\' AND object_id=?';
@@ -45,9 +45,9 @@ 
 			$id = $credential->getId();
 			$query->bindParam(1, $id, \PDO::PARAM_INT);
 			$result = $query->execute();
-			$this->logger->debug($credential->getLabel() .' is expired, checking notifications!', array('app' => 'passman'));
-			if($result->fetchRow()['rows'] === 0) {
-				$this->logger->debug($credential->getLabel() .' is expired, adding notification!', array('app' => 'passman'));
+			$this->logger->debug($credential->getLabel() . ' is expired, checking notifications!', array('app' => 'passman'));
+			if ($result->fetchRow()['rows'] === 0) {
+				$this->logger->debug($credential->getLabel() . ' is expired, adding notification!', array('app' => 'passman'));
 
 				$this->activityService->add(
 					Activity::SUBJECT_ITEM_EXPIRED, array($credential->getLabel(), $credential->getUserId()),
@@ -55,7 +55,7 @@ 
 					$link, $credential->getUserId(), Activity::TYPE_ITEM_EXPIRED);
 				$this->notificationService->credentialExpiredNotification($credential);
 			} else {
-				$this->logger->debug($credential->getLabel() .' is expired, already notified!', array('app' => 'passman'));
+				$this->logger->debug($credential->getLabel() . ' is expired, already notified!', array('app' => 'passman'));
 			}
 
 		}

controller/credentialcontroller.php

Spacing +2 added lines, -2 removed lines

@@ -258,7 +258,7 @@ 
 		try {
 			$credential = $this->credentialService->getCredentialByGUID($credential_guid);
 		}
-		catch (DoesNotExistException $ex){
+		catch (DoesNotExistException $ex) {
 			return new NotFoundJSONResponse();
 		}
 
@@ -268,7 +268,7 @@ 
 		}
 		else {
 			$acl = $this->sharingService->getACL($this->userId, $credential_guid);
-			if ($acl->hasPermission(SharingACL::HISTORY)){
+			if ($acl->hasPermission(SharingACL::HISTORY)) {
 				$result = $this->credentialRevisionService->getRevisions($credential->getId());
 			}
 			else {