Luctum / Increase

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) {

Spacing +6 added lines, -6 removed lines

@@ -162,9 +162,9 @@ 
  
     	$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());
+    		$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());
     	}
     	
@@ -172,9 +172,9 @@ 
     }
     
     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());
+            $typeUser = TypeUser::findFirst("id = " . $user->getIdTypeUser());
             $typeUserSession = $user->getIdTypeUser();
             $acl = $this->loadAclAction($typeUserSession);
             if ($acl->isAllowed($typeUser->getLibelle(), $activeResource, $activeOperation)) {
@@ -182,7 +182,7 @@ 
             } else {
                 return 0;
             }
-        }else{
+        } else {
             return 0;
         }
     }

Braces +1 added lines, -1 removed lines

@@ -182,7 +182,7 @@
             } else {
                 return 0;
             }
-        }else{
+        } else{
             return 0;
         }
     }

app/models/User.php

Doc Comments +1 added lines, -2 removed lines

@@ -89,7 +89,6 @@ 
     /**
      * Method to set the value of field role
      *
-     * @param string $role
      * @return $this
      */
     public function setIdTypeUser($idTypeUser)
@@ -150,7 +149,7 @@ 
     /**
      * Returns the value of field role
      *
-     * @return string
+     * @return integer
      */
     public function getIdTypeUser()
     {

app/controllers/UsersController.php

Indentation +22 added lines, -22 removed lines

@@ -22,34 +22,34 @@
     }
     
     public function signInAction() {
-    	$bootstrap = $this->jquery->bootstrap();
+     $bootstrap = $this->jquery->bootstrap();
     	 
-    	if(!empty($_POST['identite']) && !empty($_POST['password'])) {
+     if(!empty($_POST['identite']) && !empty($_POST['password'])) {
     		
-    		$userPseudo = User::findFirst("identite = '".$_POST['identite']."'");
+      $userPseudo = User::findFirst("identite = '".$_POST['identite']."'");
 
-    		$userMail = User::findFirst("mail = '".$_POST['identite']."'");
+      $userMail = User::findFirst("mail = '".$_POST['identite']."'");
     	
-    		if($userPseudo != NULL && password_verify($_POST['password'], $userPseudo->getPassword())) {
-    			$this->session->set("user", $userPseudo);
-    			$this->response->redirect("Index/index");
-    			$this->loadAclAction($userPseudo->getRole());
-    		} else if($userMail != NULL && password_verify($_POST['password'], $userMail->getPassword())) {
-    			$this->session->set("user", $userMail);
-    			$this->response->redirect("Index/index");
-    			$this->loadAclAction($userMail->getRole());
-    		} else {
-    			echo $bootstrap->htmlAlert("alert1","L'identifiant ou le mot de passe est incorrecte.");
-    		}
-    	}
+      if($userPseudo != NULL && password_verify($_POST['password'], $userPseudo->getPassword())) {
+       $this->session->set("user", $userPseudo);
+       $this->response->redirect("Index/index");
+       $this->loadAclAction($userPseudo->getRole());
+      } else if($userMail != NULL && password_verify($_POST['password'], $userMail->getPassword())) {
+       $this->session->set("user", $userMail);
+       $this->response->redirect("Index/index");
+       $this->loadAclAction($userMail->getRole());
+      } else {
+       echo $bootstrap->htmlAlert("alert1","L'identifiant ou le mot de passe est incorrecte.");
+      }
+     }
     }
 	
-	public function rulesAction() {
-		$accordion=$this->jquery->bootstrap()->htmlAccordion("accordion1");
-		$accordion->addPanel("Panel 1","Contenu du panel 1");
-		$accordion->addPanel("Panel 2","Contenu du panel 2");
-		echo $accordion;
-	}
+ public function rulesAction() {
+  $accordion=$this->jquery->bootstrap()->htmlAccordion("accordion1");
+  $accordion->addPanel("Panel 1","Contenu du panel 1");
+  $accordion->addPanel("Panel 2","Contenu du panel 2");
+  echo $accordion;
+ }
 
  
 

Spacing +9 added lines, -9 removed lines

@@ -24,30 +24,30 @@
     public function signInAction() {
     	$bootstrap = $this->jquery->bootstrap();
     	 
-    	if(!empty($_POST['identite']) && !empty($_POST['password'])) {
+    	if (!empty($_POST['identite']) && !empty($_POST['password'])) {
     		
-    		$userPseudo = User::findFirst("identite = '".$_POST['identite']."'");
+    		$userPseudo = User::findFirst("identite = '" . $_POST['identite'] . "'");
 
-    		$userMail = User::findFirst("mail = '".$_POST['identite']."'");
+    		$userMail = User::findFirst("mail = '" . $_POST['identite'] . "'");
     	
-    		if($userPseudo != NULL && password_verify($_POST['password'], $userPseudo->getPassword())) {
+    		if ($userPseudo != NULL && password_verify($_POST['password'], $userPseudo->getPassword())) {
     			$this->session->set("user", $userPseudo);
     			$this->response->redirect("Index/index");
     			$this->loadAclAction($userPseudo->getRole());
-    		} else if($userMail != NULL && password_verify($_POST['password'], $userMail->getPassword())) {
+    		} else if ($userMail != NULL && password_verify($_POST['password'], $userMail->getPassword())) {
     			$this->session->set("user", $userMail);
     			$this->response->redirect("Index/index");
     			$this->loadAclAction($userMail->getRole());
     		} else {
-    			echo $bootstrap->htmlAlert("alert1","L'identifiant ou le mot de passe est incorrecte.");
+    			echo $bootstrap->htmlAlert("alert1", "L'identifiant ou le mot de passe est incorrecte.");
     		}
     	}
     }
 	
 	public function rulesAction() {
-		$accordion=$this->jquery->bootstrap()->htmlAccordion("accordion1");
-		$accordion->addPanel("Panel 1","Contenu du panel 1");
-		$accordion->addPanel("Panel 2","Contenu du panel 2");
+		$accordion = $this->jquery->bootstrap()->htmlAccordion("accordion1");
+		$accordion->addPanel("Panel 1", "Contenu du panel 1");
+		$accordion->addPanel("Panel 2", "Contenu du panel 2");
 		echo $accordion;
 	}