Jasny\Controller\Session\Flash

Complexity

Total Complexity

16

Size/Duplication

Total Lines	118
Duplicated Lines	7.63 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	isIssued()	0	4	1
A	set()	0	4	1
A	get()	4	9	4
A	reissue()	5	8	3
A	clear()	0	5	1
A	getType()	0	5	2
A	getMessage()	0	5	2
A	__toString()	0	4	1

<?php

namespace Jasny\Controller\Session;

/**
 * Class for the flash message
 */
class Flash
{
    /**
     * @var array
     */
    protected $data;
    
    /**
     * @var array|\ArrayObject
     */
    protected $session;

    /**
     * Session key for flash
     * @var string
     */
    protected $key = 'flash';
    
    
    /**
     * Class constructor
     * 
     * @param array|\ArrayObject $session
     */
    public function __construct(&$session)
    {
        $this->session =& $session;
    }
    
    /**
     * Check if the flash is set.
     * 
     * @return boolean
     */
    public function isIssued()
    {
        return isset($this->session[$this->key]);
    }
    
    /**
     * Set the flash.
     * 
     * @param string $type     flash type, eg. 'error', 'notice' or 'success'
     * @param mixed  $message  flash message
     */
    public function set($type, $message)
    {
        $this->session[$this->key] = compact('type', 'message');
    }
    
    /**
     * Get the flash.
     * 
     * @return object
     */
    public function get()
    {
        if (!isset($this->data) && isset($this->session[$this->key])) {

This code seems to be duplicated across your project.

            $this->data = $this->session[$this->key];
            unset($this->session[$this->key]);
        }
        
        return $this->data ? (object)$this->data : null;
    }
    
    /**
     * Reissue the flash.
     */
    public function reissue()
    {
        if (!isset($this->data) && isset($this->session[$this->key])) {

This code seems to be duplicated across your project.

            $this->data = $this->session[$this->key];
        } else {
            $this->session[$this->key] = $this->data;
        }
    }
    
    /**
     * Clear the flash.
     */
    public function clear()
    {
        $this->data = null;

It seems like null of type null is incompatible with the declared type array of property $data.

        unset($this->session[$this->key]);
    }     
    
    /**
     * Get the flash type
     * 
     * @return string
     */
    public function getType()
    {
        $data = $this->get();
        return isset($data) ? $data->type : null;
    }
    
    /**
     * Get the flash message
     * 
     * @return string
     */
    public function getMessage()
    {
        $data = $this->get();
        return isset($data) ? $data->message : null;
    }
    
    /**
     * Cast object to string
     * 
     * @return string
     */
    public function __toString()
    {
        return (string)$this->getMessage();
    }
}