From 780ad36696099bccff7354778ca393b6c795a748 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ond=C5=99ej=20Hru=C5=A1ka?= <ondra@ondrovo.com>
Date: Thu, 11 Feb 2021 21:57:23 +0100
Subject: [PATCH] add old flash to realtion create

---
 .../templates/property_create.html.tera        |  2 --
 .../templates/relation_create.html.tera        | 12 +++++-------
 yopa-web/src/routes/relation_model.rs          | 18 +++++++++++++++++-
 3 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/yopa-web/resources/templates/property_create.html.tera b/yopa-web/resources/templates/property_create.html.tera
index 69ac221..06e9d3e 100644
--- a/yopa-web/resources/templates/property_create.html.tera
+++ b/yopa-web/resources/templates/property_create.html.tera
@@ -13,8 +13,6 @@ Define property
 {# The parent can be either object or relation model #}
 <h1>Add new property to {{object.describe}}</h1>
 
-{# TODO implement value restoration on error #}
-
 <form action="/model/property/create" method="POST">
 	<input type="hidden" name="object" value="{{object.id}}">
 
diff --git a/yopa-web/resources/templates/relation_create.html.tera b/yopa-web/resources/templates/relation_create.html.tera
index 181b951..f39c64b 100644
--- a/yopa-web/resources/templates/relation_create.html.tera
+++ b/yopa-web/resources/templates/relation_create.html.tera
@@ -12,28 +12,26 @@ Define relation
 
 <h1>Define new relation from "{{object.name}}"</h1>
 
-{# TODO implement value restoration on error #}
-
 <form action="/model/relation/create" method="POST">
 	<input type="hidden" name="object" value="{{object.id}}">
 
 	<label for="name">Name:</label>
-	<input type="text" id="name" name="name" autocomplete="off"><br>
+	<input type="text" id="name" name="name" value="{{old.name}}" autocomplete="off"><br>
 
 	<label for="reciprocal_name">Reciprocal name:</label>
-	<input type="text" id="reciprocal_name" name="reciprocal_name" autocomplete="off"><br>
+	<input type="text" id="reciprocal_name" name="reciprocal_name" value="{{old.reciprocal_name}}" autocomplete="off"><br>
 
 	<label for="optional">Optional:</label>
-	<input type="checkbox" name="optional" id="optional" value="true" autocomplete="off">
+	<input type="checkbox" name="optional" id="optional" value="true" {{opt(checked=old.optional)}} autocomplete="off">
 	<br>
 
 	<label for="multiple">Multiple:</label>
-	<input type="checkbox" name="multiple" id="multiple" value="true" autocomplete="off"><br>
+	<input type="checkbox" name="multiple" id="multiple" value="true" {{opt(checked=old.multiple)}} autocomplete="off"><br>
 
 	<label for="related">Related object:</label>
 	<select name="related" id="related" autocomplete="off">
 		{% for m in models %}
-		<option value="{{ m.id }}">{{ m.name }}</option>
+		<option value="{{ m.id }}" {{selected(val=old.related, opt=m.id)}}>{{ m.name }}</option>
 		{% endfor %}
 	</select><br>
 
diff --git a/yopa-web/src/routes/relation_model.rs b/yopa-web/src/routes/relation_model.rs
index 8c20f64..2217a3d 100644
--- a/yopa-web/src/routes/relation_model.rs
+++ b/yopa-web/src/routes/relation_model.rs
@@ -30,6 +30,19 @@ pub(crate) async fn create_form(
 
     debug!("ID = {}", object_id);
 
+    if let Ok(Some(form)) = session.take::<RelationModelCreateForm>("old") {
+        context.insert("old", &form);
+    } else {
+        context.insert("old", &RelationModelCreateForm {
+            object: Default::default(),
+            name: "".to_string(),
+            reciprocal_name: "".to_string(),
+            optional: false,
+            multiple: false,
+            related: Default::default()
+        });
+    }
+
     let object = rg.get_object_model(object_id.parse_or_bad_request()?)
         .ok_or_else(|| actix_web::error::ErrorNotFound("No such source object"))?;
 
@@ -40,10 +53,12 @@ pub(crate) async fn create_form(
     context.insert("models", &models);
     context.insert("object", &object);
 
+
+
     TERA.build_response("relation_create", &context)
 }
 
-#[derive(Deserialize)]
+#[derive(Serialize,Deserialize)]
 pub(crate) struct RelationModelCreateForm {
     pub object: ID,
     pub name: String,
@@ -80,6 +95,7 @@ pub(crate) async fn create(
         Err(e) => {
             warn!("Error creating relation model: {}", e);
             session.flash_error(e.to_string());
+            session.set("old", &form);
             redirect(format!("/model/relation/create/{}", form.object))
         }
     }