EvolutionCMS\Event

Complexity

Total Complexity

11

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	alert()	0	14	5
A	output()	0	4	1
A	setOutput()	0	4	1
A	getOutput()	0	4	1
A	stopPropagation()	0	4	1
A	_resetEventObject()	0	8	1

<?php namespace EvolutionCMS;

class Event implements Interfaces\EventInterface

The property $_propagate is not named in camelCase.

The property $_output is not named in camelCase.

{
    public $name = '';
    public $_propagate = true;
    /**
     * @deprecated use setOutput(), getOutput()
     * @var string
     */
    public $_output;
    public $activated = false;
    public $activePlugin = '';
    public $params = array();

    /**
     * @param string $name Name of the event
     */
    public function __construct($name = "")
    {
        $this->_resetEventObject();
        $this->name = $name;
    }

    /**
     * Display a message to the user
     *
     * @global array $SystemAlertMsgQueque
     * @param string $msg The message
     */
    public function alert($msg)
    {
        global $SystemAlertMsgQueque;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

        if ($msg == "") {
            return;
        }
        if (is_array($SystemAlertMsgQueque)) {
            $title = '';
            if ($this->name && $this->activePlugin) {
                $title = "<div><b>" . $this->activePlugin . "</b> - <span style='color:maroon;'>" . $this->name . "</span></div>";
            }
            $SystemAlertMsgQueque[] = "$title<div style='margin-left:10px;margin-top:3px;'>$msg</div>";
        }
    }

    /**
     * Output
     *
     * @param string $msg
     * @deprecated see addOutput
     */
    public function output($msg)
    {
        $this->_output .= $msg;

The property EvolutionCMS\Event::$_output has been deprecated with message: use setOutput(), getOutput()

    }

    /**
     * @param mixed $data
     */
    public function setOutput($data)
    {
        $this->_output = $data;

The property EvolutionCMS\Event::$_output has been deprecated with message: use setOutput(), getOutput()

    }

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use string.

     */
    public function getOutput()
    {
        return $this->_output;

The property EvolutionCMS\Event::$_output has been deprecated with message: use setOutput(), getOutput()

    }

    /**
     * Stop event propogation
     */
    public function stopPropagation()
    {
        $this->_propagate = false;
    }

    public function _resetEventObject()
    {
        unset ($this->returnedValues);
        $this->name = "";
        $this->setOutput(null);
        $this->_propagate = true;
        $this->activated = false;
    }
}