From 2d83c557b1c89b4486ffe68bf71b2def7f624e61 Mon Sep 17 00:00:00 2001 From: Mikhail Melnik Date: Tue, 17 Oct 2017 16:24:16 +0300 Subject: [PATCH] Finish minting should me called only once Multiple calls to `finishMinting` will emit multiple `MintFinished` events which may be surprising if one rely on `MintFinished` event. --- contracts/token/MintableToken.sol | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contracts/token/MintableToken.sol b/contracts/token/MintableToken.sol index 234def478..4f3e3c505 100644 --- a/contracts/token/MintableToken.sol +++ b/contracts/token/MintableToken.sol @@ -43,7 +43,7 @@ contract MintableToken is StandardToken, Ownable { * @dev Function to stop minting new tokens. * @return True if the operation was successful. */ - function finishMinting() onlyOwner public returns (bool) { + function finishMinting() onlyOwner canMint public returns (bool) { mintingFinished = true; MintFinished(); return true;