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)

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)

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;
         }
     }

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

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;
 	}