OCA\Passman\Db\VaultMapper

Complexity

Total Complexity

8

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

82.5%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	find()	0	5	1
A	findByGuid()	0	5	1
A	findVaultsFromUser()	0	6	1
A	create()	0	9	1
A	setLastAccess()	0	7	1
A	updateVault()	0	3	1
A	updateSharingKeys()	0	8	1

<?php
/**
 * Nextcloud - passman
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Sander Brand <brantje@gmail.com>
 * @copyright Sander Brand 2016
 */
namespace OCA\Passman\Db;

use OCA\Passman\Utility\Utils;
use OCP\IDBConnection;
use OCP\AppFramework\Db\Mapper;

class VaultMapper extends Mapper {
	private $utils;
	public function __construct(IDBConnection $db, Utils $utils) {
		parent::__construct($db, 'passman_vaults');
		$this->utils = $utils;
	}


	/**
	 * @throws \OCP\AppFramework\Db\DoesNotExistException if not found
	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException if more than one result
	 * @return Vault
	 */
	public function find($vault_id, $user_id) {
		$sql = 'SELECT * FROM `*PREFIX*passman_vaults` ' .
			'WHERE `id`= ? and `user_id` = ?';
		return $this->findEntities($sql, [$vault_id, $user_id]);
	}
	/**
	 * @throws \OCP\AppFramework\Db\DoesNotExistException if not found
	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException if more than one result
	 * @return Vault
	 */
	public function findByGuid($vault_guid, $user_id) {
		$sql = 'SELECT * FROM `*PREFIX*passman_vaults` ' .
			'WHERE `guid`= ? and `user_id` = ?';
		return $this->findEntity($sql, [$vault_guid, $user_id]);
	}


	/**
	 * @throws \OCP\AppFramework\Db\DoesNotExistException if not found
	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException if more than one result
	 * @return Vault[]
	 */
	public function findVaultsFromUser($userId){
		$sql = 'SELECT * FROM `*PREFIX*passman_vaults` ' .
			'WHERE `user_id` = ? ';
		$params = [$userId];
		return $this->findEntities($sql, $params);
	}

	/**
	 * Creates a vault
	 * @param $vault_name
	 * @param $userId
	 * @return Vault
	 */
	public function create($vault_name, $userId){
		$vault = new Vault();
		$vault->setName($vault_name);
		$vault->setUserId($userId);
		$vault->setGuid($this->utils->GUID());
		$vault->setCreated($this->utils->getTime());
		$vault->setLastAccess(0);
		return parent::insert($vault);

It seems like you call parent on a different method (insert() instead of create()). Are you sure this is correct? If so, you might want to change this to $this->insert().

	}

	public function setLastAccess($vault_id, $user_id){
		$vault = new Vault();
		$vault->setId($vault_id);
		$vault->setUserId($user_id);
		$vault->setLastAccess(Utils::getTime());
		$this->update($vault);
	}

	public function updateVault(Vault $vault){
		$this->update($vault);
	}

	public function updateSharingKeys($vault_id, $privateKey, $publicKey){
		$vault = new Vault();
		$vault->setId($vault_id);
		$vault->setPrivateSharingKey($privateKey);
		$vault->setPublicSharingKey($publicKey);
		$vault->setSharingKeysGenerated($this->utils->getTime());
		$this->update($vault);
	}
}