From 3f6ef50883492f77995a7e27872d0b5ae17b9d6a Mon Sep 17 00:00:00 2001
From: Michael Muré <batolettre@gmail.com>
Date: Sun, 14 Feb 2021 11:36:32 +0100
Subject: bug: migrate to the DAG entity structure!

---
 bug/op_set_metadata.go | 21 ++++-----------------
 1 file changed, 4 insertions(+), 17 deletions(-)

(limited to 'bug/op_set_metadata.go')

diff --git a/bug/op_set_metadata.go b/bug/op_set_metadata.go
index 23d11461..c4988b84 100644
--- a/bug/op_set_metadata.go
+++ b/bug/op_set_metadata.go
@@ -20,38 +20,25 @@ type SetMetadataOperation struct {
 // Sign-post method for gqlgen
 func (op *SetMetadataOperation) IsOperation() {}
 
-func (op *SetMetadataOperation) base() *OpBase {
-	return &op.OpBase
-}
-
 func (op *SetMetadataOperation) Id() entity.Id {
-	return idOperation(op)
+	return idOperation(op, &op.OpBase)
 }
 
 func (op *SetMetadataOperation) Apply(snapshot *Snapshot) {
 	for _, target := range snapshot.Operations {
 		if target.Id() == op.Target {
-			base := target.base()
-
-			if base.extraMetadata == nil {
-				base.extraMetadata = make(map[string]string)
-			}
-
 			// Apply the metadata in an immutable way: if a metadata already
 			// exist, it's not possible to override it.
-			for key, val := range op.NewMetadata {
-				if _, exist := base.extraMetadata[key]; !exist {
-					base.extraMetadata[key] = val
-				}
+			for key, value := range op.NewMetadata {
+				target.setExtraMetadataImmutable(key, value)
 			}
-
 			return
 		}
 	}
 }
 
 func (op *SetMetadataOperation) Validate() error {
-	if err := opBaseValidate(op, SetMetadataOp); err != nil {
+	if err := op.OpBase.Validate(op, SetMetadataOp); err != nil {
 		return err
 	}
 
-- 
cgit