PlaylistMapper::find() - Code Metrics - Inspection of "Properly handle deleting/unsharing folder which co..." - owncloud/music - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — scanner_improvements ( 4bed41 )

by Pauli

created 2017-06-27 07:58 UTC

PlaylistMapper::find() A

↳ Parent: PlaylistMapper::findListsContainingTrack()

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	5
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	4
nc	1
nop	2

<?php

/**
 * ownCloud - Music app
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Morris Jobke <[email protected]>
 * @author Volkan Gezer <[email protected]>
 * @copyright Morris Jobke 2014
 * @copyright Volkan Gezer 2014
 */

namespace OCA\Music\Db;

use OCP\IDBConnection;

class PlaylistMapper extends BaseMapper {

	public function __construct(IDBConnection $db){
		parent::__construct($db, 'music_playlists', '\OCA\Music\Db\Playlist');
	}

	/**
	 * @param string $condition
	 */
	private function makeSelectQuery($condition=null){
		return 'SELECT `name`, `id`, `track_ids` ' .
			'FROM `*PREFIX*music_playlists` ' .
			'WHERE `user_id` = ? ' . $condition;
	}

	/**
	 * @param string $userId
	 * @param integer $limit
	 * @param integer $offset
	 * @return Playlist[]
	 */
	public function findAll($userId, $limit=null, $offset=null){
		$sql = $this->makeSelectQuery();
		$params = array($userId);
		return $this->findEntities($sql, $params, $limit, $offset);
	}

	/**
	 * @param int $trackId
	 * @return Playlist[]
	 */
	 public function findListsContainingTrack($trackId) {
		$sql = 'SELECT * ' .
			'FROM `*PREFIX*music_playlists` ' .
			'WHERE `track_ids` LIKE ?';
		$params = array('%|' . $trackId . '|%');
		return $this->findEntities($sql, $params);
	}
}


1			<?php
2
3			/**
4			* ownCloud - Music app
5			*
6			* This file is licensed under the Affero General Public License version 3 or
7			* later. See the COPYING file.
8			*
9			* @author Morris Jobke <[email protected]>
10			* @author Volkan Gezer <[email protected]>
11			* @copyright Morris Jobke 2014
12			* @copyright Volkan Gezer 2014
13			*/
14
15			namespace OCA\Music\Db;
16
17			use OCP\IDBConnection;
18
19			class PlaylistMapper extends BaseMapper {
20
21			public function __construct(IDBConnection $db){
22			parent::__construct($db, 'music_playlists', '\OCA\Music\Db\Playlist');
23			}
24
25			/**
26			* @param string $condition
27			*/
28			private function makeSelectQuery($condition=null){
29			return 'SELECT `name`, `id`, `track_ids` ' .
30			'FROM `PREFIXmusic_playlists` ' .
31			'WHERE `user_id` = ? ' . $condition;
32			}
33
34			/**
35			* @param string $userId
36			* @param integer $limit
37			* @param integer $offset
38			* @return Playlist[]
39			*/
40			public function findAll($userId, $limit=null, $offset=null){
41			$sql = $this->makeSelectQuery();
42			$params = array($userId);
43			return $this->findEntities($sql, $params, $limit, $offset);
44			}
45
46			/**
47			* @param int $trackId
48			* @return Playlist[]
49			*/
50			public function findListsContainingTrack($trackId) {
51			$sql = 'SELECT * ' .
52			'FROM `PREFIXmusic_playlists` ' .
53			'WHERE `track_ids` LIKE ?';
54			$params = array('%\|' . $trackId . '\|%');
55			return $this->findEntities($sql, $params);
56			}
57			}
58

owncloud / music

Push — scanner_improvements ( 4bed41 )

PlaylistMapper::find() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like