ar_template_filestore::load() - Code Metrics - Inspection of "remove unused code" - Ariadne-CMS/ariadne - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — php7.2-travis ( be5e72...caaf61 )

unknown

created 2018-12-10 15:06 UTC

ar_template_filestore::load() A

↳ Parent: ar_template_filestore

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
nc	1
nop	2
dl	0
loc	6
rs	10
c	0
b	0
f	0
ccs	0
cts	5
cp	0
crap	2

<?php

	class ar_template_filestore extends arBase {

		private function getFilestore() {
			$context = ar::context()->getObject();
			if(isset($context)) {
				$templates = $context->store->get_filestore("templates");
			} else {
				global $store;
				$templates = $store->get_filestore("templates");
			}
			return $templates;
		}

		private function pathToId($path) {
			$context = ar::context()->getObject();

			if($context->path == $path) {
				$result = $context->id;
			} else {
				$result = $context->loadConfig($path)->id;
			}
			return $result;
		}

		private function getConfig($path) {
			$context = ar::context()->getObject();
			return $context->loadConfig($path);
		}

		public function get($path, $name){
			$fs = $this->getfilestore();
			$id = $this->pathtoid($path);

			return (
				$fs->import($id, $name) 
			);
		}

		public function save($path, $name, $template, $local=null, $private=null) {

			return false;
		}

		public function load($path, $name) {
			$fs = $this->getfilestore();
			$id = $this->pathtoid($path);

			return $fs->read($id, $name . '.pinp');
		}

		public function ls($path) {
			$result = [];
			$fs = $this->getFilestore();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
			$id = $this->pathToId($path);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

			$config = $this->getConfig($path);

			$templates = $config->pinpTemplates;
			if (isset($templates)) foreach($templates as $type => $names) {
				if (isset($names)) foreach($names as $name => $languages) {
					if (isset($languages)) foreach($languages as $language => $id ) {
						$tempname = sprintf("%s.%s.%s",$type,$name,$language);
						list($maintype,$subtype) = explode('.', $type, 2);
						if(!isset($result[$name])) {
							$result[$name] = [];
						}
						$result[$name][] = [
							'id'       => $config->id,
							'path'     => $path,
							'type'     => $maintype,
							'subtype'  => $subtype,
							'name'     => $name,
							'filename' => $tempname,
							'language' => $language,
							'private'  => isset($config->privatetemplates[$type][$name]),
							'local'    => !isset($config->localTemplates[$type][$name][$language]),
						];
					}
				}
			}
			return $result;

		}

		public function rm($path, $name){
			$fs = $this->getfilestore();
			$id = $this->pathtoid($path);

			return (
				$fs->remove($id, $name) 
			);
		}

		public function exists($path, $name) {
			$fs = $this->getFilestore();
			$id = $this->pathToId($path);

			return (
				$fs->exists($id, $name . '.inc') 
			);
		}

		public function compile($path, $name) {

			// FIXME
		}
	}


1			<?php
2
3			class ar_template_filestore extends arBase {
4
5			private function getFilestore() {
6			$context = ar::context()->getObject();
7			if(isset($context)) {
8			$templates = $context->store->get_filestore("templates");
9			} else {
10			global $store;
11			$templates = $store->get_filestore("templates");
12			}
13			return $templates;
14			}
15
16			private function pathToId($path) {
17			$context = ar::context()->getObject();
18
19			if($context->path == $path) {
20			$result = $context->id;
21			} else {
22			$result = $context->loadConfig($path)->id;
23			}
24			return $result;
25			}
26
27			private function getConfig($path) {
28			$context = ar::context()->getObject();
29			return $context->loadConfig($path);
30			}
31
32			public function get($path, $name){
33			$fs = $this->getfilestore();
34			$id = $this->pathtoid($path);
35
36			return (
37			$fs->import($id, $name)
38			);
39			}
40
41			public function save($path, $name, $template, $local=null, $private=null) {
			0 ignored issues – show Unused Code introduced 2018-02-08 10:26 UTC by Report Bug Copy Issue Report The parameter `$path` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2018-02-08 10:26 UTC by Report Bug Copy Issue Report The parameter `$name` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2018-02-08 10:26 UTC by Report Bug Copy Issue Report The parameter `$template` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2018-02-08 10:26 UTC by Report Bug Copy Issue Report The parameter `$local` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2018-02-08 10:26 UTC by Report Bug Copy Issue Report The parameter `$private` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
42			return false;
43			}
44
45			public function load($path, $name) {
46			$fs = $this->getfilestore();
47			$id = $this->pathtoid($path);
48
49			return $fs->read($id, $name . '.pinp');
50			}
51
52			public function ls($path) {
53			$result = [];
54			$fs = $this->getFilestore();
			0 ignored issues – show Unused Code introduced 2016-05-20 15:31 UTC by Report Bug Copy Issue Report `$fs` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
55			$id = $this->pathToId($path);
			0 ignored issues – show Unused Code introduced 2016-05-20 15:31 UTC by Report Bug Copy Issue Report `$id` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
56
57			$config = $this->getConfig($path);
58
59			$templates = $config->pinpTemplates;
60			if (isset($templates)) foreach($templates as $type => $names) {
61			if (isset($names)) foreach($names as $name => $languages) {
62			if (isset($languages)) foreach($languages as $language => $id ) {
63			$tempname = sprintf("%s.%s.%s",$type,$name,$language);
64			list($maintype,$subtype) = explode('.', $type, 2);
65			if(!isset($result[$name])) {
66			$result[$name] = [];
67			}
68			$result[$name][] = [
69			'id' => $config->id,
70			'path' => $path,
71			'type' => $maintype,
72			'subtype' => $subtype,
73			'name' => $name,
74			'filename' => $tempname,
75			'language' => $language,
76			'private' => isset($config->privatetemplates[$type][$name]),
77			'local' => !isset($config->localTemplates[$type][$name][$language]),
78			];
79			}
80			}
81			}
82			return $result;
83
84			}
85
86			public function rm($path, $name){
87			$fs = $this->getfilestore();
88			$id = $this->pathtoid($path);
89
90			return (
91			$fs->remove($id, $name)
92			);
93			}
94
95			public function exists($path, $name) {
96			$fs = $this->getFilestore();
97			$id = $this->pathToId($path);
98
99			return (
100			$fs->exists($id, $name . '.inc')
101			);
102			}
103
104			public function compile($path, $name) {
			0 ignored issues – show Unused Code introduced 2018-02-08 10:26 UTC by Report Bug Copy Issue Report The parameter `$path` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2018-02-08 10:26 UTC by Report Bug Copy Issue Report The parameter `$name` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
105			// FIXME
106			}
107			}
108

Ariadne-CMS / ariadne

Push — php7.2-travis ( be5e72...caaf61 )

ar_template_filestore::load() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like