fixup! editoast: define Model error type and use it in trait Delete

Signed-off-by: Leo Valais <leo.valais97@gmail.com>
OpenRailAssociation · Jan 10, 2025 · a15b78c · a15b78c
1 parent 2d7acaa
commit a15b78c
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/editoast/editoast_models/src/model.rs b/editoast/editoast_models/src/model.rs
@@ -31,6 +31,7 @@ impl From<diesel::result::Error> for Error {
                         constraint: captures.get(1).unwrap().as_str().to_string(),
                     }
                 } else {
+                    // falling back to the generic error — since it's still semantically correct, logging the error is enough
                     tracing::error!(?RE, %e, "failed to parse PostgreSQL error message");
                     Self::DatabaseError(e.into())
                 }
@@ -48,6 +49,7 @@ impl From<diesel::result::Error> for Error {
                         constraint: captures.get(2).unwrap().as_str().to_string(),
                     }
                 } else {
+                    // falling back to the generic error — since it's still semantically correct, logging the error is enough
                     tracing::error!(?RE, %e, "failed to parse PostgreSQL error message");
                     Self::DatabaseError(e.into())
                 }