KW\Inlagg\MinsidaController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	indexActionPost()	0	17	3
A	indexActionGet()	0	7	1

<?php

namespace KW\Inlagg;

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

class MinsidaController implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;

    public function indexActionGet()
    {
        $page = $this->di->get("page");
        $page->add("anax/v2/minsida/minsida", [
            ]);
        return $page->render([
            "title"=>"Min sida"
        ]);
    }

    public function indexActionPost()
    {
        $typ = $this->di->get("request")->getPost("knapp");
        $inloggaren = new Inloggaren($this->di);

        if ($typ == "Spara nytt lösenord") {
            $losenord = $this->di->get("request")->getPost("losenord");
            $inloggaren->uppdateraInlogg("losenord", MD5($losenord));
        }

        if ($typ == "Spara ny epostadress") {
            $epost = $this->di->get("request")->getPost("epost");
            $inloggaren->uppdateraInlogg("email", $epost);
            $this->di->session->set("email", $epost);

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

        }

        return $this->di->get("response")->redirect("minsida")->send();
    }
}