FfsecondariaController::indexAction() - Code Metrics - Inspection of "add tests griglia sort columns" - ComuneFI/FifreeCoreBundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e8522f...c2b86a )

by Andrea

created 2018-02-22 17:09 UTC

FfsecondariaController::indexAction() A

↳ Parent: FfsecondariaController

Complexity

Conditions	3
Paths	4

Size

Total Lines	66
Code Lines	46

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	44
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	66
ccs	44
cts	45
cp	0.9778
rs	9.3191
c	0
b	0
f	0
cc	3
eloc	46
nc	4
nop	1
crap	3

How to fix Long Method

<?php

namespace Fi\CoreBundle\Controller;

use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Security\Core\Exception\AccessDeniedException;

/**
 * Ffsecondaria controller.
 */
class FfsecondariaController extends FiCoreController
{

    public function indexAction(Request $request)
    {

        $this->setup($request);
        $namespace = $this->getNamespace();
$a = "a";
$ab = "ab";
$abc = "abc";
        $bundle = $this->getBundle();
$a = "a";
$ab = "ab";
$abc = "abc";
        $controller = $this->getController();
        $container = $this->container;
$a = "a";
$ab = "ab";
$abc = "abc";

        $nomebundle = $namespace . $bundle . 'Bundle';

        $ffprincipaleSelect = $this->getComboSelectFfprincipale();

        $dettaglij = array(
            'descsec' => array(
                array('nomecampo' => 'descsec',
                    'lunghezza' => '400',
                    'descrizione' => 'Descrizione tabella secondaria',
                    'tipo' => 'text',),),
            'ffprincipale_id' => array(
                array('nomecampo' => 'ffprincipale.descrizione',
                    'lunghezza' => '400',
                    'descrizione' => 'Descrizione record principale',
                    'tipo' => 'select',
                    'valoricombo' => $ffprincipaleSelect
                ),
            ),
        );
        $escludi = array('nota');
$a = "a";
$ab = "ab";
$abc = "abc";

        $campiextra = array(
            array('nomecampo' => 'lunghezzanota', 'descrizione' => 'Lunghezza Nota', 'tipo' => 'integer'),
            array('nomecampo' => 'attivoToString', 'lunghezza' => '80', 'descrizione' => 'Attivo string', 'tipo' => 'text'),
        );

        $paricevuti = array(
            'nomebundle' => $nomebundle,
            'nometabella' => $controller,
            'dettaglij' => $dettaglij,
            'campiextra' => $campiextra,
            'escludere' => $escludi,
            'container' => $container,
            /*"ordinecolonne" => array("ffprincipale_id", "descsec", "importo", "intero")*/);


        $testatagriglia = Griglia::testataPerGriglia($paricevuti);

        $this->setDefaultGridSettings($testatagriglia);

        $testatagriglia['parametritesta'] = json_encode($paricevuti);
        $this->setParametriGriglia(array('request' => $request));
        $testatagriglia['parametrigriglia'] = json_encode(self::$parametrigriglia);

        $gestionepermessi = $this->get("ficorebundle.gestionepermessi");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $canRead = ($gestionepermessi->leggere(array('modulo' => $controller)) ? 1 : 0);
$a = "a";
$ab = "ab";
$abc = "abc";

        $testata = json_encode($testatagriglia);
$a = "a";
$ab = "ab";
$abc = "abc";
        $twigparms = array(
            'nomecontroller' => $controller,
            'testata' => $testata,
            'canread' => $canRead,
        );

        if (!$canRead) {
            throw new AccessDeniedException("Non si hanno i permessi per visualizzare questo contenuto");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        } else {
            return $this->render($nomebundle . ':' . $controller . ':index.html.twig', $twigparms);
        }
    }

