Standard::setUserData() - Code Metrics - Inspection of "Small fixes and improvements to new alerts front e..." - mysociety/theyworkforyou - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions
Pull Request — master (#1846)

by Struan
created 2024-12-17 10:30 UTC
Standard::setUserData() F

↳ Parent: Standard
Complexity

Conditions	40
Paths	4160
Size

Total Lines	116
Code Lines	88
Duplication

Lines	0
Ratio	0 %
Importance

Changes	3
Bugs	0	Features	1
Metric	Value
cc	40
eloc	88
nc	4160
nop	0
dl	0
loc	116
rs	0
c	3
b	0
f	1
How to fix Long Method Complexity

<?php

namespace MySociety\TheyWorkForYou\AlertView;

include_once '../../../www/includes/easyparliament/init.php';
include_once INCLUDESPATH . "easyparliament/member.php";
include_once INCLUDESPATH . "easyparliament/searchengine.php";
include_once INCLUDESPATH . '../../commonlib/phplib/auth.php';
include_once INCLUDESPATH . '../../commonlib/phplib/crosssell.php';

class Standard extends \MySociety\TheyWorkForYou\AlertView {
    public $data;

    public function __construct($THEUSER = null) {
        parent::__construct($THEUSER);
        $this->data = [];
    }

    public function display() {
        global $this_page;
        $this_page = "alert";

        $this->processAction();
        $this->getBasicData();
        $this->checkInput();
        $this->searchForConstituenciesAndMembers();

        if ($this->data['step'] || $this->data['addword']) {
            $this->processStep();
        } elseif (!$this->data['results'] == 'changes-abandoned' && !sizeof($this->data['errors']) && $this->data['submitted'] && ($this->data['keyword'] || $this->data['pid'])) {
            $this->addAlert();
        }

        $this->formatSearchTerms();
        $this->checkForCommonMistakes();
        $this->formatSearchMemberData();
        $this->setUserData();

        return $this->data;
    }

    # This only happens if we have an alert and want to do something to it.
    private function processAction() {
        $token = get_http_var('t');
        $alert = $this->alert->check_token($token);

        $this->data['results'] = false;
        if ($action = get_http_var('action')) {
            $success = true;
            if ($action == 'Confirm') {
                $success = $this->confirmAlert($token);
                if ($success) {
                    $this->data['results'] = 'alert-confirmed';
                    $this->data['criteria'] = $this->alert->criteria;
                    $this->data['display_criteria'] = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($this->alert->criteria);
                }
            } elseif ($action == 'Suspend') {
                $success = $this->suspendAlert($token);
                if ($success) {
                    $this->data['results'] = 'alert-suspended';
                }
            } elseif ($action == 'Resume') {
                $success = $this->resumeAlert($token);
                if ($success) {
                    $this->data['results'] = 'alert-resumed';
                }
            } elseif ($action == 'Delete') {
                $success = $this->deleteAlert($token);
                if ($success) {
                    $this->data['results'] = 'alert-deleted';
                }
            } elseif ($action == 'Delete All') {
                $success = $this->deleteAllAlerts($token);
                if ($success) {
                    $this->data['results'] = 'all-alerts-deleted';
                }
            } elseif ($action == 'Abandon') {
                $this->data['results'] = 'changes-abandoned';
            }
            if (!$success) {
                $this->data['results'] = 'alert-fail';
            }
        }

        $this->data['alert'] = $alert;
    }

    # Process a screen in the alert creation wizard
    private function processStep() {
        # fetch a list of suggested terms. Need this for the define screen so we can filter out the suggested terms
        # and not show them if the user goes back
        if (($this->data['step'] == 'review' || $this->data['step'] == 'define') && !$this->data['shown_related']) {
            $suggestions = [];
            foreach ($this->data['keywords'] as $word) {
                $terms = $this->alert->get_related_terms($word);
                $terms = array_diff($terms, $this->data['keywords']);
                if ($terms && count($terms)) {

                    $suggestions = array_merge($suggestions, $terms);
                }
            }

            if (count($suggestions) > 0) {
                $this->data['step'] = 'add_vector_related';
                $this->data['suggestions'] = $suggestions;
            }
        # confirm the alert. Handles both creating and editing alerts
        } elseif ($this->data['step'] == 'confirm') {
            $success = true;
            # if there's already an alert assume we are editing it and user must be logged in
            if ($this->data['alert']) {
                $success = $this->updateAlert($this->data['alert']['id'], $this->data);
                $success = $this->updateAlert($this->data['alert']['id'], $this->data);
                if ($success) {
                    # reset all the data to stop anything getting confused
                    $this->data['results'] = 'alert-confirmed';
                    $this->data['step'] = '';
                    $this->data['pid'] = '';
                    $this->data['alertsearch'] = '';
                    $this->data['pc'] = '';
                    $this->data['members'] = false;
                    $this->data['constituencies'] = [];
                } else {
                    $this->data['results'] = 'alert-fail';
                    $this->data['step'] = 'review';
                }
            } else {
                $success = $this->addAlert();
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
                $this->data['step'] = '';
            }
        }
    }


    private function getBasicData() {
        global $this_page;

        if ($this->user->loggedin()) {
            $this->data['email'] = $this->user->email();
            $this->data['email_verified'] = true;
        } elseif ($this->data['alert']) {
            $this->data['email'] = $this->data['alert']['email'];
            $this->data['email_verified'] = true;
        } else {
            $this->data["email"] = trim(get_http_var("email"));
            $this->data['email_verified'] = false;
        }

        $this->data['token'] = get_http_var('t');
        $this->data['step'] = trim(get_http_var("step"));
        $this->data['mp_step'] = trim(get_http_var("mp_step"));
        $this->data['addword'] = trim(get_http_var("addword"));
        $this->data['this_step'] = trim(get_http_var("this_step"));
        $this->data['shown_related'] = get_http_var('shown_related');
        $this->data['match_all'] = get_http_var('match_all') == 'on';
        $this->data['keyword'] = trim(get_http_var("keyword"));
        $this->data['search_section'] = '';
        $this->data['alertsearch'] = trim(get_http_var("alertsearch"));
        $this->data['mp_search'] = trim(get_http_var("mp_search"));
        $this->data['pid'] = trim(get_http_var("pid"));
        $this->data['pc'] = get_http_var('pc');
        $this->data['submitted'] = get_http_var('submitted') || $this->data['pid'] || $this->data['keyword'] || $this->data['step'];

        if ($this->data['addword'] || $this->data['step']) {
            $alert = $this->alert->check_token($this->data['token']);

            $criteria = '';
            if ($alert) {
                $criteria = $alert['criteria'];
            }

            $this->data['alert'] = $alert;

            $this->data['alert_parts'] = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($criteria, true);

            $existing_rep = '';
            if (isset($this->data['alert_parts']['spokenby'])) {
                $existing_rep = $this->data['alert_parts']['spokenby'][0];
            }

            $existing_section = '';
            if (count($this->data['alert_parts']['sections'])) {
                $existing_section = $this->data['alert_parts']['sections'][0];
            }

            if ($this->data['alert_parts']['match_all']) {
                $this->data['match_all'] = true;
            }

            $words = get_http_var('words', $this->data['alert_parts']['words'], true);

            $this->data['words'] = [];
            $this->data['keywords'] = [];
            foreach ($words as $word) {
                if (trim($word) != '') {
                    $this->data['keywords'][] = $word;
                    $this->data['words'][] = $this->wrap_phrase_in_quotes($word);
                }
            }

            $add_all_related = get_http_var('add_all_related');
            $this->data['add_all_related'] = $add_all_related;
            $this->data['skip_keyword_terms'] = [];

            $selected_related_terms = get_http_var('selected_related_terms', [], true);
            $this->data['selected_related_terms'] = $selected_related_terms;

            if ($this->data['step'] !== 'define') {
                if ($add_all_related) {
                    $this->data['selected_related_terms'] = [];
                    $related_terms = get_http_var('related_terms', [], true);
                    foreach ($related_terms as $term) {
                        $this->data['skip_keyword_terms'][] = $term;
                        $this->data['keywords'][] = $term;
                        $this->data['words'][] = $this->wrap_phrase_in_quotes($term);
                    }
                } else {
                    $this->data['skip_keyword_terms'] = $selected_related_terms;
                    foreach ($selected_related_terms as $term) {
                        $this->data['keywords'][] = $term;
                        $this->data['words'][] = $this->wrap_phrase_in_quotes($term);
                    }
                }
            }
            $this->data['exclusions'] = trim(get_http_var("exclusions", implode('', $this->data['alert_parts']['exclusions'])));
            $this->data['representative'] = trim(get_http_var("representative", $existing_rep));

            $this->data['search_section'] = trim(get_http_var("search_section", $existing_section));

            $separator = ' OR ';
            if ($this->data['match_all']) {
                $separator = ' ';
            }
            $this->data['keyword'] = implode($separator, $this->data['words']);
            if ($this->data['exclusions']) {
                $this->data['keyword'] = '(' . $this->data['keyword'] . ') -' . $this->data["exclusions"];
            }

            $this->data['results'] = '';

            $this->getSearchSections();
        } # XXX probably should do something here if $alertsearch is set

        $this->data['sign'] = get_http_var('sign');
        $this->data['site'] = get_http_var('site');
        $this->data['message'] = '';

        $ACTIONURL = new \MySociety\TheyWorkForYou\Url($this_page);
        $ACTIONURL->reset();
        $this->data['actionurl'] = $ACTIONURL->generate();

    }

    private function wrap_phrase_in_quotes($phrase) {
        if (strpos($phrase, ' ') > 0) {
            $phrase = '"' . trim($phrase, '"') . '"';
        }

        return $phrase;
    }

    private function getRecentResults($text) {
        global $SEARCHENGINE;
        $se = new \SEARCHENGINE($text);
        $this->data['search_result_count'] = $se->run_count(0, 10);
        $se->run_search(0, 1, 'date');
    }

    private function getSearchSections() {
        $this->data['sections'] = [];
        if ($this->data['search_section']) {
            foreach (explode(' ', $this->data['search_section']) as $section) {
                $this->data['sections'][] = \MySociety\TheyWorkForYou\Utility\Alert::sectionToTitle($section);
            }
        }
    }

    protected function updateAlert($token) {
        $success = $this->alert->update($token, $this->data);
        return $success;
    }

    protected function checkInput() {
        global $SEARCHENGINE;

        $errors = [];

        # these are the initial screens and so cannot have any errors as we've not submitted
        if (!$this->data['submitted'] || $this->data['step'] == 'define' || $this->data['mp_step'] == 'mp_alert') {
            $this->data['errors'] = $errors;
            return;
        }

        // Check each of the things the user has input.
        // If there is a problem with any of them, set an entry in the $errors array.
        // This will then be used to (a) indicate there were errors and (b) display
        // error messages when we show the form again.

        // Check email address is valid and unique.
        if (!$this->data['email']) {
            $errors["email"] = gettext("Please enter your email address");
        } elseif (!validate_email($this->data["email"])) {
            // validate_email() is in includes/utilities.php
            $errors["email"] = gettext("Please enter a valid email address");
        }

        if ($this->data['pid'] && !ctype_digit($this->data['pid'])) {
            $errors['pid'] = 'Invalid person ID passed';
        }

        $text = $this->data['alertsearch'];
        if ($this->data['mp_search']) {
            $text = $this->data['mp_search'];
        }
        if (!$text) {
            $text = $this->data['keyword'];
        }

        if ($this->data['submitted'] && !$this->data['pid'] && !$text) {
            $errors['alertsearch'] = gettext('Please enter what you want to be alerted about');
        }

        if (strpos($text, '..')) {
            $errors['alertsearch'] = gettext('You probably don&rsquo;t want a date range as part of your criteria, as you won&rsquo;t be alerted to anything new!');
        }

        $se = new \SEARCHENGINE($text);
        if (!$se->valid) {
            $errors['alertsearch'] = sprintf(gettext('That search appears to be invalid - %s - please check and try again.'), $se->error);
        }

        if (strlen($text) > 255) {
            $errors['alertsearch'] = gettext('That search is too long for our database; please split it up into multiple smaller alerts.');
        }

        $this->data['errors'] = $errors;
    }

    protected function searchForConstituenciesAndMembers() {
        if ($this->data['results'] == 'changes-abandoned') {
            $this->data['members'] = false;
            return;
        }

        $text = $this->data['alertsearch'];
        if ($this->data['mp_search']) {
            $text = $this->data['mp_search'];
        }
        $errors = [];
        if ($text != '') {
            //$members_from_pids = array_values(\MySociety\TheyWorkForYou\Utility\Search::membersForIDs($this->data['alertsearch']));
            $members_from_names = [];
            $names_from_pids = array_values(\MySociety\TheyWorkForYou\Utility\Search::speakerNamesForIDs($text));
            foreach ($names_from_pids as $name) {
                $members_from_names = array_merge($members_from_names,\MySociety\TheyWorkForYou\Utility\Search::searchMemberDbLookupWithNames($name));
            }
            $members_from_words = \MySociety\TheyWorkForYou\Utility\Search::searchMemberDbLookupWithNames($text, true);
            $this->data['members'] = array_merge($members_from_words, $members_from_names);
            [$this->data['constituencies'], $this->data['valid_postcode']] = \MySociety\TheyWorkForYou\Utility\Search::searchConstituenciesByQuery($text, false);
        } elseif ($this->data['pid']) {
            $MEMBER = new \MEMBER(['person_id' => $this->data['pid']]);
            $this->data['members'] = [[
                "person_id" => $MEMBER->person_id,
                "given_name" => $MEMBER->given_name,
                "family_name" => $MEMBER->family_name,
                "house" => $MEMBER->house_disp,
                "title" => $MEMBER->title,
                "lordofname" => $MEMBER->lordofname,
                "constituency" => $MEMBER->constituency,
            ]];
        } elseif (isset($this->data['representative']) && $this->data['representative'] != '') {
            $this->data['members'] = \MySociety\TheyWorkForYou\Utility\Search::searchMemberDbLookupWithNames($this->data['representative'], true);

            $member_count = count($this->data['members']);
            if ($member_count == 0) {
                $errors["representative"] = gettext("No matching representative found");
            } elseif ($member_count > 1) {
                $errors["representative"] = gettext("Multiple matching representatives found, please select one.");
            } else {
                $this->data['pid'] = $this->data['members'][0]['person_id'];
            }
        } else {
            $this->data['members'] = [];
        }

        # If the above search returned one result for constituency
        # search by postcode, use it immediately
        if (isset($this->data['constituencies']) && count($this->data['constituencies']) == 1 && $this->data['valid_postcode']) {
            $MEMBER = new \MEMBER(['constituency' => array_values($this->data['constituencies'])[0], 'house' => 1]);
            $this->data['pid'] = $MEMBER->person_id();
            $this->data['pc'] = $text;
            unset($this->data['constituencies']);
        }

        if (isset($this->data['constituencies'])) {
            $cons = [];
            foreach ($this->data['constituencies'] as $constituency) {
                try {
                    $MEMBER = new \MEMBER(['constituency' => $constituency]);
                    $cons[$constituency] = $MEMBER;
                } catch (\MySociety\TheyWorkForYou\MemberException $e) {
                    // do nothing
                }
            }
            $this->data['constituencies'] = $cons;
            if (count($cons) == 1) {
                $cons = array_values($cons);
                $this->data['pid'] = $cons[0]->person_id();
            }
        }

        if ($this->data['alertsearch'] && !$this->data['mp_step'] && ($this->data['pid'] || $this->data['members'] || $this->data['constituencies'])) {
            if (count($this->data['members']) == 1) {
                $this->data['pid'] = $this->data['members'][0]['person_id'];
            }
            $this->data['mp_step'] = 'mp_alert';
            $this->data['mp_search'] = $this->data['alertsearch'];
            $this->data['alertsearch'] = '';
        }

        if (count($this->data["errors"]) > 0) {
            $this->data["errors"] = array_merge($this->data["errors"], $errors);
        } else {
            $this->data["errors"] = $errors;
        }
    }

    protected function addAlert() {
        $external_auth = auth_verify_with_shared_secret($this->data['email'], OPTION_AUTH_SHARED_SECRET, get_http_var('sign'));
        if ($external_auth) {
            $confirm = false;
        } elseif ($this->data['email_verified']) {
            $confirm = false;
        } else {
            $confirm = true;
        }

        // If this goes well, the alert will be added to the database and a confirmation email
        // will be sent to them.
        $success = $this->alert->add($this->data, $confirm);

        if ($success > 0 && !$confirm) {
            $this->data['step'] = '';
            $this->data['mp_step'] = '';
            $result = 'alert-added';
        } elseif ($success > 0) {
            $this->data['step'] = '';
            $this->data['mp_step'] = '';
            $result = 'alert-confirmation';
        } elseif ($success == -2) {
            // we need to make sure we know that the person attempting to sign up
            // for the alert has that email address to stop people trying to work
            // out what alerts they are signed up to
            if ($this->data['email_verified'] || ($this->user->loggedin && $this->user->email() == $this->data['email'])) {
                $result = 'alert-exists';
            } else {
                // don't throw an error message as that implies that they have already signed
                // up for the alert but instead pretend all is normal but send an email saying
                // that someone tried to sign them up for an existing alert
                $result = 'alert-already-signed';
                $this->alert->send_already_signedup_email($this->data);
            }
        } else {
            $result = 'alert-fail';
        }

        // don't need these anymore so get rid of them
        $this->data['keyword'] = '';
        $this->data['pid'] = '';
        $this->data['alertsearch'] = '';
        $this->data['pc'] = '';

        $this->data['results'] = $result;
        $this->data['criteria'] = $this->alert->criteria;
        $this->data['display_criteria'] = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($this->alert->criteria);
    }


    protected function formatSearchTerms() {
        if ($this->data['alertsearch']) {
            $this->data['alertsearch_pretty'] = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($this->data['alertsearch']);
            $this->data['search_text'] = $this->data['alertsearch'];
        } else {
            $this->data['search_text'] = $this->data['keyword'];
        }
    }

    protected function checkForCommonMistakes() {
        $mistakes = [];
        if (strstr($this->data['alertsearch'], ',') > -1) {
            $mistakes['multiple'] = 1;
        }

        if (
            preg_match('#([A-Z]{1,2}\d+[A-Z]? ?\d[A-Z]{2})#i', $this->data['alertsearch'], $m) &&
            strlen($this->data['alertsearch']) > strlen($m[1]) &&
            validate_postcode($m[1])
        ) {
            $this->data['postcode'] = $m[1];
            $mistakes['postcode_and'] = 1;
        }

        $this->data['mistakes'] = $mistakes;
    }

    protected function formatSearchMemberData() {
        if (isset($this->data['postcode'])) {
            try {
                $postcode = $this->data['postcode'];

                $MEMBER = new \MEMBER(['postcode' => $postcode]);
                // move the postcode to the front just to be tidy
                $tidy_alertsearch = $postcode . " " . trim(str_replace("$postcode", "", $this->data['alertsearch']));
                $alertsearch_display = str_replace("$postcode ", "", $tidy_alertsearch);

                $this->data['member_alertsearch'] = str_replace("$postcode", "speaker:" . $MEMBER->person_id, $tidy_alertsearch);
                $this->data['member_displaysearch'] = $alertsearch_display;
                $this->data['member'] = $MEMBER;

                if (isset($this->data['mistakes']['postcode_and'])) {
                    $constituencies = \MySociety\TheyWorkForYou\Utility\Postcode::postcodeToConstituencies($postcode);
                    if (isset($constituencies['SPC'])) {
                        $MEMBER = new \MEMBER(['constituency' => $constituencies['SPC'], 'house' => HOUSE_TYPE_SCOTLAND]);
                        $this->data['scottish_alertsearch'] = str_replace("$postcode", "speaker:" . $MEMBER->person_id, $tidy_alertsearch);
                        $this->data['scottish_member'] = $MEMBER;
                    } elseif (isset($constituencies['WAC'])) {
                        $MEMBER = new \MEMBER(['constituency' => $constituencies['WAC'], 'house' => HOUSE_TYPE_WALES]);
                        $this->data['welsh_alertsearch'] = str_replace("$postcode", "speaker:" . $MEMBER->person_id, $tidy_alertsearch);
                        $this->data['welsh_member'] = $MEMBER;
                    }
                }
            } catch (\MySociety\TheyWorkForYou\MemberException $e) {
                $this->data['member_error'] = 1;
            }
        }

        if ($this->data['pid']) {
            $MEMBER = new \MEMBER(['person_id' => $this->data['pid']]);
            $this->data['pid_member'] = $MEMBER;
        }

        if ($this->data['keyword']) {
            $this->data['display_keyword'] = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($this->data['keyword']);
        }
    }

    protected function setUserData() {
        if (!isset($this->data['criteria'])) {
            $criteria = $this->data['keyword'];
            if (!$this->data['match_all']) {
                $has_or = strpos($criteria, ' OR ') !== false;
                $missing_braces = strpos($criteria, '(') === false;

                if ($has_or && $missing_braces) {
                    $criteria = "($criteria)";
                }
            }
            if ($this->data['search_section']) {
                $criteria .= " section:" . $this->data['search_section'];
            }
            if ($this->data['pid']) {
                $criteria .= " speaker:" . $this->data['pid'];
            }
            $this->getRecentResults($criteria);

            $this->data['criteria'] = $criteria;
            $this->data['display_criteria'] = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($criteria);
        }
        if ($this->data['results'] == 'changes-abandoned') {
            $this->data['members'] = false;
            $this->data['alertsearch'] = '';
        }

        if ($this->data['alertsearch'] && !(isset($this->data['mistakes']['postcode_and']) || $this->data['members'] || $this->data['pid'])) {
            $this->data['step'] = 'define';
            $this->data['words'] = [$this->data['alertsearch']];
            $this->data['keywords'] = [$this->data['alertsearch']];
            $this->data['exclusions'] = '';
            $this->data['representative'] = '';
        } elseif ($this->data['alertsearch'] && ($this->data['members'] || $this->data['pid'])) {
            $this->data['mp_step'] = 'mp_alert';
            $this->data['mp_search'] = [$this->data['alertsearch']];
        } elseif ($this->data['members'] && $this->data['mp_step'] == 'mp_search') {
            $this->data['mp_step'] = '';
        }

        $this->data['current_mp'] = false;
        $this->data['alerts'] = [];
        $this->data['keyword_alerts'] = [];
        $this->data['speaker_alerts'] = [];
        $this->data['spoken_alerts'] = [];
        $this->data['own_member_alerts'] = [];
        $this->data['all_keywords'] = [];
        $this->data['own_mp_criteria'] = '';
        $own_mp_criteria = '';

        if ($this->data['email_verified']) {
            if ($this->user->postcode()) {
                $current_mp = new \MEMBER(['postcode' => $this->user->postcode()]);
                if ($current_mp_alert = !$this->alert->fetch_by_mp($this->data['email'], $current_mp->person_id())) {

                    $this->data['current_mp'] = $current_mp;
                    $own_mp_criteria = sprintf('speaker:%s', $current_mp->person_id());

                }
                $own_mp_criteria = $current_mp->full_name();
                $this->data['own_mp_criteria'] = $own_mp_criteria;
            }
            $this->data['alerts'] = \MySociety\TheyWorkForYou\Utility\Alert::forUser($this->data['email']);
            foreach ($this->data['alerts'] as $alert) {
                if (array_key_exists('spokenby', $alert) and sizeof($alert['spokenby']) == 1 and $alert['spokenby'][0] == $own_mp_criteria) {
                    $this->data['own_member_alerts'][] = $alert;
                } elseif (array_key_exists('spokenby', $alert)) {
                    if (!array_key_exists($alert['spokenby'][0], $this->data['spoken_alerts'])) {
                        $this->data['spoken_alerts'][$alert['spokenby'][0]] = [];
                    }
                    $this->data['spoken_alerts'][$alert['spokenby'][0]][] = $alert;
                }
            }
            foreach ($this->data['alerts'] as $alert) {
                $term = implode(' ', $alert['words']);
                $add = true;
                if (array_key_exists('spokenby', $alert)) {
                    $add = false;
                } elseif (array_key_exists($term, $this->data['spoken_alerts'])) {
                    $add = false;
                    $this->data['all_keywords'][] = $term;
                    $this->data['spoken_alerts'][$term][] = $alert;
                } elseif ($term == $own_mp_criteria) {
                    $add = false;
                    $this->data['all_keywords'][] = $term;
                    $this->data['own_member_alerts'][] = $alert;
                } elseif (\MySociety\TheyWorkForYou\Utility\Search::searchMemberDbLookupWithNames($term, true)) {
                    if (!array_key_exists($term, $this->data['spoken_alerts'])) {
                        $this->data['spoken_alerts'][$term] = [];
                    }
                    $add = false;
                    # need to add this to make it consistent so the front end know where to get the name
                    $alert['spokenby'] = [$term];
                    $this->data['all_keywords'][] = $term;
                    $this->data['spoken_alerts'][$term][] = $alert;
                }
                if ($add) {
                    $this->data['all_keywords'][] = $term;
                    $this->data['keyword_alerts'][] = $alert;
                }
            }
        } else {
            if ($this->data['alertsearch'] && $this->data['pc']) {
                $this->data['mp_step'] = 'mp_alert';
            }
        }
        if (count($this->data['alerts'])) {
            $this->data['delete_token'] = $this->data['alerts'][0]['token'];
        }
        if ($this->data['addword'] != '' || ($this->data['step'] && count($this->data['errors']) > 0)) {
            $this->data["step"] = get_http_var('this_step');
        } else {
            $this->data['this_step'] = '';
        }

        $this->data["search_term"] = $this->data['alertsearch'];
        if ($this->data['mp_search']) {
            $this->data["search_term"] = $this->data['mp_search'];
        }
    }
}

mysociety / theyworkforyou

Pull Request — master (#1846)

Standard::setUserData() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like
