From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Wed, 17 Feb 2021 22:52:16 +0000 (+0000)
Subject: magic: Prep reorg
X-Git-Tag: otter-0.4.0~421
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=b16094c2efeed5ca29f498bde2028e2ae9f264b0;p=otter.git

magic: Prep reorg

Not sure if I'm going to want this...

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---

diff --git a/src/magic.rs b/src/magic.rs
index cfeb8748..89508864 100644
--- a/src/magic.rs
+++ b/src/magic.rs
@@ -135,11 +135,12 @@ impl Piece for Hand {
     let _occults = &mut gs.occults;
 
     let did;
+    let new_owner;
     match (opname, xdata.owner.is_some()) {
       ("claim", false) => {
         let nick = Html(htmlescape::encode_minimal(&gpl.nick));
         let new_desc = Html(format!("{}'s hand", &nick.0));
-        xdata.owner = Some(MagicOwner {
+        new_owner = Some(MagicOwner {
           player,
           dasharray,
           desc: new_desc,
@@ -148,7 +149,7 @@ impl Piece for Hand {
         did = format!("claimed {}", &old_desc.0);
       }
       ("deactivate", true) => {
-        xdata.owner = None;
+        new_owner = None;
         // xxx recalculate occultations
         did = format!("deactivated {}", &old_desc.0);
       }
@@ -163,6 +164,9 @@ impl Piece for Hand {
 
     let who_by = Html(htmlescape::encode_minimal(&gpl.nick));
     let log = vec![ LogEntry { html: Html(format!("{} {}", who_by.0, did)) }];
+
+    xdata.owner = new_owner;
+
     Ok(PieceUpdate {
       wrc, log,
       ops: PUOs::Simple(PUO::Modify(())), // xxx