Clean up naming and semantics of history accessors.

1 files changed, 5 insertions, 8 deletions

diff --git a/src/channel/app.rs b/src/channel/app.rs
index bb331ec..1b2cc48 100644
--- a/src/channel/app.rs
+++ b/src/channel/app.rs
@@ -33,7 +33,7 @@ impl<'a> Channels<'a> {
         self.events
             .broadcast(channel.events().map(Event::from).collect::<Vec<_>>());
 
-        Ok(channel.snapshot())
+        Ok(channel.as_created())
     }
 
     pub async fn all(&self, resume_point: Option<Sequence>) -> Result<Vec<Channel>, InternalError> {
@@ -64,8 +64,7 @@ impl<'a> Channels<'a> {
             .channels()
             .by_id(channel)
             .await
-            .not_found(|| Error::NotFound(channel.clone()))?
-            .snapshot();
+            .not_found(|| Error::NotFound(channel.clone()))?;
 
         let messages = tx
             .messages()
@@ -90,16 +89,14 @@ impl<'a> Channels<'a> {
             .channels()
             .by_id(channel)
             .await
-            .not_found(|| Error::NotFound(channel.clone()))?
-            .snapshot();
+            .not_found(|| Error::NotFound(channel.clone()))?;
 
         let mut events = Vec::new();
 
         let messages = tx.messages().in_channel(&channel, None).await?;
         for message in messages {
-            let message = message.snapshot();
             let deleted = tx.sequence().next(deleted_at).await?;
-            let message = tx.messages().delete(&message.id, &deleted).await?;
+            let message = tx.messages().delete(message.id(), &deleted).await?;
             events.extend(
                 message
                     .events()
@@ -109,7 +106,7 @@ impl<'a> Channels<'a> {
         }
 
         let deleted = tx.sequence().next(deleted_at).await?;
-        let channel = tx.channels().delete(&channel.id, &deleted).await?;
+        let channel = tx.channels().delete(channel.id(), &deleted).await?;
         events.extend(
             channel
                 .events()