ConfigManager - Code Metrics - realshadow/satis-control-panel - Measure and Improve Code Quality continuously with Scrutinizer

ConfigManager A
last analyzed 2018-06-29 09:03 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	337
Duplicated Lines	8.9 %

Coupling/Cohesion

Components	1
Dependencies	17

Importance

Changes

Metric	Value
wmc	36
lcom	1
cbo	17
dl	30
loc	337
rs	9.52
c	0
b	0
f	0

17 Methods

Rating	Name	Duplication	Size	Complexity
A	loadSatisConfig()	0	7	1
A	getLogger()	0	9	1
A	_addOrUpdateRepository()	0	34	5
A	_addOrUpdatePackage()	0	33	5
A	_deleteRepository()	0	11	2
A	_deletePackage()	0	11	2
A	nameToId()	0	3	1
A	__construct()	0	7	1
A	__call()	0	29	3
A	__get()	0	11	2
A	setDisableBuild()	0	5	1
A	getRepositories()	15	15	3
A	getPackages()	15	15	3
A	forceBuild()	0	9	2
A	isBuilding()	0	3	1
A	getDefinition()	0	3	1
A	save()	0	11	2

How to fix Duplicated Code

<?php

namespace App\Satis;

use App\Satis\Collections\PackageCollection;
use App\Satis\Context\PrivateRepository;
use App\Satis\Context\PublicRepository;
use App\Satis\Exceptions\PackageNotFoundException;
use App\Satis\Model\Package;
use BadMethodCallException;
use App\Satis\Collections\RepositoryCollection;
use App\Satis\Exceptions\RepositoryNotFoundException;
use App\Satis\Model\Repository;
use Illuminate\Support\Collection;
use JMS\Serializer\Serializer;
use Monolog\Formatter\LineFormatter;
use Monolog\Handler\StreamHandler;
use Monolog\Logger;

/**
 * @method string addOrUpdateRepository($repositoryId, Collection $repositoryData);
 * @method string addOrUpdatePackage($packageId, Collection $packageData);
 * @method void deleteRepository($repositoryId);
 * @method void deletePackage($packageId);
 * @property \App\Satis\Model\Config $satis;
 *
 * @author Lukas Homza <[email protected]>
 */
class ConfigManager {
    const BUILD_PRIVATE = 1;
    const BUILD_PUBLIC = 2;

    /** @var \App\Satis\ConfigPersister $configPersister */
    protected $configPersister;
    /** @var \App\Satis\ConfigBuilder $configBuilder */
    protected $configBuilder;
    /** @var \JMS\Serializer\Serializer $serializer */
    protected $serializer;
    /** @var \App\Satis\Model\Config $_satis */
    protected $_satis;

    /** @var bool $disableBuild */
    protected $disableBuild = false;

    /** @var string  */
    protected $hiddenEntityPrefix = '_';

    /** @var array  */
    protected $lazyLoadedProperties = [
        '_satis' => 'loadSatisConfig'
    ];

    /**
     * @return \App\Satis\Model\Config
     * @throws \Illuminate\Contracts\Filesystem\FileNotFoundException
     */
    protected function loadSatisConfig() {
        return $this->serializer->deserialize(
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
            $this->configPersister->load(),
            'App\Satis\Model\Config',
            'json'
        );
    }

    /**
     * @return \Monolog\Logger
     */
    protected function getLogger() {
        $handler = new StreamHandler(storage_path('logs/api_request.log'), Logger::DEBUG);
        $handler->setFormatter(new LineFormatter(null, null, true, true));

        $logger = new Logger('ApiRequestLog');
        $logger->pushHandler($handler);

        return $logger;
    }

