UsersController::editAction() - Code Metrics - Inspection of "Issue #512" - cross-solution/YAWIK - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Pull Request — develop (#521)

unknown
created 2018-11-22 04:24 UTC
UsersController::editAction() B

↳ Parent: UsersController
Complexity

Conditions	10
Paths	20
Size

Total Lines	71
Code Lines	45
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
eloc	45
dl	0
loc	71
rs	7.3333
c	0
b	0
f	0
cc	10
nc	20
nop	0
How to fix Long Method Complexity

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

/** Auth controller */
namespace Auth\Controller;

use Zend\Mvc\Controller\AbstractActionController;
use Zend\View\Model\ViewModel;
use Zend\View\Model\JsonModel;
use Auth\Repository\User as UserRepository;
use Core\Form\SummaryFormInterface;

/**
 * List registered users
 *
 * @author Carsten Bleek <[email protected]>
 * @author Mathias Gelhausen <[email protected]>
 * @author Anthonius Munthi <[email protected]>
 *
 * @method array|\Core\Controller\Plugin\PaginationBuilder pagination($stack = null, $returnResult = true)
 */
class UsersController extends AbstractActionController
{

    /**
     * @var UserRepository
     */
    protected $userRepository;

    protected $formManager;
    
    protected $viewHelper;
    
    /**
     * @param UserRepository $userRepository
     */
    public function __construct(UserRepository $userRepository,$formManager,$viewHelper)
    {
        $this->userRepository = $userRepository;
        $this->formManager = $formManager;
        $this->viewHelper = $viewHelper;
    }
    
    /**
     * List users
     *
     * @return array|\Zend\Http\Response|ViewModel
     */
    public function listAction()
    {
        return $this->pagination([

            'paginator' => ['Auth/User', 'as' => 'users'],
            'form' => [
                'Core/Search',
                [
                    'text_name' => 'text',
                    'text_placeholder' => /*@translate*/ 'Type name, email address, role, or login name',
                    'button_element' => 'text',
                ],
                'as' => 'form'
            ],
        ]);
    }

    /**
     * Edit user
     *
     * @return \Zend\Http\Response|ViewModel|array
     */
    public function editAction()
    {
        /* @var $user \Auth\Entity\User */
        $user = $this->userRepository->find($this->params('id'), \Doctrine\ODM\MongoDB\LockMode::NONE, null, ['allowDeactivated' => true]);
        
        // check if user is not found
        if (!$user) {

            return $this->notFoundAction();
        }
        
        $params = $this->params();
        $forms = $this->formManager;
        /* @var $infoContainer \Auth\Form\UserProfileContainer */
        $infoContainer = $forms->get('Auth/UserProfileContainer');
        $infoContainer->setEntity($user);
        $statusContainer = $forms->get('Auth/UserStatusContainer');
        $statusContainer->setEntity($user);
        
        // set selected user to image strategy
        $imageStrategy = $infoContainer->getForm('info.image')
            ->getHydrator()
            ->/** @scrutinizer ignore-call */ getHydrator()
            ->getStrategy('image');
            ->/** @scrutinizer ignore-call */ getStrategy('image');
		$fileEntity = $imageStrategy->getFileEntity();
		$fileEntity->setUser($user);
		$imageStrategy->setFileEntity($fileEntity);
        
        if ($this->request->isPost()) {
        if ($this->request->/** @scrutinizer ignore-call */ isPost()) {
            $formName = $params->fromQuery('form');
            $container = $formName === 'status' ? $statusContainer : $infoContainer;
            $form = $container->getForm($formName);
        
            if ($form) {
                $postData  = $form->getOption('use_post_array') ? $params->fromPost() : [];
                $filesData = $form->getOption('use_files_array') ? $params->fromFiles() : [];
                $form->setData(array_merge($postData, $filesData));
                $form->setData(array_merge($postData, /** @scrutinizer ignore-type */ $filesData));
        
                if (!$form->isValid()) {
                    return new JsonModel(
                        array(
                            'valid' => false,
                            'errors' => $form->getMessages(),
                        )
                    );
                }
                
                $this->userRepository->store($user);
        
                if ('file-uri' === $params->fromPost('return')) {
                    $content = $form->getHydrator()->getLastUploadedFile()->getUri();
                    $content = $form->getHydrator()->/** @scrutinizer ignore-call */ getLastUploadedFile()->getUri();
                } else {
                    if ($form instanceof SummaryFormInterface) {
                        $form->setRenderMode(SummaryFormInterface::RENDER_SUMMARY);
                        $viewHelper = 'summaryForm';
                    } else {
                        $viewHelper = 'form';
                    }
                    $content = $this->viewHelper->get($viewHelper)->__invoke($form);
                }
        
                return new JsonModel(
                    array(
                        'valid' => $form->isValid(),
                        'content' => $content,
                    )
                );
            }
        }
        
        return [
            'infoContainer' => $infoContainer,
            'statusContainer' => $statusContainer
        ];
    }

    public function switchAction()
    {
        /* @var \Auth\Controller\Plugin\UserSwitcher $switcher */
        $do = $this->params()->fromQuery('do');
        if ('clear' == $do) {
            $switcher = $this->plugin('Auth/User/Switcher');
            $ref      = $switcher();
            $result   = ['success' => true];

            if (true !== $ref && $ref) {
                $result['ref'] = $ref;
            }

            return new JsonModel($result);
        }

        $this->acl('Auth/Users', 'admin-access');
               acl('Auth/Users', 'admin-access');

        if ('list' == $do) {
            /* @var \Auth\Entity\User $user */
            /* @var \Zend\Paginator\Paginator $paginator */
            $paginator = $this->paginator('Auth/User', ['page' => 1]);
            $paginator = $this->paginator('Auth/User', ['page' => 1]);
            $result = [];

            foreach ($paginator as $user) {
                $result[] = [
                    'id' => $user->getId(),
                    'name' => $user->getInfo()->getDisplayName(false),
                    'email' => $user->getInfo()->getEmail(),
                    'login' => $user->getLogin()
                ];
            }
            return new JsonModel([
                'items' => $result,
                'total' => $paginator->getTotalItemCount(),
            ]);
        }

        $switcher = $this->plugin('Auth/User/Switcher');
        $success  = $switcher($this->params()->fromQuery('id'), [ 'ref' => urldecode($this->params()->fromQuery('ref')) ]);


        return new JsonModel(['success' => true]);
    }
}

cross-solution / YAWIK

Pull Request — develop (#521)

UsersController::editAction() B

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like
