OCA\Music\BusinessLayer\PlaylistBusinessLayer

Complexity

Total Complexity

10

Size/Duplication

Total Lines	75
Duplicated Lines	18.67 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	find()	0	3	1
A	findAll()	0	3	1
A	addTracks()	7	7	1
A	removeTracks()	7	7	1
A	create()	0	7	1
A	rename()	0	6	1
A	removeTracksFromAllLists()	0	11	3

<?php

/**
 * ownCloud - Music app
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Pauli J�rvinen <pauli.jarvinen@gmail.com>
 * @copyright Pauli J�rvinen 2016
 */

namespace OCA\Music\BusinessLayer;

use \OCA\Music\AppFramework\BusinessLayer\BusinessLayer;
use \OCA\Music\AppFramework\BusinessLayer\BusinessLayerException;
use \OCA\Music\AppFramework\Core\Logger;
use \OCP\AppFramework\Db\DoesNotExistException;
use \OCP\AppFramework\Db\MultipleObjectsReturnedException;


use \OCA\Music\Db\PlaylistMapper;
use \OCA\Music\Db\Playlist;

class PlaylistBusinessLayer extends BusinessLayer {

	private $logger;

	public function __construct(PlaylistMapper $playlistMapper, Logger $logger) {
		parent::__construct($playlistMapper);
		$this->logger = $logger;
	}

	/**
	 * Return a playlist
	 * @param int $playlistId the id of the playlist
	 * @param string $userId the name of the user
	 * @return Playlist playlist
	 */
	public function find($playlistId, $userId) {
		return $this->mapper->find($playlistId, $userId);
	}

	/**
	 * Returns all playlists
	 * @param string $userId the name of the user
	 * @return Playlist[] playlists
	 */
	public function findAll($userId) {
		return $this->mapper->findAll($userId);
	}

	public function addTracks($trackIds, $playlistId, $userId) {

This method seems to be duplicated in your project.

		$playlist = $this->find($playlistId, $userId);
		$prevTrackIds = $playlist->getTrackIdsAsArray();
		$playlist->setTrackIdsFromArray(array_merge($prevTrackIds, $trackIds));
		$this->mapper->update($playlist);
		return $playlist;
	}

	public function removeTracks($trackIds, $playlistId, $userId) {

This method seems to be duplicated in your project.

		$playlist = $this->find($playlistId, $userId);
		$prevTrackIds = $playlist->getTrackIdsAsArray();
		$playlist->setTrackIdsFromArray(array_diff($prevTrackIds, $trackIds));
		$this->mapper->update($playlist);
		return $playlist;
	}

	public function create($name, $userId) {
		$playlist = new Playlist();
		$playlist->setName($name);
		$playlist->setUserId($userId);

		return $this->mapper->insert($playlist);
	}

	public function rename($name, $playlistId, $userId) {
		$playlist = $this->find($playlistId, $userId);
		$playlist->setName($name);
		$this->mapper->update($playlist);
		return $playlist;
	}

	/**
	 * removes tracks from all available playlists
	 * @param int[] $trackIds array of all track IDs to remove
	 */
	public function removeTracksFromAllLists($trackIds) {
		foreach ($trackIds as $trackId) {
			$affectedLists = $this->mapper->findListsContainingTrack($trackId);

			foreach ($affectedLists as $playlist) {
				$prevTrackIds = $playlist->getTrackIdsAsArray();
				$playlist->setTrackIdsFromArray(array_diff($prevTrackIds, [$trackId]));
				$this->mapper->update($playlist);
			}
		}
	}
}