vm: improve WitnessPos Display implementation

src/vm/contract.rs

@@ -23,14 +23,15 @@
 use std::borrow::Borrow;
 use std::cell::RefCell;
 use std::cmp::Ordering;
-use std::fmt::Debug;
+use std::fmt::{self, Debug, Display, Formatter};
 use std::num::NonZeroU32;
 use std::rc::Rc;
 
 use amplify::confinement;
 use amplify::num::u24;
 use bp::seals::txout::{CloseMethod, ExplicitSeal, VerifyError, Witness};
 use bp::{dbc, Tx, Txid};
+use chrono::{MappedLocalTime, TimeZone, Utc};
 use commit_verify::mpc;
 use single_use_seals::SealWitness;
 use strict_encoding::{StrictDecode, StrictDumb, StrictEncode};
@@ -290,15 +291,14 @@ impl<'op> Operation for OrdOpRef<'op> {
     }
 }
 
-#[derive(Getters, Copy, Clone, PartialEq, Eq, Hash, Debug, Display)]
+#[derive(Getters, Copy, Clone, PartialEq, Eq, Hash, Debug)]
 #[derive(StrictType, StrictEncode, StrictDecode)]
 #[strict_type(lib = LIB_NAME_RGB_LOGIC)]
 #[cfg_attr(
     feature = "serde",
     derive(Serialize, Deserialize),
     serde(crate = "serde_crate", rename_all = "camelCase")
 )]
-#[display("{height}@{timestamp}")]
 pub struct WitnessPos {
     #[getter(as_copy)]
     layer1: Layer1,
@@ -389,6 +389,16 @@ impl Ord for WitnessPos {
     }
 }
 
+impl Display for WitnessPos {
+    fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
+        write!(f, "{}:{}, ", self.layer1, self.height)?;
+        match Utc.timestamp_opt(self.timestamp, 0) {
+            MappedLocalTime::Single(time) => write!(f, "{}", time.format("%Y-%m-%d %H:%M:%S")),
+            _ => f.write_str("invalid timestamp"),
+        }
+    }
+}
+
 /// RGB consensus information about the status of a witness transaction. This
 /// information is used in ordering state transition and state extension
 /// processing in the AluVM during the validation, as well as consensus ordering