    /**
     * @param \App\Satis\Collections\RepositoryCollection $repositoryCollection
     * @param string $repositoryId
     * @param \Illuminate\Support\Collection $input
     * @return \App\Satis\Collections\RepositoryCollection
     * @throws \App\Satis\Exceptions\RepositoryNotFoundException
     */
    protected function _addOrUpdateRepository(RepositoryCollection $repositoryCollection, $repositoryId,
        Collection $input
    ) {
        $repository = new Repository();
        $repository->setUrl($input->get('url'));
        $repository->setType($input->get('type', config('satis.default_repository_type')));

        if($input->has('async_mode')) {
            $this->configBuilder->setAsyncMode(filter_var($input->get('async_mode'), FILTER_VALIDATE_BOOLEAN));
        }

        if($input->has('disable_build')) {
            $this->setDisableBuild(filter_var($input->get('disable_build'), FILTER_VALIDATE_BOOLEAN));
        }

        if($repositoryId !== null) {
            if($repositoryCollection->has($repositoryId)) {
                $repositoryCollection->put($repositoryId, $repository);
            } else {
                throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
            }
        } else {
            $repositoryId = static::nameToId($repository->getUrl());

            $repositoryCollection->put($repositoryId, $repository);
        }

        $buildContext = new PrivateRepository();
        $buildContext->setItemId($repositoryId)->setItemName($repository->getUrl());

        $this->configBuilder->setBuildContext($buildContext);

        return $repositoryCollection;
    }

    /**
     * @param \App\Satis\Collections\PackageCollection $packageCollection
     * @param string $packageId
     * @param \Illuminate\Support\Collection $input
     * @return \App\Satis\Collections\RepositoryCollection
     * @throws \App\Satis\Exceptions\PackageNotFoundException
     */
    protected function _addOrUpdatePackage(PackageCollection $packageCollection, $packageId,
        Collection $input
    ) {
        $package = new Package();
        $package->setName($input->get('name'));

        if($input->has('async_mode')) {
            $this->configBuilder->setAsyncMode(filter_var($input->get('async_mode'), FILTER_VALIDATE_BOOLEAN));
        }

        if($input->has('disable_build')) {
            $this->setDisableBuild(filter_var($input->get('disable_build'), FILTER_VALIDATE_BOOLEAN));
        }

        if($packageId !== null) {
            if($packageCollection->has($packageId)) {
                $packageCollection->put($packageId, $package);
            } else {
                throw new PackageNotFoundException('Package with ID "' . $packageId . '" does not exist.');
            }
        } else {
            $packageId = static::nameToId($package->getName());

            $packageCollection->put($packageId, $package);
        }

        $buildContext = new PublicRepository();
        $buildContext->setItemId($packageId)->setItemName($package->getName());

        $this->configBuilder->setBuildContext($buildContext);

        return $packageCollection;
    }

    /**
     * @param \App\Satis\Collections\RepositoryCollection $repositoryCollection
     * @param string $repositoryId
     * @return \App\Satis\Collections\RepositoryCollection
     * @throws \App\Satis\Exceptions\RepositoryNotFoundException
     */
    protected function _deleteRepository(RepositoryCollection $repositoryCollection, $repositoryId) {
        if($repositoryCollection->has($repositoryId)) {
            $repositoryCollection->forget($repositoryId);
        } else {
            throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
        }

        $this->setDisableBuild(true);

        return $repositoryCollection;
    }

    /**
     * @param \App\Satis\Collections\PackageCollection $packageCollection
     * @param string $packageId
     * @return \App\Satis\Collections\RepositoryCollection
     * @throws \App\Satis\Exceptions\RepositoryNotFoundException
     */
    protected function _deletePackage(PackageCollection $packageCollection, $packageId) {
        if($packageCollection->has($packageId)) {
            $packageCollection->forget($packageId);
        } else {
            throw new RepositoryNotFoundException('Package with ID "' . $packageId . '" does not exist.');
        }

        $this->setDisableBuild(true);

        return $packageCollection;
    }

    /**
     * @param string $repositoryUrl
     * @return string
     */
    public static function nameToId($repositoryUrl) {
        return md5($repositoryUrl);
    }

    /**
     * @param \App\Satis\ConfigPersister $configPersister
     * @param ConfigBuilder $configBuilder
     * @param \JMS\Serializer\Serializer $serializer
     */
    public function __construct(ConfigPersister $configPersister, ConfigBuilder $configBuilder,
                                Serializer $serializer
    ) {
        $this->configPersister = $configPersister;
        $this->configBuilder = $configBuilder;
        $this->serializer = $serializer;
    }

