ControllerBase - Code Metrics - Luctum/Increase - Measure and Improve Code Quality continuously with Scrutinizer

ControllerBase B
last analyzed 2016-06-05 16:11 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	285
Duplicated Lines	3.16 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	24
Bugs	0	Features	2

Metric	Value
wmc	51
c	24
b	0
f	2
lcom	1
cbo	6
dl	9
loc	285
rs	8.3206

24 Methods

Rating	Name	Duplication	Size	Complexity
A	afterExecuteRoute()	0	7	1
A	frmAction()	0	3	1
A	indexAction()	0	19	4
A	getInstance()	0	10	2
A	readAction()	0	12	3
A	setValuesToObject()	0	4	1
B	updateAction()	0	27	6
B	soloUpdateAction()	0	14	5
A	deleteAction()	9	9	2
A	asAdminAction()	0	6	1
A	asUserAction()	0	6	1
A	logoutAction()	0	5	1
B	loadAclAction()	0	30	5
A	verifyAccessAction()	0	15	3
A	moreAction()	0	3	1
A	_showDisplayedMessage()	0	4	1
A	_showMessage()	0	5	1
A	messageSuccess()	0	4	1
A	messageWarning()	0	4	1
A	messageDanger()	0	4	1
A	messageInfo()	0	4	1
A	isAdmin()	0	8	2
A	isActual()	0	7	3
A	isAdminAndActual()	0	7	3

How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class

Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.

Complex classes like ControllerBase often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use ControllerBase, and based on these observations, apply Extract Interface, too.

<?php

use Phalcon\Mvc\Controller;

use Phalcon\Acl\Adapter\Memory as AclList;
use Phalcon\Acl\Role;
use Phalcon\Acl\Resource;
use Phalcon\Mvc\Url;

class ControllerBase extends Controller
namespace YourVendor;

class YourClass { }
{

    protected $model;
    protected $title;
    protected $controller;
    protected $messageTimerInterval = 3000;

    public function afterExecuteRoute($dispatcher)
    {
        $baseUrl = $this->baseUrl;
        $this->view->setVar("baseUrl", $baseUrl);
        $this->view->setVar("controller", $this->controller);
        $this->view->setVar("title", $this->title);
    }

    public function frmAction(){

    }

    public function indexAction($message = NULL)
    {
    if ($this->verifyAccessAction($this->controller, "index")) {
        $msg = "";
        if (isset($message)) {
            if (is_string($message)) {
                $message = new DisplayedMessage($message);
            }
            $message->setTimerInterval($this->messageTimerInterval);
            $msg = $this->_showDisplayedMessage($message);
        }
        $this->view->setVar("msg", $msg);
        $objects = call_user_func($this->model . "::find");
        $this->view->setVar("objects", $objects);
        $this->view->pick("main/index");
        } else {
            $this->view->pick("main/error");
    	}
    }

    public function getInstance($id = NULL)
    {
        if (isset($id)) {
            $object = call_user_func($this->model . "::findfirst", $id);
        } else {
            $className = $this->model;
            $object = new $className();
        }
        return $object;
    }

    public function readAction($id = NULL)
    {
        if ($this->verifyAccessAction($this->controller, "read")) {
            if ($id != null) {
                $object = call_user_func($this->model . '::find', "id = $id");
                $this->view->setVar("object", $object);
                $this->view->pick("main/read");
            }
        }else{
if (rand(1, 6) > 3) {
print "Check failed";
} else {
    //print "Check succeeded";
}

        }
    }


