Luctum / Increase

app/library/DisplayedMessage.php

Doc Comments +4 added lines

@@ -10,6 +10,10 @@
 	private $timerInterval;
 	private $dismissable;
 	private $visible;
+
+	/**
+	 * @param string $content
+	 */
 	public function DisplayedMessage($content,$type="success",$timerInterval=0,$dismissable=true,$visible=true){
 		$this->content=$content;
 		$this->type=$type;

app/config/services.php

Spacing +10 added lines, -10 removed lines

@@ -20,7 +20,7 @@ 
 /**
  * The URL component is used to generate all kind of urls in the application
  */
-$di->set('url', function () use ($config) {
+$di->set('url', function() use ($config) {
     $url = new UrlResolver();
     $url->setBaseUri($config->application->baseUri);
 
@@ -30,14 +30,14 @@ 
 /**
  * Setting up the view component
  */
-$di->set('view', function () use ($config) {
+$di->set('view', function() use ($config) {
 
     $view = new View();
 
     $view->setViewsDir($config->application->viewsDir);
 
     $view->registerEngines(array(
-        '.volt' => function ($view, $di) use ($config) {
+        '.volt' => function($view, $di) use ($config) {
 
             $volt = new VoltEngine($view, $di);
 
@@ -57,7 +57,7 @@ 
 /**
  * Database connection is created based in the parameters defined in the configuration file
  */
-$di->set('db', function () use ($config) {
+$di->set('db', function() use ($config) {
     return new DbAdapter(array(
         'host' => $config->database->host,
         'username' => $config->database->username,
@@ -70,7 +70,7 @@ 
 });
 
 
-$di->set('baseUrl', function () use ($config) {
+$di->set('baseUrl', function() use ($config) {
     $baseUrl = $config->application->baseUri;
     return $baseUrl;
 });
@@ -78,34 +78,34 @@ 
 /**
  * If the configuration specify the use of metadata adapter use it or use memory otherwise
  */
-$di->set('modelsMetadata', function () {
+$di->set('modelsMetadata', function() {
     return new MetaDataAdapter();
 });
 
 /**
  * Start the session the first time some component request the session service
  */
-$di->set('session', function () {
+$di->set('session', function() {
     $session = new SessionAdapter();
     $session->start();
 
     return $session;
 });
 
-$di->set("jquery", function () {
+$di->set("jquery", function() {
     $jquery = new JsUtils(array("driver" => "Jquery"));
     $jquery->bootstrap(new Bootstrap());
     //$jquery->ui(new Ajax\JqueryUI());//optional for JQuery UI
     return $jquery;
 });
 
-$di->set('dispatcher', function () {
+$di->set('dispatcher', function() {
 
     // Create an event manager
     $eventsManager = new EventsManager();
 
     // Attach a listener for type "dispatch"
-    $eventsManager->attach("dispatch", function ($event, $dispatcher) {
+    $eventsManager->attach("dispatch", function($event, $dispatcher) {
         // ...
     });
 

app/controllers/IndexController.php

Spacing +2 added lines, -2 removed lines

@@ -6,10 +6,10 @@
     protected $title;
     protected $controller;
 
-    public function initialize(){
+    public function initialize() {
         $this->model = "Index";
         $this->title = "Accueil";
-        $this->controller= "Index";
+        $this->controller = "Index";
     }
 
     public function indexAction()

app/controllers/MessagesController.php

Spacing +2 added lines, -2 removed lines

@@ -3,10 +3,10 @@
 class MessagesController extends ControllerBase
 {
 
-    public function initialize(){
+    public function initialize() {
         $this->model = "Message";
         $this->title = "Messages";
-        $this->controller= "Messages";
+        $this->controller = "Messages";
 
     }
 }

app/models/Message.php

Spacing +1 added lines, -1 removed lines

@@ -249,7 +249,7 @@
         return parent::findFirst($parameters);
     }
 
-    public function toString(){
+    public function toString() {
         return $this->content;
     }
 

app/models/TypeUser.php

Doc Comments +1 added lines, -2 removed lines

@@ -31,7 +31,6 @@ 
     /**
      * Method to set the value of field objet
      *
-     * @param string $objet
      * @return $this
      */
     public function setLibelle($libelle)
@@ -94,7 +93,7 @@ 
     /**
      * Allows to query the first record that match the specified conditions
      *
-     * @param mixed $parameters
+     * @param string $parameters
      * @return TypeUser
      */
     public static function findFirst($parameters = null)

Indentation +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
     }
     
     public function toString() {
-    	return $this->libelle;
+     return $this->libelle;
     }
 
 }

app/models/Operation.php

Indentation +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
     }
     
     public function toString() {
-    	return $this->operation;
+     return $this->operation;
     }
 
 }

app/models/Ressource.php

Indentation +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
     }
     
     public function toString() {
-    	return $this->libelle;
+     return $this->libelle;
     }
 
 }

app/controllers/ControllerBase.php

Doc Comments +8 added lines, -4 removed lines

@@ -171,6 +171,10 @@ 
     	return $acl;
     }
     
+    /**
+     * @param string $activeResource
+     * @param string $activeOperation
+     */
     public function verifyAccessAction($activeResource, $activeOperation) {
         if($this->session->has("user")){
             $user = $this->session->get("user");
@@ -200,7 +204,7 @@ 
      * 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 integer $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)
@@ -218,7 +222,7 @@ 
     /**
      * 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 integer $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)
@@ -229,7 +233,7 @@ 
     /**
      * 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 integer $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)
@@ -240,7 +244,7 @@ 
     /**
      * 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 integer $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)

Unused Use Statements -4 removed lines

@@ -1,11 +1,7 @@
 <?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
 {

Indentation +25 added lines, -25 removed lines

@@ -43,7 +43,7 @@ 
         $this->view->pick("main/index");
         } else {
             $this->view->pick("main/error");
-    	}
+     }
     }
 
     public function getInstance($id = NULL)
@@ -141,34 +141,34 @@ 
     }    
     
     public function loadAclAction($typeUser) {
-    	$acl = new AclList();
-    	$acl->setDefaultAction(Phalcon\Acl::DENY);
+     $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();
-    	}
+     $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);
-    	}
+     $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());
-    	}
+     $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;
+     return $acl;
     }
     
     public function verifyAccessAction($activeResource, $activeOperation) {

Braces +5 added lines, -5 removed lines

@@ -66,7 +66,7 @@ 
                 $this->view->setVar("object", $object);
                 $this->view->pick("main/read");
             }
-        }else{
+        } else{
 
         }
     }
@@ -195,7 +195,7 @@ 
             } else {
                 return 0;
             }
-        }else{
+        } else{
             return 0;
         }
     }
@@ -271,7 +271,7 @@ 
         $user = User::findFirst($userId);
         if($user->getTypeUser() == 0){
             return true;
-        }else{
+        } else{
             return false;
         }
     }
@@ -279,7 +279,7 @@ 
     public function isActual($user){
         if($this->session->has("user") || $user == $this->session->get("user")->getId()){
             return true;
-        }else{
+        } else{
             return false;
         }
     }
@@ -287,7 +287,7 @@ 
     public function isAdminAndActual($user){
         if($this->isAdmin($user) || $this->isActual($user)){
             return true;
-        }else{
+        } else{
             return false;
         }
     }

Spacing +25 added lines, -25 removed lines

@@ -23,7 +23,7 @@ 
         $this->view->setVar("title", $this->title);
     }
 
-    public function frmAction(){
+    public function frmAction() {
 
     }
 
@@ -39,7 +39,7 @@ 
             $msg = $this->_showDisplayedMessage($message);
         }
         $this->view->setVar("msg", $msg);
-        $objects = call_user_func($this->model . "::find");
+        $objects = call_user_func($this->model."::find");
         $this->view->setVar("objects", $objects);
         $this->view->pick("main/index");
         } else {
@@ -50,7 +50,7 @@ 
     public function getInstance($id = NULL)
     {
         if (isset($id)) {
-            $object = call_user_func($this->model . "::findfirst", $id);
+            $object = call_user_func($this->model."::findfirst", $id);
         } else {
             $className = $this->model;
             $object = new $className();
@@ -62,11 +62,11 @@ 
     {
         if ($this->verifyAccessAction($this->controller, "read")) {
             if ($id != null) {
-                $object = call_user_func($this->model . '::find', "id = $id");
+                $object = call_user_func($this->model.'::find', "id = $id");
                 $this->view->setVar("object", $object);
                 $this->view->pick("main/read");
             }
-        }else{
+        } else {
 
         }
     }
@@ -89,16 +89,16 @@ 
                 if ($id) {
                     try {
                         $object->save();
-                        $msg = new DisplayedMessage("Instance de " . $this->model . " modifiée");
+                        $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");
+                        $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");
+                        $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");
+                        $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)));
@@ -110,11 +110,11 @@ 
     public function soloUpdateAction()
     {
         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){
+            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']);
+                $func = 'set'.ucfirst($name);
+                $projet = call_user_func($this->model.'::findFirst', $_POST['pk']);
                 $projet->$func($_POST['value']);
                 $projet->save();
             }
@@ -126,7 +126,7 @@ 
     public function deleteAction($id = null)
     {
         if ($this->verifyAccessAction($this->controller, "write")) {
-            $object = call_user_func($this->model . '::findFirst', "$id");
+            $object = call_user_func($this->model.'::findFirst', "$id");
             $object->delete();
         }
 
@@ -185,7 +185,7 @@ 
     }
     
     public function verifyAccessAction($activeResource, $activeOperation) {
-        if($this->session->has("user")){
+        if ($this->session->has("user")) {
             $user = $this->session->get("user");
             $typeUser = TypeUser::findFirst("id = ".$user->getIdTypeUser());
             $typeUserSession = $user->getIdTypeUser();
@@ -195,13 +195,13 @@ 
             } else {
                 return 0;
             }
-        }else{
+        } else {
             return 0;
         }
     }
 
 
-    public function moreAction(){
+    public function moreAction() {
         $this->view->pick("main/more");
     }
 
@@ -267,27 +267,27 @@ 
     }
 
 
-    public function isAdmin($userId){
+    public function isAdmin($userId) {
         $user = User::findFirst($userId);
-        if($user->getTypeUser() == 0){
+        if ($user->getTypeUser() == 0) {
             return true;
-        }else{
+        } else {
             return false;
         }
     }
 
-    public function isActual($user){
-        if($this->session->has("user") || $user == $this->session->get("user")->getId()){
+    public function isActual($user) {
+        if ($this->session->has("user") || $user == $this->session->get("user")->getId()) {
             return true;
-        }else{
+        } else {
             return false;
         }
     }
 
-    public function isAdminAndActual($user){
-        if($this->isAdmin($user) || $this->isActual($user)){
+    public function isAdminAndActual($user) {
+        if ($this->isAdmin($user) || $this->isActual($user)) {
             return true;
-        }else{
+        } else {
             return false;
         }
     }