GLOSSARY::delete() - Code Metrics - Inspection of "Ignore dot in URL, so e.g. index.php not matched." - mysociety/theyworkforyou - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions
Branch — master (3ce7e2)

by Nick
created 2018-06-21 14:13 UTC
GLOSSARY::delete() A

↳ Parent: GLOSSARY
Complexity

Conditions	2
Paths	2
Size

Total Lines	8
Code Lines	4
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	0
CRAP Score	6
Importance

Changes
Metric	Value
cc	2
eloc	4
c	0
b	0
f	0
nc	2
nop	1
dl	0
loc	8
rs	9.4285
ccs	0
cts	5
cp	0
crap	6
<?php


/*

The Glossary item handles:
    1. Search matching for particular items.
    2. Addition of glossary items
    3. Removal of glossary items
    4. Notification of pending glossary additions

Glossary items can only (at present) be added on the Search page,
and only in the event that the term has not already been defined.

[?] will it be possible to amend the term?

It should not be possible to add a term if no results are found during the search.

All Glossary items need to be confirmed by a moderator (unless posted by a moderator).
As they are being approved/declined they can be modified (spelling etc...).

*/

// This handles basic insertion and approval functions for all epobjects
include_once INCLUDESPATH."easyparliament/searchengine.php";


class GLOSSARY {
namespace YourVendor;

class YourClass { }

    public $num_terms;			// how many glossary entries do we have
                            // (changes depending on how GLOSSARY is called
    public $hansard_count;		// how many times does the phrase appear in hansard?
    public $query;				// search term
    public $glossary_id;		// if this is set then we only have 1 glossary term
    public $current_term;		// will only be set if we have a valid epobject_id
    public $current_letter;

    // constructor...
    public function __construct($args=array()) {
    // We can optionally start the glossary with one of several arguments
    //		1. glossary_id - treat the glossary as a single term
    //		2. glossary_term - search within glossary for a term
    // With no argument it will pick up all items.

            $this->db = new ParlDB;


            $this->replace_order = array();

            if (isset($args['s']) && ($args['s'] != "")) {
                $args['s'] = urldecode($args['s']);
                $this->search_glossary($args);
            }
        $got = $this->get_glossary_item($args);
            if ($got && isset($args['sort']) && ($args['sort'] == 'regexp_replace')) {
                // We need to sort the terms in the array by "number of words in term".
                // This way, "prime minister" gets dealt with before "minister" when generating glossary links.

                // sort by number of words
                foreach ($this->terms as $glossary_id => $term) {
                    $this->replace_order[$glossary_id] = count(explode(" ", $term['title']));
                }
                arsort($this->replace_order);

                // secondary sort for number of letters?
                // pending functionality...

                // We can either turn off the "current term" completely -
                // so that it never links to its own page,
                // Or we can handle it in $this->glossarise below
                /*

                if (isset($this->epobject_id)) {
                    unset ($this->replace_order[$this->epobject_id]);
                }
                */
            }

            // These stop stupid submissions.
            // everything should be lowercase.
            $this->stopwords = array( "the", "of", "to", "and", "for", "in", "a", "on", "is", "that", "will", "secretary", "are", "ask", "state", "have", "be", "has", "by", "with", "i", "not", "what", "as", "it", "hon", "he", "which", "from", "if", "been", "this", "s", "we", "at", "government", "was", "my", "an", "department", "there", "make", "or", "made", "their", "all", "but", "they", "how", "debate" );


    }