    protected function setValuesToObject(&$object)
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

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

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        $object->assign($_POST);
    }


    public function updateAction()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

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

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if ($this->verifyAccessAction($this->controller, "write")) {

            $id = $this->request->getPost('id', 'int');
            if ($this->request->isPost()) {
                $object = $this->getInstance(@$_POST["id"]);
                $this->setValuesToObject($object);
                if ($id) {
                    try {
                        $object->save();
                        $msg = new DisplayedMessage("Instance de " . $this->model . " modifiée");
                    } catch (\Exception $e) {
                        $msg = new DisplayedMessage("Impossible d'ajouter l'instance de " . $this->model, "danger : $e");
                    }
                } else {
                    try {
                        $object->save();
                        $msg = new DisplayedMessage("Instance de " . $this->model . " ajoutée");
                    } catch (\Exception $e) {
                        $msg = new DisplayedMessage("Impossible d'ajouter l'instance de " . $this->model, "danger : $e");
                    }
                }
                $this->dispatcher->forward(array("controller" => $this->dispatcher->getControllerName(), "action" => "index", "params" => array($msg)));
            }
        }
    }

    //PErmet l'édition d'un seul champ à la fois
    public function soloUpdateAction()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

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

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if ($this->verifyAccessAction($this->controller, "write")) {
            if($this->session->has("user") && ($this->session->get("user")->getId() == $this->request->getPost('pk', 'int')) || $this->session->get("user")->getIdTypeUser() == 0){
                $name = $this->request->getPost('name', 'string');
                //Créer la fonction variable 'set' en fonction du name en POST
                $func = 'set' . ucfirst($name);
                $projet = call_user_func($this->model . '::findFirst', $_POST['pk']);
                $projet->$func($_POST['value']);
                $projet->save();
            }
        }

    }


    public function deleteAction($id = null)

    {
        if ($this->verifyAccessAction($this->controller, "write")) {
            $object = call_user_func($this->model . '::findFirst', "$id");
            $object->delete();
        }

        $this->response->redirect("$this->controller/index");
    }

    public function asAdminAction()
    {
        $user = User::findFirst("id=3");
        $this->session->set("user", $user);
        $this->response->redirect("$this->controller/index");
    }

    public function asUserAction()
    {
        $user = User::findFirst("id=1");
        $this->session->set("user", $user);
        $this->response->redirect("$this->controller/index");
    }

    public function logoutAction()
    {
        $this->session->destroy();
        $this->response->redirect("Index/index");
    }    
    
    public function loadAclAction($typeUser) {
    	$acl = new AclList();
    	$acl->setDefaultAction(Phalcon\Acl::DENY);
    	
    	$roles = TypeUser::find();
    	foreach ($roles as $role) {
    		$acl->addRole($role->getLibelle());
    	}

    	$operationsBdd = Operation::find();
    	$operations = array();
    	foreach ($operationsBdd as $operation) {
    		$operations[] = $operation->getOperation();
    	}
    	
    	$ressources = Ressource::find();
    	foreach ($ressources as $ressource) {
    		$acl->addResource($ressource->getLibelle(), $operations);
    	}
 
    	$aclsBdd = Acl::find();
    	foreach ($aclsBdd as $aclBdd) {
    		$typeUserBdd = TypeUser::findFirst("id = ".$aclBdd->getIdTypeUser());
    		$ressourceBdd = Ressource::findFirst("id = ".$aclBdd->getIdRessource());
    		$operationBdd = Operation::findFirst("id = ".$aclBdd->getIdOperation());
    		$acl->allow($typeUserBdd->getLibelle(), $ressourceBdd->getLibelle(), $operationBdd->getOperation());
    	}
    	
    	return $acl;
    }
    
    public function verifyAccessAction($activeResource, $activeOperation) {
        if($this->session->has("user")){
            $user = $this->session->get("user");
            $typeUser = TypeUser::findFirst("id = ".$user->getIdTypeUser());
            $typeUserSession = $user->getIdTypeUser();
            $acl = $this->loadAclAction($typeUserSession);
            if ($acl->isAllowed($typeUser->getLibelle(), $activeResource, $activeOperation)) {
                return 1;
            } else {
                return 0;
            }
        }else{
            return 0;
        }
    }


    public function moreAction(){
        $this->view->pick("main/more");
    }

    /**
     * Affiche un message Alert bootstrap
     * @param DisplayedMessage $message
     */
    public function _showDisplayedMessage($message)
    {
        return $message->compile($this->jquery);
    }

    /**
     * Affiche un message Alert bootstrap
     * @param string $message texte du message
     * @param string $type type du message (info, success, warning ou danger)
     * @param number $timerInterval durée en millisecondes d'affichage du message (0 pour que le message reste affiché)
     * @param string $dismissable si vrai, l'alert dispose d'une croix de fermeture
     */
    public function _showMessage($message, $type = "success", $timerInterval = 0, $dismissable = true, $visible = true)
    {
        $message = new DisplayedMessage($message, $type, $timerInterval, $dismissable, $visible);

        $this->_showDisplayedMessage($message);
    }


    public function messageSuccess($message, $timerInterval = 0, $dismissable = true)
    {
        $this->_showMessage($message, "success", $timerInterval, $dismissable);
    }

    /**
     * Affiche un message Alert bootstrap de type warning
     * @param string $message texte du message
     * @param number $timerInterval durée en millisecondes d'affichage du message (0 pour que le message reste affiché)
     * @param string $dismissable si vrai, l'alert dispose d'une croix de fermeture
     */
    public function messageWarning($message, $timerInterval = 0, $dismissable = true)
    {
        $this->_showMessage($message, "warning", $timerInterval, $dismissable);
    }

    /**
     * Affiche un message Alert bootstrap de type danger
     * @param string $message texte du message
     * @param number $timerInterval durée en millisecondes d'affichage du message (0 pour que le message reste affiché)
     * @param string $dismissable si vrai, l'alert dispose d'une croix de fermeture
     */
    public function messageDanger($message, $timerInterval = 0, $dismissable = true)
    {
        $this->_showMessage($message, "danger", $timerInterval, $dismissable);
    }

    /**
     * Affiche un message Alert bootstrap de type info
     * @param string $message texte du message
     * @param number $timerInterval durée en millisecondes d'affichage du message (0 pour que le message reste affiché)
     * @param string $dismissable si vrai, l'alert dispose d'une croix de fermeture
     */
    public function messageInfo($message, $timerInterval = 0, $dismissable = true)
    {
        $this->_showMessage($message, "info", $timerInterval, $dismissable);
    }


    public function isAdmin($userId){
        $user = User::findFirst($userId);
        if($user->getTypeUser() == 0){

            return true;
        }else{
            return false;
        }
    }

    public function isActual($user){
        if($this->session->has("user") || $user == $this->session->get("user")->getId()){

            return true;
        }else{
            return false;
        }
    }

    public function isAdminAndActual($user){
        if($this->isAdmin($user) || $this->isActual($user)){

            return true;
        }else{
            return false;
        }
    }
}