    /**
     * @param string $method
     * @param array $arguments
     *
     * @return mixed
     *
     * @throws \Exception
     */
    public function __call($method, array $arguments) {
        $method = $this->hiddenEntityPrefix . $method;

        if(method_exists($this, $method)) {
            $logger = $this->getLogger();

            $logger->info(str_repeat('=', 30));
            $logger->info('Request => ' . \Request::method() . ' (' . $method .') => '. json_encode($arguments));

            $type = str_plural(last(explode('_', snake_case($method))), 2);

            array_unshift($arguments, $this->satis->{'get' . ucfirst($type)}());


            try {
                $items = call_user_func_array([$this, $method], $arguments);
            } catch(\Exception $e) {
                $logger->info('Response => ' . $e->getMessage());
                $logger->info(str_repeat('=', 30));

                throw $e;
            }

            $this->satis->{'set' . $type}($items);


            return $this->save();
        } else {
            throw new BadMethodCallException('Called method "' . $method . '" does not exist.');
        }
    }

    /**
     * @param string $property
     * @return mixed
     */
    public function __get($property) {
        $property = $this->hiddenEntityPrefix.$property;

        if(isset($this->lazyLoadedProperties[$property])) {
            $this->{$property} = $this->{$this->lazyLoadedProperties[$property]}();
            
            unset($this->lazyLoadedProperties[$property]);
        }

        return $this->{$property};
    }

    /**
     * @param boolean $disableBuild
     * @return ConfigManager
     */
    public function setDisableBuild($disableBuild) {
        $this->disableBuild = $disableBuild;

        return $this;
    }

    /**
     * @param string|null $repositoryId
     * @return string
     * @throws \App\Satis\Exceptions\RepositoryNotFoundException
     */
    public function getRepositories($repositoryId = null) {

        $repositoryCollection = $this->satis->getRepositories();


        if($repositoryId !== null) {
            if(!$repositoryCollection->has($repositoryId)) {
                throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
            }

            $output = $repositoryCollection->get($repositoryId);
        } else {
            $output = $repositoryCollection;
        }

        return $this->serializer->serialize($output, 'json');
    }

	/**
     * @param string|null $packageId
     * @return string
     * @throws \App\Satis\Exceptions\PackageNotFoundException
     */
    public function getPackages($packageId = null) {

        $packageCollection = $this->satis->getPackages();


        if($packageId !== null) {
            if(!$packageCollection->has($packageId)) {
                throw new PackageNotFoundException('Package with ID "' . $packageId . '" does not exist.');
            }

            $output = $packageCollection->get($packageId);
        } else {
            $output = $packageCollection;
        }

        return $this->serializer->serialize($output, 'json');
    }

	/**
     * @param \App\Satis\BuildContext $buildContext
     */
    public function forceBuild(BuildContext $buildContext) {
        if($buildContext->getItemName() !== null) {
            $buildContext->setItemId(self::nameToId($buildContext->getItemName()));
        }

        $this->configBuilder
            ->setBuildContext($buildContext)
            ->build();
    }

    /**
     * @return bool
     */
    public function isBuilding() {
        return $this->configPersister->isLocked();
    }

    /**
     * @return \App\Satis\Model\Config
     */
    public function getDefinition() {
        return $this->satis;

    }

    /**
     * @return string
     */
    public function save() {
        $satisConfig = $this->serializer->serialize($this->satis, 'json');


        $this->configPersister->updateWith($satisConfig);

        if($this->disableBuild === false) {
            return $this->configBuilder->build();
        }

        return true;
    }
}


