PathManipulation::getReducedPath() - Code Metrics - Inspection of "Cast share ID to string before getting/setting the..." - interfasys/galleryplus - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — stable9 (#52)

by Olivier

created 2016-06-29 16:45 UTC

PathManipulation::getReducedPath() A

↳ Parent: PathManipulation

Complexity

Conditions	3
Paths	4

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	19
ccs	10
cts	10
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	12
nc	4
nop	2
crap	3

<?php
/**
 * ownCloud - galleryplus
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Olivier Paroz <[email protected]>
 *
 * @copyright Olivier Paroz 2016
 */

namespace OCA\GalleryPlus\Controller;

/**
 * @package OCA\GalleryPlus\Controller
 */
trait PathManipulation {

	/**
	 * Returns a shortened path for the gallery view
	 *
	 * We only want to keep one folder between the current folder and the found media file
	 * /root/folder/sub1/sub2/file.ext
	 * becomes
	 * /root/folder/file.ext
	 *
	 * @param string $path the full path to a file, which never starts with a slash
	 * @param string $currFolderPath the current folder, which never starts with a slash
	 *
	 * @return string
	 */
	private function getReducedPath($path, $currFolderPath) {
		// Adding a slash to make sure we don't cut a folder in half
		if ($currFolderPath) {
			$currFolderPath .= '/';
			$relativePath = str_replace($currFolderPath, '', $path);
		} else {
			$relativePath = $path;
		}

		$subFolders = explode('/', $relativePath);

		if (count($subFolders) > 2) {
			$reducedPath = $currFolderPath . $subFolders[0] . '/' . array_pop($subFolders);
		} else {
			$reducedPath = $path;
		}

		return $reducedPath;
	}

}


1		<?php
2		/**
3		* ownCloud - galleryplus
4		*
5		* This file is licensed under the Affero General Public License version 3 or
6		* later. See the COPYING file.
7		*
8		* @author Olivier Paroz <[email protected]>
9		*
10		* @copyright Olivier Paroz 2016
11		*/
12
13		namespace OCA\GalleryPlus\Controller;
14
15		/**
16		* @package OCA\GalleryPlus\Controller
17		*/
18		trait PathManipulation {
19
20		/**
21		* Returns a shortened path for the gallery view
22		*
23		* We only want to keep one folder between the current folder and the found media file
24		* /root/folder/sub1/sub2/file.ext
25		* becomes
26		* /root/folder/file.ext
27		*
28		* @param string $path the full path to a file, which never starts with a slash
29		* @param string $currFolderPath the current folder, which never starts with a slash
30		*
31		* @return string
32		*/
33	16	private function getReducedPath($path, $currFolderPath) {
34		// Adding a slash to make sure we don't cut a folder in half
35	16	if ($currFolderPath) {
36	13	$currFolderPath .= '/';
37	13	$relativePath = str_replace($currFolderPath, '', $path);
38		} else {
39	3	$relativePath = $path;
40		}
41
42	16	$subFolders = explode('/', $relativePath);
43
44	16	if (count($subFolders) > 2) {
45	3	$reducedPath = $currFolderPath . $subFolders[0] . '/' . array_pop($subFolders);
46		} else {
47	13	$reducedPath = $path;
48		}
49
50	16	return $reducedPath;
51		}
52
53		}
54

interfasys / galleryplus

Pull Request — stable9 (#52)

PathManipulation::getReducedPath() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like