diff --git a/actix_mvc_app/src/controllers/contract.rs b/actix_mvc_app/src/controllers/contract.rs index 18d14e4..d8578b6 100644 --- a/actix_mvc_app/src/controllers/contract.rs +++ b/actix_mvc_app/src/controllers/contract.rs @@ -62,9 +62,15 @@ impl ContractController { context.insert("draft_contracts", &draft_contracts); - Ok(HttpResponse::Ok().content_type("text/html").body( - tmpl.render("contracts/index.html", &context).unwrap() - )) + let rendered = tmpl.render("contracts/index.html", &context) + .map_err(|e| { + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", context); + actix_web::error::ErrorInternalServerError("Template rendering error") + })?; + + Ok(HttpResponse::Ok().content_type("text/html").body(rendered)) } // Display the list of all contracts @@ -83,9 +89,15 @@ impl ContractController { context.insert("contracts", &contracts_data); context.insert("filter", &"all"); - Ok(HttpResponse::Ok().content_type("text/html").body( - tmpl.render("contracts/contracts.html", &context).unwrap() - )) + let rendered = tmpl.render("contracts/contracts.html", &context) + .map_err(|e| { + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", context); + actix_web::error::ErrorInternalServerError("Template rendering error") + })?; + + Ok(HttpResponse::Ok().content_type("text/html").body(rendered)) } // Display the list of user's contracts @@ -103,9 +115,15 @@ impl ContractController { context.insert("contracts", &contracts_data); - Ok(HttpResponse::Ok().content_type("text/html").body( - tmpl.render("contracts/my_contracts.html", &context).unwrap() - )) + let rendered = tmpl.render("contracts/my_contracts.html", &context) + .map_err(|e| { + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", context); + actix_web::error::ErrorInternalServerError("Template rendering error") + })?; + + Ok(HttpResponse::Ok().content_type("text/html").body(rendered)) } // Display a specific contract @@ -128,9 +146,15 @@ impl ContractController { context.insert("contract", &contract_json); context.insert("user_has_signed", &false); // Mock data - Ok(HttpResponse::Ok().content_type("text/html").body( - tmpl.render("contracts/contract_detail.html", &context).unwrap() - )) + let rendered = tmpl.render("contracts/contract_detail.html", &context) + .map_err(|e| { + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", context); + actix_web::error::ErrorInternalServerError("Template rendering error") + })?; + + Ok(HttpResponse::Ok().content_type("text/html").body(rendered)) }, None => { Ok(HttpResponse::NotFound().finish()) @@ -156,9 +180,15 @@ impl ContractController { context.insert("contract_types", &contract_types); - Ok(HttpResponse::Ok().content_type("text/html").body( - tmpl.render("contracts/create_contract.html", &context).unwrap() - )) + let rendered = tmpl.render("contracts/create_contract.html", &context) + .map_err(|e| { + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", context); + actix_web::error::ErrorInternalServerError("Template rendering error") + })?; + + Ok(HttpResponse::Ok().content_type("text/html").body(rendered)) } // Process the create contract form diff --git a/actix_mvc_app/src/controllers/flow.rs b/actix_mvc_app/src/controllers/flow.rs index 00b5b78..8df1f7f 100644 --- a/actix_mvc_app/src/controllers/flow.rs +++ b/actix_mvc_app/src/controllers/flow.rs @@ -27,9 +27,9 @@ impl FlowController { let rendered = tmpl.render("flows/index.html", &ctx) .map_err(|e| { - eprintln!("Template rendering error: {}", e); - eprintln!("Error details: {:?}", e); - eprintln!("Context: {:?}", ctx); + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", ctx); actix_web::error::ErrorInternalServerError("Template rendering error") })?; @@ -48,9 +48,9 @@ impl FlowController { let rendered = tmpl.render("flows/flows.html", &ctx) .map_err(|e| { - eprintln!("Template rendering error: {}", e); - eprintln!("Error details: {:?}", e); - eprintln!("Context: {:?}", ctx); + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", ctx); actix_web::error::ErrorInternalServerError("Template rendering error") })?; @@ -78,9 +78,9 @@ impl FlowController { let rendered = tmpl.render("flows/flow_detail.html", &ctx) .map_err(|e| { - eprintln!("Template rendering error: {}", e); - eprintln!("Error details: {:?}", e); - eprintln!("Context: {:?}", ctx); + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", ctx); actix_web::error::ErrorInternalServerError("Template rendering error") })?; @@ -91,9 +91,9 @@ impl FlowController { let rendered = tmpl.render("error.html", &ctx) .map_err(|e| { - eprintln!("Template rendering error: {}", e); - eprintln!("Error details: {:?}", e); - eprintln!("Context: {:?}", ctx); + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", ctx); actix_web::error::ErrorInternalServerError("Template rendering error") })?; @@ -111,9 +111,9 @@ impl FlowController { let rendered = tmpl.render("flows/create_flow.html", &ctx) .map_err(|e| { - eprintln!("Template rendering error: {}", e); - eprintln!("Error details: {:?}", e); - eprintln!("Context: {:?}", ctx); + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", ctx); actix_web::error::ErrorInternalServerError("Template rendering error") })?; @@ -150,9 +150,9 @@ impl FlowController { let rendered = tmpl.render("flows/my_flows.html", &ctx) .map_err(|e| { - eprintln!("Template rendering error: {}", e); - eprintln!("Error details: {:?}", e); - eprintln!("Context: {:?}", ctx); + log::error!("Template rendering error: {}", e); + log::error!("Error details: {:?}", e); + log::error!("Context: {:?}", ctx); actix_web::error::ErrorInternalServerError("Template rendering error") })?; diff --git a/actix_mvc_app/src/views/base.html b/actix_mvc_app/src/views/base.html index 1986b06..516d2da 100644 --- a/actix_mvc_app/src/views/base.html +++ b/actix_mvc_app/src/views/base.html @@ -154,10 +154,10 @@
View Mode: {{ view_mode }}
diff --git a/actix_mvc_app/src/views/calendar/new_event.html b/actix_mvc_app/src/views/calendar/new_event.html index 9dcf41f..cfcc9c8 100644 --- a/actix_mvc_app/src/views/calendar/new_event.html +++ b/actix_mvc_app/src/views/calendar/new_event.html @@ -3,7 +3,7 @@ {% block title %}New Calendar Event{% endblock %} {% block content %} -