From d9e20a27f85da43a5e53b08a0f7905eff5c7a8cb Mon Sep 17 00:00:00 2001 From: thejoelinux Date: Mon, 21 Dec 2015 17:34:56 +0100 Subject: [PATCH] refactoring des editions de liste --- classes/record.php | 2 +- controllers/app.php | 13 ++++- views/esar_categories/edit.html | 28 +++++----- views/esar_categories/list.html | 62 ++++++++++----------- views/payment_methods/edit.html | 54 +++++++++--------- views/payment_methods/list.html | 97 ++++++++++++++++++++++----------- 6 files changed, 147 insertions(+), 109 deletions(-) diff --git a/classes/record.php b/classes/record.php index 78adaf0..96ac887 100644 --- a/classes/record.php +++ b/classes/record.php @@ -48,7 +48,7 @@ class Record { $sql = " UPDATE ".$this->table." SET ".$update_sql." updated_at = now() WHERE id = ".$this->id; return $GLOBALS["data"]->update($sql); - } + } } } diff --git a/controllers/app.php b/controllers/app.php index 4bdb0f2..22a97ce 100644 --- a/controllers/app.php +++ b/controllers/app.php @@ -59,7 +59,11 @@ class AppController { try { $classname = $this->model; $object = new $classname(0); - $object->create(); + if($object->create()) { + $this->set_message("Création effectuée"); + } else { + $this->set_message("Aucun nouvel objet n'a été créé"); + } // FIXME : may be this fail !!! if($render == "list") { $classname::fetch_all($objects); @@ -97,8 +101,10 @@ class AppController { try { $classname = $this->model; $object = $classname::fetch($GLOBALS["data"]->db_escape_string($_REQUEST["i"])); - if($object->id != 0) { - $object->update(); + if($object->id != 0) { + if($object->update()) { + $this->set_message("Modification enregistrée"); + } $objects = array(); $classname::fetch_all($objects); $this->set("objects", $objects); @@ -115,6 +121,7 @@ class AppController { try { $classname = $this->model; if($_REQUEST["i"] = $classname::delete($_REQUEST["i"])) { + $this->set_message("Suppression effectuée"); $classname::fetch_all($objects); $this->set("objects", $objects); return $_REQUEST["o"]."/list"; diff --git a/views/esar_categories/edit.html b/views/esar_categories/edit.html index 9c01e1c..b7517d9 100644 --- a/views/esar_categories/edit.html +++ b/views/esar_categories/edit.html @@ -2,12 +2,12 @@
- +
@@ -44,22 +44,22 @@ $('#save_button').click(function(){ alert ("Vous n'avez pas saisi de nom !"); return false; } - if($('#esar_category_id').val() == 0) { - $('#a').val('create'); - } else { + if($('#esar_category_id').val() == 0) { + $('#a').val('create'); + } else { $('#i').val($('#esar_category_id').val()); - $('#a').val('update'); - } + $('#a').val('update'); + } document.defaultform.submit(); return true; }); $('#delete_button').click(function(){ - var msg = 'Voulez-vous réellement supprimer cette catégorie ?\n' + - 'Cette action n\'est possible que si celle-ci n\'a pas été\n' + - 'utilisée pour la classification d\'un jeu.'; - if(confirm(msg)) { - $('#a').val('delete'); - document.defaultform.submit(); - } + var msg = 'Voulez-vous réellement supprimer cette catégorie ?\n' + + 'Cette action n\'est possible que si celle-ci n\'a pas été\n' + + 'utilisée pour la classification d\'un jeu.'; + if(confirm(msg)) { + $('#a').val('delete'); + document.defaultform.submit(); + } }); diff --git a/views/esar_categories/list.html b/views/esar_categories/list.html index 1e2c03c..d139ee7 100644 --- a/views/esar_categories/list.html +++ b/views/esar_categories/list.html @@ -3,10 +3,10 @@ {% block content %}
- - + + Catégories ESAR - +
+ {% include 'ihm_messages.html' %} +
- - - - - - - - - + + + + + + + + + {% for key, val in objects %} - - - - + + + - + + + + + + + {% else %} {% endfor %} - +
@@ -72,7 +70,7 @@ diff --git a/views/payment_methods/list.html b/views/payment_methods/list.html index 4e1e20c..8e61bce 100644 --- a/views/payment_methods/list.html +++ b/views/payment_methods/list.html @@ -1,43 +1,50 @@ +{% extends "base.html" %} +{% block title %}Méthode de paiement{% endblock %} +{% block content %}
- - - Méthodes de paiement - + + + Méthodes de paiement + - - - Nouvelle méthode... - +
+ {% include 'ihm_messages.html' %} +
- - - - - - - - +
NomDescriptionActions
+ + + + + + + + {% for key, val in objects %} - - + + - - + + {% else %} {% endfor %} +
+ + + +{% endblock %} -- GitLab