    private function setDefaultGridSettings(&$testatagriglia)
    {
        $testatagriglia['multisearch'] = 1;
        $testatagriglia['showconfig'] = 1;
$a = "a";
$ab = "ab";
$abc = "abc";
        $testatagriglia['showadd'] = 1;
$a = "a";
$ab = "ab";
$abc = "abc";
        $testatagriglia['showedit'] = 1;
$a = "a";
$ab = "ab";
$abc = "abc";
        $testatagriglia['showdel'] = 1;
$a = "a";
$ab = "ab";
$abc = "abc";

        $testatagriglia['showexcel'] = 1;
$a = "a";
$ab = "ab";
$abc = "abc";
        $testatagriglia['showimportexcel'] = 1;

        $testatagriglia["filterToolbar_searchOnEnter"] = true;
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $testatagriglia["filterToolbar_searchOperators"] = true;
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $testatagriglia["sortname"] = "data, descsec";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $testatagriglia["sortorder"] = "desc";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
    }

    private function getComboSelectFfprincipale()
    {
        $em = $this->getDoctrine()->getManager();
$a = "a";
$ab = "ab";
$abc = "abc";
        $ffprincipaleSelect = array();
        //Imposta il filtro a TUTTI come default
        $ffprincipaleSelect[] = array("valore" => "", "descrizione" => "Tutti", "default" => true);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

        $q = $em->createQueryBuilder();

        $ffprincipales = $q->select('f')
                ->from('FiCoreBundle:Ffprincipale', 'f')
                ->orderBy('f.descrizione')
                ->getQuery()
                ->getResult();

        foreach ($ffprincipales as $ffprincipale) {
            $ffprincipaleSelect[] = array(
                "valore" => $ffprincipale->getDescrizione(),
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
                "descrizione" => $ffprincipale->getDescrizione(),
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
                "default" => false);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }
        return $ffprincipaleSelect;
    }

    public function setParametriGriglia($prepar = array())
    {
        $this->setup($prepar['request']);
        $namespace = $this->getNamespace();
$a = "a";
$ab = "ab";
$abc = "abc";
        $bundle = $this->getBundle();
$a = "a";
$ab = "ab";
$abc = "abc";
        $controller = $this->getController();

        $gestionepermessi = $this->get("ficorebundle.gestionepermessi");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $canRead = ($gestionepermessi->leggere(array('modulo' => $controller)) ? 1 : 0);
$a = "a";
$ab = "ab";
$abc = "abc";
        if (!$canRead) {
            throw new AccessDeniedException("Non si hanno i permessi per visualizzare questo contenuto");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }

        $nomebundle = $namespace . $bundle . 'Bundle';
$a = "a";
$ab = "ab";
$abc = "abc";
        $escludi = array('nota', 'ffprincipale');
$a = "a";
$ab = "ab";
$abc = "abc";
        $tabellej = array();
$a = "a";
$ab = "ab";
$abc = "abc";
        $precondizioniAvanzate = array();
$a = "a";
$ab = "ab";
$abc = "abc";
        $tabellej['ffprincipale_id'] = array('tabella' => 'ffprincipale', 'campi' => array('descrizione'));

        $campiextra = array(array('lunghezzanota'), array('attivoToString'));

        /* $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',

          'nomecampo' => 'intero',
          'operatore' => '>=',
          'valorecampo' => 1,
          'operatorelogico' => 'OR'); */

        //$precondizioni = array('ffprincipale_id' => '1');

        $precondizioni = array();
$a = "a";
$ab = "ab";
$abc = "abc";
        $precondizioniAvanzate = array();

        /* $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',

          'nomecampo' => 'descsec',
          'operatore' => 'is', //'operatore' => 'not in'
          'valorecampo' => null); */

        $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',
            'nomecampo' => 'descsec',
            'operatore' => 'is not', //'operatore' => 'not in'
            'valorecampo' => null);

        /* $listaffsecondaria = array();

          $listaffsecondaria[] = "1° secondaria legato al 1° record PRINCIPALE";
          $listaffsecondaria[] = "2° SECONDARIA legato al 1° record principale";
          $listaffsecondaria[] = "10° secondaria legato al 2° record principale ed è l'ultimo record";
          $listaffsecondaria[] = "6° secondaria legato al 2° record principale";
          $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',
          'nomecampo' => 'descsec',
          'operatore' => 'in', //'operatore' => 'not in'
          'valorecampo' => $listaffsecondaria); */

