Core\View\Helper\SocialButtons

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	__invoke()	0	10	4

<?php
/**
 * YAWIK
 *
 * @filesource
 * @copyright (c) 2013-2015 Cross Solution (http://cross-solution.de)
 * @license   MIT
 */

namespace Core\View\Helper;

use Zend\View\Helper\AbstractHelper;
use Auth\Options\ModuleOptions;

class SocialButtons extends AbstractHelper
{
    /**
     * @var ModuleOptions $options;
     */
    protected $options;

    /**
     * @var array $options;
     */
    protected $config;

    /**
     * @param $options ModuleOptions
     * @param $config array
     */
    public function __construct($options,$config)
    {
        $this->options = $options;
        $this->config = $config;
        return $this;
    }

    /**
     * returns the advertised title
     *
     * @return array
     */
    public function __invoke()
    {
        $SocialNetworksEnabled=[];

$SocialNetworksEnabled does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        foreach($this->config['hybridauth'] as $key => $val) {
            if ($val['enabled'] and in_array(strtolower($key), $this->options->getEnableLogins())){

Using logical operators such as and instead of && is generally not recommended.

                $SocialNetworksEnabled[strtolower($key)] = $key;

$SocialNetworksEnabled does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            }
        }
        return $SocialNetworksEnabled;

$SocialNetworksEnabled does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
}