OCA\NextNote\ShareBackend\NextNoteShareBackend

Complexity

Total Complexity

12

Size/Duplication

Total Lines	134
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getSharedNotes()	0	3	1
A	isValidSource()	0	6	1
A	generateTarget()	0	4	1
B	formatItems()	0	30	5
A	isShareTypeAllowed()	0	3	1
A	checkPermissions()	0	13	2

<?php
/**
 * Nextcloud - NextNote
 *
 * @copyright Copyright (c) 2015, Ben Curtis <ownclouddev@nosolutions.com>
 * @copyright Copyright (c) 2017, Sander Brand (brantje@gmail.com)
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */

namespace OCA\NextNote\ShareBackend;

use OC\Share\Share;
use OCA\NextNote\Db\NextNote;
use OCA\NextNote\Fixtures\ShareFix;
use \OCP\Share_Backend;

class NextNoteShareBackend implements Share_Backend {

	private $db;
	private static $defaultPermissions = 31;

The property $defaultPermissions is not used and could be removed.

	public function __construct() {
		$this->db = \OC::$server->getDatabaseConnection();
	}

	public function getSharedNotes(){
		return Share::getItemsSharedWith('nextnote', 'populated_shares');
	}

	/**
	 * Check if this $itemSource exist for the user
	 * @param string $itemSource
	 * @param string $uidOwner Owner of the item
	 * @return boolean|null Source
	 *
	 * Return false if the item does not exist for the user
	 * @since 5.0.0
	 */
	public function isValidSource($itemSource, $uidOwner) {
		// todo: real test
		// id => 1, has admin 
		// has owner this note?
		return true;
	}
	
	/**
	 * Get a unique name of the item for the specified user
	 * @param string $itemSource
	 * @param string|false $shareWith User the item is being shared with
	 * @param array|null $exclude List of similar item names already existing as shared items @deprecated since version OC7
	 * @return string Target name
	 *
	 * This function needs to verify that the user does not already have an item with this name.
	 * If it does generate a new name e.g. name_#
	 * @since 5.0.0
	 */
	public function generateTarget($itemSource, $shareWith, $exclude = null) {
		// note id (should be unique)
		return $itemSource;
	}

	/**
	 * Converts the shared item sources back into the item in the specified format
	 * @param array $items Shared items
	 * @param int $format
	 * @return array
	 *
	 * The items array is a 3-dimensional array with the item_source as the
	 * first key and the share id as the second key to an array with the share
	 * info.
	 *
	 * The key/value pairs included in the share info depend on the function originally called:
	 * If called by getItem(s)Shared: id, item_type, item, item_source,
	 * share_type, share_with, permissions, stime, file_source
	 *
	 * If called by getItem(s)SharedWith: id, item_type, item, item_source,
	 * item_target, share_type, share_with, permissions, stime, file_source,
	 * file_target
	 *
	 * This function allows the backend to control the output of shared items with custom formats.
	 * It is only called through calls to the public getItem(s)Shared(With) functions.
	 * @since 5.0.0
	 */
	public function formatItems($items, $format, $parameters = null) {
		if ($format === 'shares') {

The strict comparison === seems to always evaluate to false as the types of $format (integer) and 'shares' (string) can never be identical. Maybe you want to use a loose comparison == instead?

			return $items;
		}

		// get the ownnote ids
		$ids = Array();
		foreach($items as $item) {
			$ids[] = $item['item_source'];
		}

		// get notes from database
		$select_clause = "SELECT id, uid, name, grouping, mtime, deleted FROM *PREFIX*nextnote WHERE id in (";
		$select_clause .= implode(',', $ids);
		$select_clause .= ") ORDER BY id";
		$q = $this->db->executeQuery($select_clause, array());
		//$query = \OCP\DB::prepare($select_clause);
		$results = $q->fetchAll();

		// add permissions to items
		if ($format === 'populated_shares') {

The strict comparison === seems to always evaluate to false as the types of $format (integer) and 'populated_shares' (string) can never be identical. Maybe you want to use a loose comparison == instead?

			$full_items = Array();
			foreach($results as $index => $result) {
				$full_items[] = array_merge($items[$index], $result);
			}
			$results = $full_items;
		}

		return $results;
	}
	
	/**
	 * Check if a given share type is allowd by the back-end
	 *
	 * @param int $shareType share type
	 * @return boolean
	 *
	 * The back-end can enable/disable specific share types. Just return true if
	 * the back-end doesn't provide any specific settings for it and want to allow
	 * all share types defined by the share API
	 * @since 8.0.0
	 */
	public function isShareTypeAllowed($shareType) {
		return true;
	}

	/**
	 * Check if the current user has the requested permission.
	 * For permissions
	 * @see \OCP\Constants
	 * @param $permission
	 * @param $note NextNote
	 * @return bool|int
	 */
	public function checkPermissions($permission, $note) {
		// gather information
		$uid = \OC::$server->getUserSession()->getUser()->getUID();
		// owner is allowed to do everything
		if ($uid === $note->getUid()) {
			return true;
		}

		// check share permissions
		$gotPerm = ShareFix::getPermissions('nextnote', $note->getId(), $uid);

		return $gotPerm & $permission;
	}
}