Backend

Complexity

Total Complexity

18

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	digestTest()	0	12	2
A	loading()	0	4	1
C	help()	0	75	15

<?php
class Backend extends Handler {
    public function loading() {
        header("Content-type: text/html");
        print __("Loading, please wait...")." ".
            "<img src='images/indicator_tiny.gif'>";
    }

    public function digestTest() {
        if (isset($_SESSION['uid'])) {
            header("Content-type: text/html");

            $rv = Digest::prepare_headlines_digest($_SESSION['uid'], 1, 1000);

            print "<h1>HTML</h1>";
            print $rv[0];
            print "<h1>Plain text</h1>";
            print "<pre>".$rv[3]."</pre>";
        } else {
            print error_json(6);
        }
    }

    public function help() {
        $topic = clean_filename($_REQUEST["topic"]); // only one for now

        if ($topic == "main") {
            $info = get_hotkeys_info();
            $imap = get_hotkeys_map();
            $omap = array();

            foreach ($imap[1] as $sequence => $action) {
                if (!isset($omap[$action])) {
                    $omap[$action] = array();
                }

                array_push($omap[$action], $sequence);
            }

            print "<ul class='panel panel-scrollable hotkeys-help' style='height : 300px'>";

            $cur_section = "";
            foreach ($info as $section => $hotkeys) {

                if ($cur_section) {
                    print "<li>&nbsp;</li>";
                }
                print "<li><h3>".$section."</h3></li>";
                $cur_section = $section;

                foreach ($hotkeys as $action => $description) {

                    if (is_array($omap[$action])) {
                        foreach ($omap[$action] as $sequence) {
                            if (strpos($sequence, "|") !== false) {
                                $sequence = substr($sequence,
                                    strpos($sequence, "|") + 1,
                                    strlen($sequence));
                            } else {
                                $keys = explode(" ", $sequence);

                                for ($i = 0; $i < count($keys); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

                                    if (strlen($keys[$i]) > 1) {
                                        $tmp = '';
                                        foreach (str_split($keys[$i]) as $c) {
                                            switch ($c) {
                                            case '*':
                                                $tmp .= __('Shift').'+';
                                                break;
                                            case '^':
                                                $tmp .= __('Ctrl').'+';
                                                break;
                                            default:
                                                $tmp .= $c;
                                            }
                                        }
                                        $keys[$i] = $tmp;
                                    }
                                }
                                $sequence = join(" ", $keys);
                            }

                            print "<li>";
                            print "<div class='hk'><code>$sequence</code></div>";
                            print "<div class='desc'>$description</div>";
                            print "</li>";
                        }
                    }
                }
            }

            print "</ul>";
        }

        print "<footer class='text-center'>";
        print "<button dojoType='dijit.form.Button'
			onclick=\"return dijit.byId('helpDlg').hide()\">".__('Close this window')."</button>";
        print "</footer>";

    }
}