* Error handling in ERC20 and ERC721 * Added message string for require. * Fixed solhint errors. * Updated PR as per issue #1709 * changes as per #1709 and openzeppelin forum. * Changes in require statement * Changes in require statement * build pipeline fix * Changes as per @nventuro's comment. * Update revert reason strings. * Fianal update of revert reason strings. * WIP: Updating reason strings in test cases * WIP: Added changes to ERC20 and ERC721 * Fixes linting errors in *.tes.js files * Achieved 100% code coverage * Updated the test cases with shouldFail.reverting.withMessage() * Fix package-lock. * address review comments * fix linter issues * fix remaining revert reasons
44 lines
992 B
Solidity
44 lines
992 B
Solidity
pragma solidity ^0.5.7;
|
|
|
|
import "../Roles.sol";
|
|
|
|
contract SignerRole {
|
|
using Roles for Roles.Role;
|
|
|
|
event SignerAdded(address indexed account);
|
|
event SignerRemoved(address indexed account);
|
|
|
|
Roles.Role private _signers;
|
|
|
|
constructor () internal {
|
|
_addSigner(msg.sender);
|
|
}
|
|
|
|
modifier onlySigner() {
|
|
require(isSigner(msg.sender), "SignerRole: caller does not have the Signer role");
|
|
_;
|
|
}
|
|
|
|
function isSigner(address account) public view returns (bool) {
|
|
return _signers.has(account);
|
|
}
|
|
|
|
function addSigner(address account) public onlySigner {
|
|
_addSigner(account);
|
|
}
|
|
|
|
function renounceSigner() public {
|
|
_removeSigner(msg.sender);
|
|
}
|
|
|
|
function _addSigner(address account) internal {
|
|
_signers.add(account);
|
|
emit SignerAdded(account);
|
|
}
|
|
|
|
function _removeSigner(address account) internal {
|
|
_signers.remove(account);
|
|
emit SignerRemoved(account);
|
|
}
|
|
}
|