SeneddView::display_front_senedd() - Code Metrics - Inspection of "Welsh parliament support" - mysociety/theyworkforyou - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#1700)

by Matthew

created 2023-06-09 13:27 UTC

SeneddView::display_front_senedd() B

↳ Parent: SeneddView

Complexity

Conditions	8
Paths	12

Size

Total Lines	57
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	8
eloc	36
nc	12
nop	0
dl	0
loc	57
rs	8.0995
c	0
b	0
f	0

How to fix Long Method

<?php

namespace MySociety\TheyWorkForYou\SectionView;

class SeneddView extends SectionView {
    protected $index_template = 'section/senedd_index';

    public function __construct() {
        if (LANGUAGE == 'cy') {

            $this->major = 11;
            $this->class = 'SENEDDCYLIST';
        } else {
            $this->major = 10;
            $this->class = 'SENEDDENLIST';
        }
        parent::__construct();
    }

    protected function display_front() {
        if (get_http_var('more')) {
            return parent::display_front();
        } else {
            return $this->display_front_senedd();
        }
    }

    protected function front_content() {
        return $this->list->display('biggest_debates', array('days'=>30, 'num'=>20), 'none');
    }

    protected function getURLs($data) {
        $urls = array();

        $day = new \MySociety\TheyWorkForYou\Url('senedddebates');
        $urls['seneddday'] = $day;
        $urls['day'] = $day;

        return $urls;
    }

    protected function getSearchSections() {
        return array(
            array( 'section' => 'wales' )
        );
    }

    protected function display_front_senedd() {
        global $this_page;
        $this_page = "seneddoverview";

        $data = array();

        $data['popular_searches'] = null;

        $user = new \MySociety\TheyWorkForYou\User();
        $data['mp_data'] = $user->getRep("WMC", 1);

        $data['urls'] = $this->getURLs($data);

        $DEBATELIST = new $this->class();

        $debates = $DEBATELIST->display('recent_debates', array('days' => 30, 'num' => 6), 'none');
        $MOREURL = new \MySociety\TheyWorkForYou\Url('senedddebatesfront');
        $MOREURL->insert( array( 'more' => 1 ) );

        // this makes sure that we don't repeat this debate in the list below
        $random_debate = null;
        if ( isset($debates['data']) && count($debates['data']) ) {
            $random_debate = $debates['data'][0];
        }

        $recent = array();
        if ( isset($debates['data']) && count($debates['data']) ) {
            // at the start of a session there may be less than 6
            // debates
            $max = 6;
            if ( count($debates['data']) < 6 ) {
                $max = count($debates['data']);
            }
            for ( $i = 1; $i < $max; $i++ ) {
                $debate = $debates['data'][$i];
                $debate['desc'] = "Senedd";
                $debate['more_url'] = $MOREURL->generate();
                $recent[] = $debate;
            }
        }

        $featured = array();
        if ( $random_debate ) {
            $featured = $random_debate;
            $featured['more_url'] = $MOREURL->generate();
            $featured['desc'] = 'Senedd';
            $featured['related'] = array();
            $featured['featured'] = false;
        }

        $data['featured'] = $featured;
        $data['debates'] = array( 'recent' => $recent);

        $data['regional'] = $this->getMSList();
        $data['template'] = 'senedd/index';

        return $data;
    }

    protected function getMSList() {
        global $THEUSER;

        $mreg = array();
        if ($THEUSER->isloggedin() && $THEUSER->postcode() != '' || $THEUSER->postcode_is_set()) {

            return array_merge(
                \MySociety\TheyWorkForYou\Member::getRegionalList($THEUSER->postcode, 5, 'WAC'),
                \MySociety\TheyWorkForYou\Member::getRegionalList($THEUSER->postcode, 5, 'WAE')
            );
        }

        return $mreg;
    }
}


