KW\Inlagg\FragorController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	taggActionGet()	0	36	3
A	indexActionGet()	0	36	3

<?php

namespace KW\Inlagg;

use Anax\Commons\ContainerInjectableInterface;
use Anax\Commons\ContainerInjectableTrait;

class FragorController implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;

    public function indexActionGet()
    {
        $fragehamtaren = new Fragehamtaren($this->di);
        $sort   = $this->di->get("request")->getGet("sortfr");
        $res    = $fragehamtaren->allaInlagg($sort);

        $taggar     = [];
        $antalsvar  = [];

        foreach ($res as $rad) {
            $tags = $fragehamtaren->hamtaTaggar($rad->id);
            array_push($taggar, $tags);
            $nrsvar = $fragehamtaren->raknaSvar($rad->id);
            array_push($antalsvar, $nrsvar);
        }

        $alltags = $fragehamtaren->allaTaggar();

        $page = $this->di->get("page");
        $page->add("anax/v2/fragor/allafragor", [
            "res"       => $res,
            "taggar"    => $taggar,
            "alltags"   => $alltags,
            "antalsvar" => $antalsvar
            ]);

        if ($this->di->session->get("anvandarid")!=null) {

Accessing session on the interface Psr\Container\ContainerInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            $page->add("anax/v2/fragor/nyfraga", [
                "res"       => $res,
                "taggar"    => $taggar,
                "alltags"   => $alltags
            ]);
        }

        return $page->render([
            "title"=>"Frågor",
        ]);
    }

    public function taggActionGet($tagg)
    {
        $fragehamtaren = new \KW\Inlagg\Fragehamtaren($this->di);
        $sort = $this->di->get("request")->getGet("sortfr");

        $res = $fragehamtaren->fragorMedTaggen($tagg, $sort);
        $taggar     = [];
        $antalsvar  = [];

        foreach ($res as $rad) {
            $tags = $fragehamtaren->hamtaTaggar($rad->id);
            array_push($taggar, $tags);
            $nrsvar = $fragehamtaren->raknaSvar($rad->id);
            array_push($antalsvar, $nrsvar);
        }

        $alltags = $fragehamtaren->allaTaggar();
        $page = $this->di->get("page");
        $page->add("anax/v2/fragor/allafragor", [
            "res"       => $res,
            "taggar"    => $taggar,
            "alltags"   => $alltags,
            "antalsvar" => $antalsvar,
            "tagg"      => $tagg
            ]);

        if ($this->di->session->get("anvandarid")!=null) {

Accessing session on the interface Psr\Container\ContainerInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            $page->add("anax/v2/fragor/nyfraga", [
                "res"       => $res,
                "taggar"    => $taggar,
                "alltags"   => $alltags
            ]);
        }

        return $page->render([
            "title"=>"Frågor",
        ]);
    }
}