SettingsManager::getAllProfiles() - Code Metrics - Inspection of "Heavy ammo upgrade" - ongr-io/SettingsBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#170)

by Simonas

created 2016-07-19 14:55 UTC

SettingsManager::getAllProfiles() B

↳ Parent: SettingsManager

Complexity

Conditions	4
Paths	5

Size

Total Lines	32
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	32
rs	8.5806
cc	4
eloc	20
nc	5
nop	0

<?php

/*
 * This file is part of the ONGR package.
 *
 * (c) NFQ Technologies UAB <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace ONGR\SettingsBundle\Service;

use ONGR\ElasticsearchBundle\Result\Aggregation\AggregationValue;
use ONGR\ElasticsearchDSL\Aggregation\TermsAggregation;
use ONGR\ElasticsearchDSL\Aggregation\TopHitsAggregation;
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
use ONGR\ElasticsearchBundle\Service\Repository;
use ONGR\ElasticsearchBundle\Service\Manager;
use ONGR\SettingsBundle\Document\Setting;

/**
 * Class SettingsManager responsible for managing settings actions.
 */
class SettingsManager
{
    /**
     * Symfony event dispatcher.
     *
     * @var EventDispatcherInterface
     */
    private $eventDispatcher;

    /**
     * Elasticsearch manager which handles setting repository.
     *
     * @var Manager
     */
    private $manager;

    /**
     * Settings repository.
     *
     * @var Repository
     */
    private $repo;

    /**
     * Active profiles setting name.
     *
     * @var string
     */
    private $activeProfilesSetting;

    /**
     * @param Repository               $repo
     * @param EventDispatcherInterface $eventDispatcher
     */
    public function __construct(
        $repo,
        EventDispatcherInterface $eventDispatcher
    ) {
        $this->repo = $repo;
        $this->manager = $repo->getManager();
        $this->eventDispatcher = $eventDispatcher;
    }

    /**
     * @return string
     */
    public function getActiveProfilesSetting()
    {
        return $this->activeProfilesSetting;
    }

    /**
     * @param string $activeProfilesSetting
     */
    public function setActiveProfilesSetting($activeProfilesSetting)
    {
        $this->activeProfilesSetting = $activeProfilesSetting;
    }

    /**
     * Overwrites setting with given name.
     *
     * @param string       $name
     * @param array        $data
     * @param bool         $force
     *
     * @return Setting
     */
    public function create($name, array $data = [], $force = false)
    {
        $existingSetting = $this->get($name);
        if ($existingSetting && !$force) {
            return false;
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());
}
        }

        if ($existingSetting && $force) {
            /** @var Setting $setting */
            $setting = $existingSetting;
        } else {
            $settingClass = $this->repo->getClassName();
            /** @var Setting $setting */
            $setting = new $settingClass();
        }

        $setting->setName($name);
        
        foreach ($data as $key => $value) {
            $setting->{'set'.ucfirst($key)}($value);
        }

        $this->manager->persist($setting);
        $this->manager->commit();

        return $setting;
    }

    /**
     * Overwrites setting with given name.
     *
     * @param string      $name
     * @param array       $data
     *
     * @return Setting
     */
    public function update($name, $data)
    {
        return $this->create($name, $data, true);
    }

    /**
     * Deletes a setting.
     *
     * @param string    $name
     */
    public function delete($name)
    {
        $setting = $this->repo->findOneBy(['name' => $name]);
        $this->repo->remove($setting->getId());
    }

    /**
     * Returns setting value.
     *
     * @param string $name
     * @param mixed  $default
     *
     * @return Setting
     */
    public function get($name, $default = null)
    {
        $setting = $this->repo->findOneBy(['name' => $name]);

        if ($setting) {
            return $setting;
        } else {
            return $default;
        }
    }

    public function getAllProfiles()
    {
        $profiles = [];

        $search = $this->repo->createSearch();
        $topHitsAgg = new TopHitsAggregation('documents', 10000);
        $termAgg = new TermsAggregation('profiles', 'profile');
        $termAgg->addAggregation($topHitsAgg);
        $search->addAggregation($termAgg);

        $result = $this->repo->execute($search);

        /** @var Setting $activeProfiles */
        $activeProfiles = $this->get($this->activeProfilesSetting, []);

        /** @var AggregationValue $agg */
        foreach ($result->getAggregation('profiles') as $agg) {
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            $settings = [];
            $docs = $agg->getAggregation('documents');
            foreach ($docs['hits']['hits'] as $doc) {
                $settings[] = $doc['_source']['name'];
            }
            $name = $agg->getValue('key');
            $profiles[] = [
                'active' => $activeProfiles ? in_array($agg->getValue('key'), $activeProfiles->getValue()) : false,
                'name' => $name,
                'settings' => implode(', ', $settings),
            ];
        }

        return $profiles;
    }

    public function getAllProfilesNameList($onlyActive = false)
    {
        $profiles = [];
        $allProfiles = $this->getAllProfiles();

        foreach ($allProfiles as $profile) {
            if ($onlyActive and !$profile['active']) {
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
                continue;
            }

            $profiles[] = $profile['name'];
        }

        return $profiles;
    }
}


Pull Request — master (#170)

SettingsManager::getAllProfiles() B

Complexity

Size

Duplication

Importance

Available Fixes

Logical Operators are used for Control-Flow

ongr-io / SettingsBundle

Pull Request — master (#170)

SettingsManager::getAllProfiles() B

Complexity

Size

Duplication

Importance

Available Fixes

Logical Operators are used for Control-Flow

Duplication Side-by-Side

Filter issues like