1			<?php
2
3			namespace MySociety\TheyWorkForYou\SectionView;
4
5			class SeneddView extends SectionView {
6			protected $index_template = 'section/senedd_index';
7
8			public function __construct() {
9			if (LANGUAGE == 'cy') {
			0 ignored issues – show introduced 2023-04-05 16:47 UTC by Report Bug Copy Issue Report The condition `MySociety\TheyWorkForYou...onView\LANGUAGE == 'cy'` is always `false`. Loading history...
10			$this->major = 11;
11			$this->class = 'SENEDDCYLIST';
12			} else {
13			$this->major = 10;
14			$this->class = 'SENEDDENLIST';
15			}
16			parent::__construct();
17			}
18
19			protected function display_front() {
20			if (get_http_var('more')) {
21			return parent::display_front();
22			} else {
23			return $this->display_front_senedd();
24			}
25			}
26
27			protected function front_content() {
28			return $this->list->display('biggest_debates', array('days'=>30, 'num'=>20), 'none');
29			}
30
31			protected function getURLs($data) {
32			$urls = array();
33
34			$day = new \MySociety\TheyWorkForYou\Url('senedddebates');
35			$urls['seneddday'] = $day;
36			$urls['day'] = $day;
37
38			return $urls;
39			}
40
41			protected function getSearchSections() {
42			return array(
43			array( 'section' => 'wales' )
44			);
45			}
46
47			protected function display_front_senedd() {
48			global $this_page;
49			$this_page = "seneddoverview";
50
51			$data = array();
52
53			$data['popular_searches'] = null;
54
55			$user = new \MySociety\TheyWorkForYou\User();
56			$data['mp_data'] = $user->getRep("WMC", 1);
57
58			$data['urls'] = $this->getURLs($data);
59
60			$DEBATELIST = new $this->class();
61
62			$debates = $DEBATELIST->display('recent_debates', array('days' => 30, 'num' => 6), 'none');
63			$MOREURL = new \MySociety\TheyWorkForYou\Url('senedddebatesfront');
64			$MOREURL->insert( array( 'more' => 1 ) );
65
66			// this makes sure that we don't repeat this debate in the list below
67			$random_debate = null;
68			if ( isset($debates['data']) && count($debates['data']) ) {
69			$random_debate = $debates['data'][0];
70			}
71
72			$recent = array();
73			if ( isset($debates['data']) && count($debates['data']) ) {
74			// at the start of a session there may be less than 6
75			// debates
76			$max = 6;
77			if ( count($debates['data']) < 6 ) {
78			$max = count($debates['data']);
79			}
80			for ( $i = 1; $i < $max; $i++ ) {
81			$debate = $debates['data'][$i];
82			$debate['desc'] = "Senedd";
83			$debate['more_url'] = $MOREURL->generate();
84			$recent[] = $debate;
85			}
86			}
87
88			$featured = array();
89			if ( $random_debate ) {
90			$featured = $random_debate;
91			$featured['more_url'] = $MOREURL->generate();
92			$featured['desc'] = 'Senedd';
93			$featured['related'] = array();
94			$featured['featured'] = false;
95			}
96
97			$data['featured'] = $featured;
98			$data['debates'] = array( 'recent' => $recent);
99
100			$data['regional'] = $this->getMSList();
101			$data['template'] = 'senedd/index';
102
103			return $data;
104			}
105
106			protected function getMSList() {
107			global $THEUSER;
108
109			$mreg = array();
110			if ($THEUSER->isloggedin() && $THEUSER->postcode() != '' \|\| $THEUSER->postcode_is_set()) {
			0 ignored issues – show introduced 2019-07-26 09:15 UTC by Report Bug Copy Issue Report Consider adding parentheses for clarity. Current Interpretation: `($THEUSER->isloggedin() ...USER->postcode_is_set()`, Probably Intended Meaning: `$THEUSER->isloggedin() &...SER->postcode_is_set())` Loading history...
111			return array_merge(
112			\MySociety\TheyWorkForYou\Member::getRegionalList($THEUSER->postcode, 5, 'WAC'),
113			\MySociety\TheyWorkForYou\Member::getRegionalList($THEUSER->postcode, 5, 'WAE')
114			);
115			}
116
117			return $mreg;
118			}
119			}
120

mysociety / theyworkforyou

Pull Request — master (#1700)

SeneddView::display_front_senedd() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like