        /* $listaffsecondaria = array();

          $listaffsecondaria[] = "1° secondaria legato al 1° record PRINCIPALE";
          $listaffsecondaria[] = "2° SECONDARIA legato al 1° record principale";
          $listaffsecondaria[] = "10° secondaria legato al 2° record principale ed è l'ultimo record";
          $listaffsecondaria[] = "6° secondaria legato al 2° record principale";
          $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',
          'nomecampo' => 'descsec',
          'operatore' => 'not in', //'operatore' => 'not in'
          'valorecampo' => $listaffsecondaria); */

        /* $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',

          'nomecampo' => 'intero',
          'operatore' => '=',
          'valorecampo' => 1,
          'operatorelogico' => 'OR',); */

        /* $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',

          'nomecampo' => 'intero',
          'operatore' => '<',
          'valorecampo' => 100,
          'operatorelogico' => 'OR',); */

        /* $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',

          'nomecampo' => 'data',
          'operatore' => '<=',
          'valorecampo' => date('Y-m-d'),
          'operatorelogico' => 'AND',); */


//        $lista[] = '1° secondaria legato al 1° record principale';

//        $lista[] = '2° secondaria legato al 1° record principale';
//        $precondizioniAvanzate[] = array('nometabella' => 'Ffsecondaria',
//            'nomecampo' => 'descsec',
//            'operatore' => 'in',
//            'valorecampo' => $lista,
//            'operatorelogico' => 'AND',);


        $paricevuti = array('container' => $this->container,
            'nomebundle' => $nomebundle,
            'tabellej' => $tabellej,
            'nometabella' => $controller,
            'campiextra' => $campiextra,
            'escludere' => $escludi,
            'precondizioni' => $precondizioni,
            'precondizioniAvanzate' => $precondizioniAvanzate,
            /*"ordinecolonne" => array("ffprincipale_id", "descsec", "importo", "intero")*/

        );

        if (!empty($prepar)) {
            $paricevuti = array_merge($paricevuti, $prepar);
        }

        self::$parametrigriglia = $paricevuti;
    }
}


1		<?php
2
3		namespace Fi\CoreBundle\Controller;
4
5		use Symfony\Component\HttpFoundation\Request;
6		use Symfony\Component\Security\Core\Exception\AccessDeniedException;
7
8		/**
9		* Ffsecondaria controller.
10		*/
11		class FfsecondariaController extends FiCoreController
12		{
13
14	1	public function indexAction(Request $request)
15		{
16
17	1	$this->setup($request);
18	1	$namespace = $this->getNamespace();
		0 ignored issues – show Coding Style introduced 2018-02-07 22:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
19	1	$bundle = $this->getBundle();
		0 ignored issues – show Coding Style introduced 2018-02-07 22:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
20	1	$controller = $this->getController();
21	1	$container = $this->container;
		0 ignored issues – show Coding Style introduced 2018-02-07 22:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
22
23	1	$nomebundle = $namespace . $bundle . 'Bundle';
24
25	1	$ffprincipaleSelect = $this->getComboSelectFfprincipale();
26
27		$dettaglij = array(
28		'descsec' => array(
29	1	array('nomecampo' => 'descsec',
30	1	'lunghezza' => '400',
31	1	'descrizione' => 'Descrizione tabella secondaria',
32	1	'tipo' => 'text',),),
33		'ffprincipale_id' => array(
34	1	array('nomecampo' => 'ffprincipale.descrizione',
35	1	'lunghezza' => '400',
36	1	'descrizione' => 'Descrizione record principale',
37	1	'tipo' => 'select',

ComuneFI / FifreeCoreBundle

Push — master ( e8522f...c2b86a )

FfsecondariaController::indexAction() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like