1		<?php
2
3		namespace App\Satis;
4
5		use App\Satis\Collections\PackageCollection;
6		use App\Satis\Context\PrivateRepository;
7		use App\Satis\Context\PublicRepository;
8		use App\Satis\Exceptions\PackageNotFoundException;
9		use App\Satis\Model\Package;
10		use BadMethodCallException;
11		use App\Satis\Collections\RepositoryCollection;
12		use App\Satis\Exceptions\RepositoryNotFoundException;
13		use App\Satis\Model\Repository;
14		use Illuminate\Support\Collection;
15		use JMS\Serializer\Serializer;
16		use Monolog\Formatter\LineFormatter;
17		use Monolog\Handler\StreamHandler;
18		use Monolog\Logger;
19
20		/**
21		* @method string addOrUpdateRepository($repositoryId, Collection $repositoryData);
22		* @method string addOrUpdatePackage($packageId, Collection $packageData);
23		* @method void deleteRepository($repositoryId);
24		* @method void deletePackage($packageId);
25		* @property \App\Satis\Model\Config $satis;
26		*
27		* @author Lukas Homza <[email protected]>
28		*/
29		class ConfigManager {
30		const BUILD_PRIVATE = 1;
31		const BUILD_PUBLIC = 2;
32
33		/** @var \App\Satis\ConfigPersister $configPersister */
34		protected $configPersister;
35		/** @var \App\Satis\ConfigBuilder $configBuilder */
36		protected $configBuilder;
37		/** @var \JMS\Serializer\Serializer $serializer */
38		protected $serializer;
39		/** @var \App\Satis\Model\Config $_satis */
40		protected $_satis;
41
42		/** @var bool $disableBuild */
43		protected $disableBuild = false;
44
45		/** @var string */
46		protected $hiddenEntityPrefix = '_';
47
48		/** @var array */
49		protected $lazyLoadedProperties = [
50		'_satis' => 'loadSatisConfig'
51		];
52
53		/**
54		* @return \App\Satis\Model\Config
55		* @throws \Illuminate\Contracts\Filesystem\FileNotFoundException
56		*/
57		protected function loadSatisConfig() {
58		return $this->serializer->deserialize(
		0 ignored issues – show Bug Best Practice introduced 2016-01-28 23:09 UTC by Report Bug Copy Issue Report The return type of `return $this->serializer...odel\\Config', 'json');` (`object\|array\|integer\|double\|string\|boolean`) is incompatible with the return type documented by `App\Satis\ConfigManager::loadSatisConfig` of type `App\Satis\Model\Config`. If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design. Let’s take a look at an example: class Author { private $name; public function __construct($name) { $this->name = $name; } public function getName() { return $this->name; } } abstract class Post { public function getAuthor() { return 'Johannes'; } } class BlogPost extends Post { public function getAuthor() { return new Author('Johannes'); } } class ForumPost extends Post { /* ... */ } function my_function(Post $post) { echo strtoupper($post->getAuthor()); } Our function `my_function` expects a `Post` object, and outputs the author of the post. The base class `Post` returns a simple string and outputting a simple string will work just fine. However, the child class `BlogPost` which is a sub-type of `Post` instead decided to return an `object`, and is therefore violating the SOLID principles. If a `BlogPost` were passed to `my_function`, PHP would not complain, but ultimately fail when executing the `strtoupper` call in its body. Loading history...
59		$this->configPersister->load(),
60		'App\Satis\Model\Config',
61		'json'
62		);
63		}
64
65		/**
66		* @return \Monolog\Logger
67		*/
68		protected function getLogger() {
69		$handler = new StreamHandler(storage_path('logs/api_request.log'), Logger::DEBUG);
70		$handler->setFormatter(new LineFormatter(null, null, true, true));
71
72		$logger = new Logger('ApiRequestLog');
73		$logger->pushHandler($handler);
74
75		return $logger;
76		}
77
78		/**
79		* @param \App\Satis\Collections\RepositoryCollection $repositoryCollection
80		* @param string $repositoryId
81		* @param \Illuminate\Support\Collection $input
82		* @return \App\Satis\Collections\RepositoryCollection
83		* @throws \App\Satis\Exceptions\RepositoryNotFoundException
84		*/
85		protected function _addOrUpdateRepository(RepositoryCollection $repositoryCollection, $repositoryId,
86		Collection $input
87		) {
88		$repository = new Repository();
89		$repository->setUrl($input->get('url'));
90		$repository->setType($input->get('type', config('satis.default_repository_type')));
91
92		if($input->has('async_mode')) {
93		$this->configBuilder->setAsyncMode(filter_var($input->get('async_mode'), FILTER_VALIDATE_BOOLEAN));
94		}
95
96		if($input->has('disable_build')) {
97		$this->setDisableBuild(filter_var($input->get('disable_build'), FILTER_VALIDATE_BOOLEAN));
98		}
99
100		if($repositoryId !== null) {
101		if($repositoryCollection->has($repositoryId)) {
102		$repositoryCollection->put($repositoryId, $repository);
103		} else {
104		throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
105		}
106		} else {
107		$repositoryId = static::nameToId($repository->getUrl());
108
109		$repositoryCollection->put($repositoryId, $repository);
110		}
111
112		$buildContext = new PrivateRepository();
113		$buildContext->setItemId($repositoryId)->setItemName($repository->getUrl());
114
115		$this->configBuilder->setBuildContext($buildContext);
116
117		return $repositoryCollection;
118		}
119
120		/**
121		* @param \App\Satis\Collections\PackageCollection $packageCollection
122		* @param string $packageId
123		* @param \Illuminate\Support\Collection $input
124		* @return \App\Satis\Collections\RepositoryCollection
125		* @throws \App\Satis\Exceptions\PackageNotFoundException
126		*/
127		protected function _addOrUpdatePackage(PackageCollection $packageCollection, $packageId,
128		Collection $input
129		) {
130		$package = new Package();
131		$package->setName($input->get('name'));
132
133		if($input->has('async_mode')) {
134		$this->configBuilder->setAsyncMode(filter_var($input->get('async_mode'), FILTER_VALIDATE_BOOLEAN));
135		}
136
137		if($input->has('disable_build')) {
138		$this->setDisableBuild(filter_var($input->get('disable_build'), FILTER_VALIDATE_BOOLEAN));
139		}
140
141		if($packageId !== null) {
142		if($packageCollection->has($packageId)) {
143		$packageCollection->put($packageId, $package);
144		} else {
145		throw new PackageNotFoundException('Package with ID "' . $packageId . '" does not exist.');
146		}
147		} else {
148		$packageId = static::nameToId($package->getName());
149
150		$packageCollection->put($packageId, $package);
151		}
152
153		$buildContext = new PublicRepository();
154		$buildContext->setItemId($packageId)->setItemName($package->getName());
155
156		$this->configBuilder->setBuildContext($buildContext);
157
158		return $packageCollection;
159		}
160
161		/**
162		* @param \App\Satis\Collections\RepositoryCollection $repositoryCollection
163		* @param string $repositoryId
164		* @return \App\Satis\Collections\RepositoryCollection
165		* @throws \App\Satis\Exceptions\RepositoryNotFoundException
166		*/
167		protected function _deleteRepository(RepositoryCollection $repositoryCollection, $repositoryId) {
168		if($repositoryCollection->has($repositoryId)) {
169		$repositoryCollection->forget($repositoryId);
170		} else {
171		throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
172		}
173
174		$this->setDisableBuild(true);
175
176		return $repositoryCollection;
177		}
178
179		/**
180		* @param \App\Satis\Collections\PackageCollection $packageCollection
181		* @param string $packageId
182		* @return \App\Satis\Collections\RepositoryCollection
183		* @throws \App\Satis\Exceptions\RepositoryNotFoundException
184		*/
185		protected function _deletePackage(PackageCollection $packageCollection, $packageId) {
186		if($packageCollection->has($packageId)) {
187		$packageCollection->forget($packageId);
188		} else {
189		throw new RepositoryNotFoundException('Package with ID "' . $packageId . '" does not exist.');
190		}
191
192		$this->setDisableBuild(true);
193
194		return $packageCollection;
195		}
196
197		/**
198		* @param string $repositoryUrl
199		* @return string
200		*/
201		public static function nameToId($repositoryUrl) {
202		return md5($repositoryUrl);
203		}
204
205		/**
206		* @param \App\Satis\ConfigPersister $configPersister
207		* @param ConfigBuilder $configBuilder
208		* @param \JMS\Serializer\Serializer $serializer
209		*/
210		public function __construct(ConfigPersister $configPersister, ConfigBuilder $configBuilder,
211		Serializer $serializer
212		) {
213		$this->configPersister = $configPersister;
214		$this->configBuilder = $configBuilder;
215		$this->serializer = $serializer;
216		}
217
218		/**
219		* @param string $method
220		* @param array $arguments
221		*
222		* @return mixed
223		*
224		* @throws \Exception
225		*/
226		public function __call($method, array $arguments) {
227		$method = $this->hiddenEntityPrefix . $method;
228
229		if(method_exists($this, $method)) {
230		$logger = $this->getLogger();
231
232		$logger->info(str_repeat('=', 30));
233		$logger->info('Request => ' . \Request::method() . ' (' . $method .') => '. json_encode($arguments));
234
235		$type = str_plural(last(explode('_', snake_case($method))), 2);
236
237		array_unshift($arguments, $this->satis->{'get' . ucfirst($type)}());
		0 ignored issues – show Bug introduced 2016-01-28 23:09 UTC by Report Bug Copy Issue Report The property `satis` does not seem to exist. Did you mean `_satis`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
238
239		try {
240		$items = call_user_func_array([$this, $method], $arguments);
241		} catch(\Exception $e) {
242		$logger->info('Response => ' . $e->getMessage());
243		$logger->info(str_repeat('=', 30));
244
245		throw $e;
246		}
247
248		$this->satis->{'set' . $type}($items);
		0 ignored issues – show Bug introduced 2016-01-28 23:09 UTC by Report Bug Copy Issue Report The property `satis` does not seem to exist. Did you mean `_satis`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
249
250		return $this->save();
251		} else {
252		throw new BadMethodCallException('Called method "' . $method . '" does not exist.');
253		}
254		}
255
256		/**
257		* @param string $property
258		* @return mixed
259		*/
260		public function __get($property) {
261		$property = $this->hiddenEntityPrefix.$property;
262
263		if(isset($this->lazyLoadedProperties[$property])) {
264		$this->{$property} = $this->{$this->lazyLoadedProperties[$property]}();
265
266		unset($this->lazyLoadedProperties[$property]);
267		}
268
269		return $this->{$property};
270		}
271
272		/**
273		* @param boolean $disableBuild
274		* @return ConfigManager
275		*/
276		public function setDisableBuild($disableBuild) {
277		$this->disableBuild = $disableBuild;
278
279		return $this;
280		}
281
282		/**
283		* @param string\|null $repositoryId
284		* @return string
285		* @throws \App\Satis\Exceptions\RepositoryNotFoundException
286		*/
287	View Code Duplication	public function getRepositories($repositoryId = null) {
		0 ignored issues – show Duplication introduced 2018-06-20 10:54 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
288		$repositoryCollection = $this->satis->getRepositories();
		0 ignored issues – show Bug introduced 2016-01-28 23:09 UTC by Report Bug Copy Issue Report The property `satis` does not seem to exist. Did you mean `_satis`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
289
290		if($repositoryId !== null) {
291		if(!$repositoryCollection->has($repositoryId)) {
292		throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
293		}
294
295		$output = $repositoryCollection->get($repositoryId);
296		} else {
297		$output = $repositoryCollection;
298		}
299
300		return $this->serializer->serialize($output, 'json');
301		}
302
303		/**
304		* @param string\|null $packageId
305		* @return string
306		* @throws \App\Satis\Exceptions\PackageNotFoundException
307		*/
308	View Code Duplication	public function getPackages($packageId = null) {
		0 ignored issues – show Duplication introduced 2018-06-20 10:54 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
309		$packageCollection = $this->satis->getPackages();
		0 ignored issues – show Bug introduced 2016-01-28 23:09 UTC by Report Bug Copy Issue Report The property `satis` does not seem to exist. Did you mean `_satis`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
310
311		if($packageId !== null) {
312		if(!$packageCollection->has($packageId)) {
313		throw new PackageNotFoundException('Package with ID "' . $packageId . '" does not exist.');
314		}
315
316		$output = $packageCollection->get($packageId);
317		} else {
318		$output = $packageCollection;
319		}
320
321		return $this->serializer->serialize($output, 'json');
322		}
323
324		/**
325		* @param \App\Satis\BuildContext $buildContext
326		*/
327		public function forceBuild(BuildContext $buildContext) {
328		if($buildContext->getItemName() !== null) {
329		$buildContext->setItemId(self::nameToId($buildContext->getItemName()));
330		}
331
332		$this->configBuilder
333		->setBuildContext($buildContext)
334		->build();
335		}
336
337		/**
338		* @return bool
339		*/
340		public function isBuilding() {
341		return $this->configPersister->isLocked();
342		}
343
344		/**
345		* @return \App\Satis\Model\Config
346		*/
347		public function getDefinition() {
348		return $this->satis;
		0 ignored issues – show Bug introduced 2016-01-28 23:09 UTC by Report Bug Copy Issue Report The property `satis` does not seem to exist. Did you mean `_satis`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
349		}
350
351		/**
352		* @return string
353		*/
354		public function save() {
355		$satisConfig = $this->serializer->serialize($this->satis, 'json');
		0 ignored issues – show Bug introduced 2016-01-28 23:09 UTC by Report Bug Copy Issue Report The property `satis` does not seem to exist. Did you mean `_satis`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
356
357		$this->configPersister->updateWith($satisConfig);
358
359		if($this->disableBuild === false) {
360		return $this->configBuilder->build();
361		}
362
363		return true;
364		}
365		}
366

realshadow / satis-control-panel

ConfigManager A last analyzed 2018-06-29 09:03 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

17 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like

ConfigManager A
last analyzed 2018-06-29 09:03 UTC