StingerSoft\PlatformBundle\DataFixtures\AbstractStingerFixture

Complexity

Total Complexity

9

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	load()	0	14	2
A	reportMemoryUsage()	0	3	1
	import()	0	1	?
A	skipMe()	0	3	1
A	getLogger()	0	8	4
A	getDependencies()	0	2	1

<?php

/*
 * This file is part of the Stinger Soft Platform package.
 *
 * (c) Oliver Kotte <oliver.kotte@stinger-soft.net>
 * (c) Florian Meyer <florian.meyer@stinger-soft.net>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace StingerSoft\PlatformBundle\DataFixtures;

use Doctrine\Common\DataFixtures\AbstractFixture;
use Doctrine\Common\DataFixtures\DependentFixtureInterface;
use Doctrine\Common\Persistence\ObjectManager;
use Symfony\Component\DependencyInjection\ContainerAwareTrait;
use Psr\Log\LoggerInterface;
use Psr\Log\NullLogger;

abstract class AbstractStingerFixture extends AbstractFixture implements DependentFixtureInterface {
	
	use ContainerAwareTrait;

	/**
	 *
	 * @var ObjectManager
	 */
	protected $manager;

	/**
	 *
	 * @var LoggerInterface
	 */
	private $logger;

	public final function load(ObjectManager $manager) {

As per PSR2, final should precede the visibility keyword.

		$this->manager = $manager;
		if(!$this->skipMe()) {
			$this->info("Starting import of fixtures ...");

The method info() does not seem to exist on object<StingerSoft\PlatformBundle\DataFixtures\AbstractStingerFixture>.

			$this->reportMemoryUsage();
// 			$startTime = microtime(true);

50% of this comment could be valid code. Did you maybe forget this after debugging?

			$this->import();
// 			$endTime = microtime(true);

59% of this comment could be valid code. Did you maybe forget this after debugging?

// 			$this->info("Finished importing fixtures in " . $this->formatTime($startTime, $endTime));
			$this->reportMemoryUsage();
		} else {
			$this->info('Skipping import of fixtures !');

The method info() does not seem to exist on object<StingerSoft\PlatformBundle\DataFixtures\AbstractStingerFixture>.

		}
	}

	protected final function reportMemoryUsage() {

As per PSR2, final should precede the visibility keyword.

		$this->info("Currently using %s of RAM", $this->getMemoryUsage());

The method getMemoryUsage() does not seem to exist on object<StingerSoft\PlatformBundle\DataFixtures\AbstractStingerFixture>.

The method info() does not seem to exist on object<StingerSoft\PlatformBundle\DataFixtures\AbstractStingerFixture>.

	}

	/**
	 * Load data fixtures
	 */
	protected abstract function import();

The abstract declaration must precede the visibility declaration

	/**
	 * Decides whether this fixture should be skipped or not.
	 *
	 * @return bool
	 */
	protected function skipMe() {
		return false;
	}
	

	/**
	 *
	 * @return LoggerInterface|null

Should the return type not be object?

	 */
	protected function getLogger() {
		if(!$this->logger && $this->container && $this->container->has('logger')) {
			$this->logger = $this->container->get('logger');
		} else {
			$this->logger = new NullLogger();
		}
		return $this->logger;
	}

	/**
	 */
	protected function getDependencies() {
	}
}