1		<?php
2
3		use Phalcon\Mvc\Controller;
4
5		use Phalcon\Acl\Adapter\Memory as AclList;
6		use Phalcon\Acl\Role;
7		use Phalcon\Acl\Resource;
8		use Phalcon\Mvc\Url;
9
10		class ControllerBase extends Controller
		0 ignored issues – show Coding Style Compatibility introduced 2015-12-01 16:27 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
11		{
12
13		protected $model;
14		protected $title;
15		protected $controller;
16		protected $messageTimerInterval = 3000;
17
18		public function afterExecuteRoute($dispatcher)
19		{
20		$baseUrl = $this->baseUrl;
21		$this->view->setVar("baseUrl", $baseUrl);
22		$this->view->setVar("controller", $this->controller);
23		$this->view->setVar("title", $this->title);
24		}
25
26		public function frmAction(){
27
28		}
29
30		public function indexAction($message = NULL)
31		{
32		if ($this->verifyAccessAction($this->controller, "index")) {
33		$msg = "";
34		if (isset($message)) {
35		if (is_string($message)) {
36		$message = new DisplayedMessage($message);
37		}
38		$message->setTimerInterval($this->messageTimerInterval);
39		$msg = $this->_showDisplayedMessage($message);
40		}
41		$this->view->setVar("msg", $msg);
42		$objects = call_user_func($this->model . "::find");
43		$this->view->setVar("objects", $objects);
44		$this->view->pick("main/index");
45		} else {
46		$this->view->pick("main/error");
47		}
48		}
49
50		public function getInstance($id = NULL)
51		{
52		if (isset($id)) {
53		$object = call_user_func($this->model . "::findfirst", $id);
54		} else {
55		$className = $this->model;
56		$object = new $className();
57		}
58		return $object;
59		}
60
61		public function readAction($id = NULL)
62		{
63		if ($this->verifyAccessAction($this->controller, "read")) {
64		if ($id != null) {
65		$object = call_user_func($this->model . '::find', "id = $id");
66		$this->view->setVar("object", $object);
67		$this->view->pick("main/read");
68		}
69		}else{
		0 ignored issues – show Unused Code introduced 2015-12-17 01:33 UTC by Report Bug Copy Issue Report This `else` statement is empty and can be removed. This check looks for the `else` branches of `if` statements that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete. These `else` branches can be removed. if (rand(1, 6) > 3) { print "Check failed"; } else { //print "Check succeeded"; } could be turned into if (rand(1, 6) > 3) { print "Check failed"; } This is much more concise to read. Loading history...
70
71		}
72		}
73
74
75		protected function setValuesToObject(&$object)
		0 ignored issues – show Coding Style introduced 2015-12-01 16:27 UTC by Report Bug Copy Issue Report `setValuesToObject` uses the super-global variable `$_POST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
76		{
77		$object->assign($_POST);
78		}
79
80
81		public function updateAction()
		0 ignored issues – show Coding Style introduced 2015-12-01 16:27 UTC by Report Bug Copy Issue Report `updateAction` uses the super-global variable `$_POST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
82		{
83		if ($this->verifyAccessAction($this->controller, "write")) {
84
85		$id = $this->request->getPost('id', 'int');
86		if ($this->request->isPost()) {
87		$object = $this->getInstance(@$_POST["id"]);
88		$this->setValuesToObject($object);
89		if ($id) {
90		try {
91		$object->save();
92		$msg = new DisplayedMessage("Instance de " . $this->model . " modifiée");
93		} catch (\Exception $e) {
94		$msg = new DisplayedMessage("Impossible d'ajouter l'instance de " . $this->model, "danger : $e");
95		}
96		} else {
97		try {
98		$object->save();
99		$msg = new DisplayedMessage("Instance de " . $this->model . " ajoutée");
100		} catch (\Exception $e) {
101		$msg = new DisplayedMessage("Impossible d'ajouter l'instance de " . $this->model, "danger : $e");
102		}
103		}
104		$this->dispatcher->forward(array("controller" => $this->dispatcher->getControllerName(), "action" => "index", "params" => array($msg)));
105		}
106		}
107		}
108
109		//PErmet l'édition d'un seul champ à la fois
110		public function soloUpdateAction()
		0 ignored issues – show Coding Style introduced 2015-12-07 11:16 UTC by Report Bug Copy Issue Report `soloUpdateAction` uses the super-global variable `$_POST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
111		{
112		if ($this->verifyAccessAction($this->controller, "write")) {
113		if($this->session->has("user") && ($this->session->get("user")->getId() == $this->request->getPost('pk', 'int')) \|\| $this->session->get("user")->getIdTypeUser() == 0){
114		$name = $this->request->getPost('name', 'string');
115		//Créer la fonction variable 'set' en fonction du name en POST
116		$func = 'set' . ucfirst($name);
117		$projet = call_user_func($this->model . '::findFirst', $_POST['pk']);
118		$projet->$func($_POST['value']);
119		$projet->save();
120		}
121		}
122
123		}
124
125
126	View Code Duplication	public function deleteAction($id = null)
		0 ignored issues – show Duplication introduced 2015-12-16 22:35 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
127		{
128		if ($this->verifyAccessAction($this->controller, "write")) {
129		$object = call_user_func($this->model . '::findFirst', "$id");
130		$object->delete();
131		}
132
133		$this->response->redirect("$this->controller/index");
134		}
135
136		public function asAdminAction()
137		{
138		$user = User::findFirst("id=3");
139		$this->session->set("user", $user);
140		$this->response->redirect("$this->controller/index");
141		}
142
143		public function asUserAction()
144		{
145		$user = User::findFirst("id=1");
146		$this->session->set("user", $user);
147		$this->response->redirect("$this->controller/index");
148		}
149
150		public function logoutAction()
151		{
152		$this->session->destroy();
153		$this->response->redirect("Index/index");
154		}
155
156		public function loadAclAction($typeUser) {
157		$acl = new AclList();
158		$acl->setDefaultAction(Phalcon\Acl::DENY);
159
160		$roles = TypeUser::find();
161		foreach ($roles as $role) {
162		$acl->addRole($role->getLibelle());
163		}
164
165		$operationsBdd = Operation::find();
166		$operations = array();
167		foreach ($operationsBdd as $operation) {
168		$operations[] = $operation->getOperation();
169		}
170
171		$ressources = Ressource::find();
172		foreach ($ressources as $ressource) {
173		$acl->addResource($ressource->getLibelle(), $operations);
174		}
175
176		$aclsBdd = Acl::find();
177		foreach ($aclsBdd as $aclBdd) {
178		$typeUserBdd = TypeUser::findFirst("id = ".$aclBdd->getIdTypeUser());
179		$ressourceBdd = Ressource::findFirst("id = ".$aclBdd->getIdRessource());
180		$operationBdd = Operation::findFirst("id = ".$aclBdd->getIdOperation());
181		$acl->allow($typeUserBdd->getLibelle(), $ressourceBdd->getLibelle(), $operationBdd->getOperation());
182		}
183
184		return $acl;
185		}
186
187		public function verifyAccessAction($activeResource, $activeOperation) {
188		if($this->session->has("user")){
189		$user = $this->session->get("user");
190		$typeUser = TypeUser::findFirst("id = ".$user->getIdTypeUser());
191		$typeUserSession = $user->getIdTypeUser();
192		$acl = $this->loadAclAction($typeUserSession);
193		if ($acl->isAllowed($typeUser->getLibelle(), $activeResource, $activeOperation)) {
194		return 1;
195		} else {
196		return 0;
197		}
198		}else{
199		return 0;
200		}
201		}
202
203
204		public function moreAction(){
205		$this->view->pick("main/more");
206		}
207
208		/**
209		* Affiche un message Alert bootstrap
210		* @param DisplayedMessage $message
211		*/
212		public function _showDisplayedMessage($message)
213		{
214		return $message->compile($this->jquery);
215		}
216
217		/**
218		* Affiche un message Alert bootstrap
219		* @param string $message texte du message
220		* @param string $type type du message (info, success, warning ou danger)
221		* @param number $timerInterval durée en millisecondes d'affichage du message (0 pour que le message reste affiché)
222		* @param string $dismissable si vrai, l'alert dispose d'une croix de fermeture
223		*/
224		public function _showMessage($message, $type = "success", $timerInterval = 0, $dismissable = true, $visible = true)
225		{
226		$message = new DisplayedMessage($message, $type, $timerInterval, $dismissable, $visible);
		0 ignored issues – show Bug introduced 2016-06-05 16:11 UTC by Report Bug Copy Issue Report It seems like `$dismissable` defined by parameter `$dismissable` on line 224 can also be of type `string`; however, `DisplayedMessage::DisplayedMessage()` does only seem to accept `boolean`, maybe add an additional type check? This check looks at variables that have been passed in as parameters and are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
227		$this->_showDisplayedMessage($message);
228		}
229
230
231		public function messageSuccess($message, $timerInterval = 0, $dismissable = true)
232		{
233		$this->_showMessage($message, "success", $timerInterval, $dismissable);
234		}
235
236		/**
237		* Affiche un message Alert bootstrap de type warning
238		* @param string $message texte du message
239		* @param number $timerInterval durée en millisecondes d'affichage du message (0 pour que le message reste affiché)
240		* @param string $dismissable si vrai, l'alert dispose d'une croix de fermeture
241		*/
242		public function messageWarning($message, $timerInterval = 0, $dismissable = true)
243		{
244		$this->_showMessage($message, "warning", $timerInterval, $dismissable);
245		}
246
247		/**
248		* Affiche un message Alert bootstrap de type danger
249		* @param string $message texte du message
250		* @param number $timerInterval durée en millisecondes d'affichage du message (0 pour que le message reste affiché)
251		* @param string $dismissable si vrai, l'alert dispose d'une croix de fermeture
252		*/
253		public function messageDanger($message, $timerInterval = 0, $dismissable = true)
254		{
255		$this->_showMessage($message, "danger", $timerInterval, $dismissable);
256		}
257
258		/**
259		* Affiche un message Alert bootstrap de type info
260		* @param string $message texte du message
261		* @param number $timerInterval durée en millisecondes d'affichage du message (0 pour que le message reste affiché)
262		* @param string $dismissable si vrai, l'alert dispose d'une croix de fermeture
263		*/
264		public function messageInfo($message, $timerInterval = 0, $dismissable = true)
265		{
266		$this->_showMessage($message, "info", $timerInterval, $dismissable);
267		}
268
269
270		public function isAdmin($userId){
271		$user = User::findFirst($userId);
272		if($user->getTypeUser() == 0){
		0 ignored issues – show Coding Style introduced 2015-12-17 01:33 UTC by Report Bug Copy Issue Report The `if`-`else` statement can be simplified to `return $user->getTypeUser() == 0;`. Loading history...
273		return true;
274		}else{
275		return false;
276		}
277		}
278
279		public function isActual($user){
280		if($this->session->has("user") \|\| $user == $this->session->get("user")->getId()){
		0 ignored issues – show Coding Style introduced 2015-12-17 01:33 UTC by Report Bug Copy Issue Report The `if`-`else` statement can be simplified to `return $this->session->h...->get('user')->getId();`. Loading history...
281		return true;
282		}else{
283		return false;
284		}
285		}
286
287		public function isAdminAndActual($user){
288		if($this->isAdmin($user) \|\| $this->isActual($user)){
		0 ignored issues – show Coding Style introduced 2015-12-17 01:33 UTC by Report Bug Copy Issue Report The `if`-`else` statement can be simplified to `return $this->isAdmin($u...$this->isActual($user);`. Loading history...
289		return true;
290		}else{
291		return false;
292		}
293		}
294		}
295

Luctum / Increase

ControllerBase B last analyzed 2016-06-05 16:11 UTC