Cv\Controller\IndexController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	86
Duplicated Lines	37.21 %

Coupling/Cohesion

Components	2
Dependencies	11

Importance

Changes	4
Bugs	3	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	attachDefaultListeners()	9	9	1
B	indexAction()	23	55	9

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

/** ActionController of Core */
namespace Cv\Controller;

use Cv\Form\SearchForm;
use Zend\Mvc\Controller\AbstractActionController;
use Zend\View\Model\ViewModel;
use Zend\Session\Container as Session;

/**
 * Main Action Controller for the application.
 * Responsible for displaying the home site.
 *
 */
class IndexController extends AbstractActionController
{
    /**
     * @var SearchForm
     */
    protected $searchForm;

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

    /**
     * attaches further Listeners for generating / processing the output
     * @return $this
     */
    public function attachDefaultListeners()

This method seems to be duplicated in your project.

    {
        parent::attachDefaultListeners();
        $serviceLocator  = $this->serviceLocator;
        $defaultServices = $serviceLocator->get('DefaultListeners');
        $events          = $this->getEventManager();
        $events->attach($defaultServices);

$defaultServices is of type object|array, but the function expects a string.

        return $this;
    }

    /**
     * Home site
     *
     */
    public function indexAction()
    {
        /* @var \Zend\Http\Request $request */
        $request = $this->getRequest();
        $params = $request->getQuery();
        $jsonFormat = 'json' == $request->getQuery()->get('format');
        $event = $this->getEvent();
        $routeMatch = $event->getRouteMatch();
        $matchedRouteName = $routeMatch->getMatchedRouteName();
        $url = $this->url()->fromRoute($matchedRouteName, array(), array('force_canonical' => true));


        if (!$jsonFormat && !$request->isXmlHttpRequest()) {

This code seems to be duplicated across your project.

            $session = new Session('Cv\Index');
            $sessionKey = $this->auth()->isLoggedIn() ? 'userParams' : 'guestParams';

The method auth does not exist on object<Cv\Controller\IndexController>? Since you implemented __call, maybe consider adding a @method annotation.

            $sessionParams = $session[$sessionKey];
            if ($sessionParams) {
                foreach ($sessionParams as $key => $value) {
                    $params->set($key, $params->get($key, $value));
                }
            }
            $session[$sessionKey] = $params->toArray();

            $this->searchForm->bind($params);
        }

        $params = $params->get('params', []);

        if (isset($params['l']['data']) &&

This code seems to be duplicated across your project.

            isset($params['l']['name']) &&
            !empty($params['l']['name'])
        ) {
            /* @var \Geo\Form\GeoText $geoText */
            $geoText = $this->searchForm->get('params')->get('l');

It seems like you code against a concrete implementation and not the interface Zend\Form\ElementInterface as the method get() does only exist in the following implementations of said interface: Applications\Form\Apply, Applications\Form\Attributes, Applications\Form\Base, Applications\Form\BaseFieldset, Applications\Form\CarbonCopyFieldset, Applications\Form\CommentForm, Applications\Form\ContactContainer, Applications\Form\Facts, Applications\Form\FactsFieldset, Applications\Form\FilterApplication, Applications\Form\Mail, Applications\Form\SettingsFieldset, Auth\Form\ForgotPassword, Auth\Form\Group, Auth\Form\GroupFieldset, Auth\Form\GroupUsersCollection, Auth\Form\Login, Auth\Form\Register, Auth\Form\SocialProfiles, Auth\Form\SocialProfilesFieldset, Auth\Form\UserBase, Auth\Form\UserBaseFieldset, Auth\Form\UserInfo, Auth\Form\UserInfoContainer, Auth\Form\UserInfoFieldset, Auth\Form\UserPassword, Auth\Form\UserPasswordFieldset, Auth\Form\UserProfileContainer, Auth\Form\UserStatus, Auth\Form\UserStatusContainer, Auth\Form\UserStatusFieldset, Core\Form\BaseForm, Core\Form\ButtonsFieldset, Core\Form\CollectionContainer, Core\Form\Container, Core\Form\DefaultButtonsFieldset, Core\Form\Form, Core\Form\FormSubmitButtonsFieldset, Core\Form\ListFilterButtonsFieldset, Core\Form\LocalizationSettingsFieldset, Core\Form\PermissionsCollection, Core\Form\PermissionsFieldset, Core\Form\RatingFieldset, Core\Form\SummaryForm, Core\Form\SummaryFormButtonsFieldset, Core\Form\TextSearchForm, Core\Form\TextSearchFormButtonsFieldset, Core\Form\TextSearchFormFieldset, Core\Form\ViewPartialProviderAbstract, Core\Form\WizardContainer, Cv\Form\CvContainer, Cv\Form\EducationFieldset, Cv\Form\EducationForm, Cv\Form\EmploymentFieldset, Cv\Form\EmploymentForm, Cv\Form\LanguageSkillFieldset, Cv\Form\LanguageSkillForm, Cv\Form\NativeLanguageFieldset, Cv\Form\NativeLanguageForm, Cv\Form\PreferredJobFieldset, Cv\Form\PreferredJobForm, Cv\Form\SearchForm, Cv\Form\SearchFormFieldset, Cv\Form\SkillFieldset, Cv\Form\SkillForm, Install\Form\Installation, Jobs\Form\AdminJobEdit, Jobs\Form\AdminSearchForm, Jobs\Form\AdminSearchFormElementsFieldset, Jobs\Form\AtsMode, Jobs\Form\AtsModeFieldset, Jobs\Form\Base, Jobs\Form\BaseFieldset, Jobs\Form\CompanyName, Jobs\Form\CompanyNameFieldset, Jobs\Form\Import, Jobs\Form\ImportFieldset, Jobs\Form\Job, Jobs\Form\JobDescription, Jobs\Form\JobDescriptionBenefits, Jobs\Form\JobDescriptionDescription, Jobs\Form\JobDescriptionFieldset, Jobs\Form\JobDescriptionQualifications, Jobs\Form\JobDescriptionRequirements, Jobs\Form\JobDescriptionTemplate, Jobs\Form\JobDescriptionTitle, Jobs\Form\ListFilter, Jobs\Form\ListFilterAdmin, Jobs\Form\ListFilterAdminFieldset, Jobs\Form\ListFilterBaseFieldset, Jobs\Form\ListFilterLocation, Jobs\Form\ListFilterLocationFieldset, Jobs\Form\ListFilterPersonal, Jobs\Form\ListFilterPersonalFieldset, Jobs\Form\Multipost, Jobs\Form\MultipostButtonFieldset, Jobs\Form\MultipostFieldset, Jobs\Form\Preview, Jobs\Form\PreviewFieldset, Jobs\Form\TemplateLabelBenefits, Jobs\Form\TemplateLabelQualifications, Jobs\Form\TemplateLabelRequirements, Orders\Form\InvoiceAddress, Orders\Form\InvoiceAddressFieldset, Orders\Form\InvoiceAddressSettingsFieldset, Organizations\Form\EmployeeFieldset, Organizations\Form\Employees, Organizations\Form\EmployeesFieldset, Organizations\Form\Organizations, Organizations\Form\OrganizationsContactFieldset, Organizations\Form\OrganizationsContactForm, Organizations\Form\OrganizationsDescriptionFieldset, Organizations\Form\OrganizationsDescriptionForm, Organizations\Form\OrganizationsFieldset, Organizations\Form\OrganizationsNameFieldset, Organizations\Form\OrganizationsNameForm, Organizations\Form\WorkflowSettings, Organizations\Form\WorkflowSettingsFieldset, Settings\Form\AbstractSettingsForm, Settings\Form\DisableElementsCapableFormSettingsFieldset, Settings\Form\FormAbstract, Settings\Form\Settings, Settings\Form\SettingsFieldset, Zend\Form\Element\Collection, Zend\Form\Fieldset, Zend\Form\Form, Zend\Form\InputFilterProviderFieldset.

            $geoText->setValue($params['l']);
            $params['location'] = $geoText->getValue('entity');
        }


        $this->searchForm->setAttribute('action', $url);
        $paginator = $this->paginator('Cv/Paginator', $params);

The method paginator does not exist on object<Cv\Controller\IndexController>? Since you implemented __call, maybe consider adding a @method annotation.

        $options = $this->searchForm->getOptions();
        $options['showButtons'] = false;
        $this->searchForm->setOptions($options);

        $return = array(
            'resumes' => $paginator,
            'filterForm' => $this->searchForm
        );
        $model = new ViewModel($return);

        return $model;
    }
}