Refactor

MinterTeam · Oct 14, 2020 · 2f0c6d3 · 2f0c6d3
1 parent 221b66e
commit 2f0c6d3
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/core/state/coins/coins.go b/core/state/coins/coins.go
@@ -267,12 +267,12 @@ func (c *Coins) Recreate(newID types.CoinID, name string, symbol types.CoinSymbo
 
 func (c *Coins) ChangeOwner(symbol types.CoinSymbol, owner types.Address) {
 	info := c.getSymbolInfo(symbol)
-	info.SetOwnerAddress(&owner)
+	info.setOwnerAddress(owner)
 
 	coin := c.GetCoinBySymbol(symbol, 0)
 	coin.symbolInfo = info
 
-	c.setToMap(coin.id, coin)
+	c.setToMap(coin.ID(), coin)
 	c.markDirty(coin.ID())
 }
 

diff --git a/core/state/coins/model.go b/core/state/coins/model.go
@@ -142,8 +142,8 @@ type SymbolInfo struct {
 	isDirty bool
 }
 
-func (i *SymbolInfo) SetOwnerAddress(address *types.Address) {
-	i.COwnerAddress = address
+func (i *SymbolInfo) setOwnerAddress(address types.Address) {
+	i.COwnerAddress = &address
 	i.isDirty = true
 }
 

diff --git a/core/transaction/edit_coin_owner.go b/core/transaction/edit_coin_owner.go
@@ -29,7 +29,7 @@ func (data EditCoinOwnerData) BasicCheck(tx *Transaction, context *state.CheckSt
 		}
 	}
 
-	if info.OwnerAddress() == nil || info.OwnerAddress().Compare(sender) != 0 {
+	if info.OwnerAddress() == nil || *info.OwnerAddress() != sender {
 		owner := info.OwnerAddress().String()
 		return &Response{
 			Code: code.IsNotOwnerOfCoin,