    public function get_glossary_item($args=array()) {
        // Search for and fetch glossary item with title or glossary_id
        // We could also search glossary text that contains the title text, for cross references

        $this->alphabet = array();

        foreach (range ("A", "Z") as $letter) {
            $this->alphabet[$letter] = array();
        }

        $q = $this->db->query("SELECT g.glossary_id, g.title, g.body, u.user_id, u.firstname, u.lastname
            FROM editqueue AS eq, glossary AS g, users AS u
            WHERE g.glossary_id=eq.glossary_id AND u.user_id=eq.user_id AND g.visible=1 AND eq.approved=1
            ORDER by g.title");
        if ($q->success() && $q->rows()) {
            for ($i=0; $i < $q->rows(); $i++) {
                $this->terms[ $q->field($i,"glossary_id") ] = $q->row($i);

                // Now add the epobject to the alphabet navigation.
                $first_letter = strtoupper(substr($q->field($i,"title"),0,1));
                $this->alphabet[$first_letter][] = $q->field($i,"glossary_id");
            }

            $this->num_terms = $q->rows();

            // If we were given a glossary_id, then we need one term in particular,
            // as well as knowing the next and previous terms for the navigation
            if (isset($args['glossary_id']) && ($args['glossary_id'] != "")) {
                $next = 0; $first_term = null;
                foreach ($this->terms as $term) {
                    if (!$first_term) $first_term = $term;
                    $last_term = $term;
                    if ($next == 1) {
                        $this->next_term = $term;

                        break;
                    }
                    elseif ($term['glossary_id'] == $args['glossary_id']) {
                        $this->glossary_id = $args['glossary_id'];
                        $this->current_term = $term;
                        $next = 1;

                    }
                    else {
                        $this->previous_term = $term;

                    }
                }
                // The first term in the list has no previous, so we'll make it the last term
                if (!isset($this->previous_term)) {
                    $this->previous_term = $last_term;

                }
                // and the last has no next, so we'll make it the first
                if (!isset($this->next_term)) {
                    $this->next_term = $first_term;
                }
            }

            return ($this->num_terms);
        }
        else {
            return false;
        }
    }

    public function search_glossary($args=array()) {
        // Search for and fetch glossary item with a title
        // Useful for the search page, and nowhere else (so far)

        $this->query = addslashes($args['s']);
        $this->search_matches = array();

        $this->num_search_matches = 0;


        $query = "SELECT g.glossary_id, g.title, g.body, u.user_id, u.firstname, u.lastname
            FROM editqueue AS eq, glossary AS g, users AS u
            WHERE g.glossary_id=eq.glossary_id AND u.user_id=eq.user_id AND g.visible=1
                AND g.title LIKE '%" . $this->query . "%'
            ORDER by g.title";
        $q = $this->db->query($query);
        if ($q->success() && $q->rows()) {
            for ($i=0; $i < $q->rows(); $i++) {
                $this->search_matches[ $q->field($i,"glossary_id") ] = $q->row($i);
            }
            $this->num_search_matches = $q->rows();
        }
    }

    public function create(&$data) {
        // Add a Glossary definition.
        // Sets visiblity to 0, and awaits moderator intervention.
        // For this we need to start up an epobject of type 2 and then an editqueue item
        // where editqueue.epobject_id_l = epobject.epobject_id

        $EDITQUEUE = new \MySociety\TheyWorkForYou\GlossaryEditQueue;

        // Assuming that everything is ok, we will need:
        // For epobject:
        // 		title VARCHAR(255),

        // 		body TEXT,
        // 		type INTEGER,
        // 		created DATETIME,
        // 		modified DATETIME,
        // and for editqueue:
        //		edit_id INTEGER PRIMARY KEY NOT NULL,
        //		user_id INTEGER,
        //		edit_type INTEGER,
        //		epobject_id_l INTEGER,
        //		title VARCHAR(255),

        //		body TEXT,
        //		submitted DATETIME,
        //		editor_id INTEGER,
        //		approved BOOLEAN,
        //		decided DATETIME

        global $THEUSER;
function myFunction($a, $b) {
    // Do something
}

        if ($data['title'] == '') {
            error ("Sorry, you can't define a term without a title");
            return false;
        }

        if ($data['body'] == '') {
            error ("You haven't entered a definition!");
            return false;
        }

        if (is_numeric($THEUSER->user_id())) {
            // Flood check - make sure the user hasn't just posted a term recently.
            // To help prevent accidental duplicates, among other nasty things.

            $flood_time_limit = 20; // How many seconds until a user can post again?

            $q = $this->db->query("SELECT glossary_id
                            FROM	editqueue
                            WHERE	user_id = '" . $THEUSER->user_id() . "'
                            AND		submitted + 0 > NOW() - $flood_time_limit");

            if ($q->rows() > 0) {
                error("Sorry, we limit people to posting one term per $flood_time_limit seconds to help prevent duplicate postings. Please go back and try again, thanks.");
                return false;
            }
        }

        // OK, let's get on with it...

        // Tidy up the HTML tags
        // (but we don't make URLs into links; only when displaying the comment).
        // We can display Glossary terms the same as the comments
        $data['title'] = filter_user_input($data['title'], 'comment_title'); // In utility.php
        $data['body'] = filter_user_input($data['body'], 'comment'); // In utility.php
        // Add the time and the edit type for the editqueue
        $data['posted'] = date('Y-m-d H:i:s', time());
        $data['edit_type'] = 2;

        // Add the item to the edit queue
        $success = $EDITQUEUE->add($data);

        if ($success) {
            return ($success);
        } else {
            return false;
        }
    }

    public function delete($glossary_id)
    {
        $q = $this->db->query("DELETE from glossary where glossary_id=$glossary_id LIMIT 1;");
        // if that worked, we need to update the editqueue,
        // and remove the term from the already generated object list.
        if ($q->affected_rows() >= 1) {
            unset($this->replace_order[$glossary_id]);
            unset($this->terms[$glossary_id]);
        }
    }

    public function glossarise($body, $tokenize=0, $urlize=0) {
    public function glossarise($body, /** @scrutinizer ignore-unused */ $tokenize=0, $urlize=0) {
    // Turn a body of text into a link-up wonderland of glossary joy

        global $this_page;
function myFunction($a, $b) {
    // Do something
}

        $findwords = array();
        $replacewords = array();
        $URL = new \MySociety\TheyWorkForYou\Url("glossary");
        $URL->insert(array("gl" => ""));

        // External links shown within their own definition
        // should be the complete and linked url.
        // NB. This should only match when $body is a definition beginning with "http:"
        if (is_string($body) && preg_match("/^(http:*[^\s])$/i", $body)) {
            $body = "<a href=\"" . $body . "\" title=\"External link to " . $body . "\">" . $body . "</a>";
            return ($body);
        }

        // otherwise, just replace everything.

        // generate links from URL when wanted
        // NB WRANS is already doing this
        if ($urlize == 1) {
            $body = preg_replace("~(http(s)?:\/\/[^\s\n]*)\b(\/)?~i", "<a href=\"\\0\">\\0</a>", $body);
        }

        // check for any glossary terms to replace
        foreach ($this->replace_order as $glossary_id => $count) {
            if ($glossary_id == $this->glossary_id)
                continue;

            $term_body = $this->terms[$glossary_id]['body'];
            $term_title = $this->terms[$glossary_id]['title'];

            $URL->update(array("gl" => $glossary_id));
            # The regex here ensures that the phrase is only matched if it's not already within <a> tags, preventing double-linking. Kudos to http://stackoverflow.com/questions/7798829/php-regular-expression-to-match-keyword-outside-html-tag-a
            $findwords[$glossary_id] = "/\b(" . $term_title . ")\b(?!(?>[^<]*(?:<(?!\/?a\b)[^<]*)*)<\/a>)/i";
            // catch glossary terms within their own definitions
            if ($glossary_id == $this->glossary_id) {
                $replacewords[] = "<strong>\\1</strong>";
            }
            else {
                if ($this_page == "admin_glossary") {
                    $link_url = "#gl".$glossary_id;
                }
                else {
                    $link_url = $URL->generate('url');
                }
                $title = _htmlentities(trim_characters($term_body, 0, 80));
                $replacewords[] = "<a href=\"$link_url\" title=\"$title\" class=\"glossary\">\\1</a>";
            }
        }
        // Highlight all occurrences of another glossary term in the definition.
        $body = preg_replace($findwords, $replacewords, $body, 1);
        if (isset($this->glossary_id))
            $body = preg_replace("/(?<![>\.\'\/])\b(" . $this->terms[$this->glossary_id]['title'] . ")\b(?![<\'])/i", '<strong>\\1</strong>', $body, 1);

        // Replace any phrases in wikipedia
        // TODO: Merge this code into above, so our gloss and wikipedia
        // don't clash (e.g. URLs getting doubly munged etc.)
        $body = \MySociety\TheyWorkForYou\Utility\Wikipedia::wikipedize($body);

        return ($body);
    }

}

Branch — master (3ce7e2)

GLOSSARY::delete() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

mysociety / theyworkforyou

Branch — master (3ce7e2)

GLOSSARY::delete() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like
