DisplayedMessage

Complexity

Total Complexity

13

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	DisplayedMessage()	0	7	1
A	getContent()	0	3	1
A	setContent()	0	4	1
A	getType()	0	3	1
A	setType()	0	4	1
A	getTimerInterval()	0	3	1
A	setTimerInterval()	0	4	1
A	getDismissable()	0	3	1
A	setDismissable()	0	4	1
A	compile()	0	11	4

<?php
/**
 * Repr�sente un message � afficher dans une alert Bootstrap
* @author jcheron
* @version 1.1
*/
class DisplayedMessage {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

	private $content;
	private $type;
	private $timerInterval;
	private $dismissable;
	private $visible;
	public function DisplayedMessage($content,$type="success",$timerInterval=0,$dismissable=true,$visible=true){

Please use __construct() instead of a PHP4-style constructor that is named after the class.

		$this->content=$content;
		$this->type=$type;
		$this->dismissable=$dismissable;
		$this->timerInterval=$timerInterval;
		$this->visible=$visible;
	}
	public function getContent() {
		return $this->content;
	}
	public function setContent($content) {
		$this->content=$content;
		return $this;
	}
	public function getType() {
		return $this->type;
	}
	public function setType($type) {
		$this->type=$type;
		return $this;
	}
	public function getTimerInterval() {
		return $this->timerInterval;
	}
	public function setTimerInterval($timerInterval) {
		$this->timerInterval=$timerInterval;
		return $this;
	}
	public function getDismissable() {
		return $this->dismissable;
	}
	public function setDismissable($dismissable) {
		$this->dismissable=$dismissable;
		return $this;
	}
	public function compile($jquery){
		$alert=$jquery->bootstrap()->htmlAlert("message",$this->content,$this->type);
		if($this->visible==false){

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

			$alert->setProperty("style","display:none;");
		}
		$alert->setCloseable($this->dismissable);
		if(isset($this->timerInterval) && $this->timerInterval>0){
			$jquery->hide(".alert",$this->timerInterval,'',true);
		}
		return $alert->compile($jquery);
	}
}