KW\Inlagg\Inloggaren

Complexity

Total Complexity

7

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	uppdateraInlogg()	0	7	1
A	__construct()	0	3	1
A	loggain()	0	15	2
A	skapaNyanvandare()	0	7	1
A	anvandarnamnUpptaget()	0	10	2

<?php

namespace KW\Inlagg;

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

class Inloggaren implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;

    public function __construct($di)
    {
        $this->di = $di;
    }

    public function loggain($anvandarnamn, $losenord)
    {

        $db = $this->di->get("db");
        $db->connect();
        $sql = "SELECT anvandarid, anvandarnamn, email FROM anvandare2 WHERE anvandarnamn=? AND losenord=?;";
        $res = $db->executeFetch($sql, [$anvandarnamn, MD5($losenord)]);

        if (is_null($res)) {
            return false;
        }
        $this->di->get("session")->set("anvandarid", $res->anvandarid);
        $this->di->get("session")->set("anvandarnamn", $res->anvandarnamn);
        $this->di->get("session")->set("email", $res->email);
        return true;
    }


    public function anvandarnamnUpptaget($anvandarnamn)
    {
        $db = $this->di->get("db");
        $db->connect();
        $sql = "SELECT * FROM anvandare2 WHERE anvandarnamn=?;";
        $res = $db->executeFetch($sql, [$anvandarnamn]);
        if ($res == null) {
            return false;
        }
        return true;
    }

    public function skapaNyanvandare($anvandarnamn, $losenord, $email)
    {
        $db = $this->di->get("db");
        $db->connect();
        $sql = "INSERT INTO `anvandare2` (`anvandarnamn`, `losenord`, `email`, `fraga`, `svar`, `kommentar`, `rfraga`, `rsvar`, `rkommentar`) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?);";
        $db->execute($sql, [$anvandarnamn, MD5($losenord), $email, 0, 0 ,0 ,0 ,0 ,0]);
        return;
    }

    public function uppdateraInlogg($typ, $varde)
    {
        $db = $this->di->get("db");
        $db->connect();
        $sql = 'UPDATE anvandare2 SET ' . $typ . ' = "'  . $varde . '" WHERE anvandarid = ?;';
        $db->execute($sql, [$this->di->session->get("anvandarid")]);

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

        return;
    }
}