将一些public方法改为external
This commit is contained in:
parent
78c690174d
commit
6edd2a756d
File diff suppressed because one or more lines are too long
@ -198,7 +198,7 @@ contract BEMultiSigWallet is AccessControlEnumerable {
|
|||||||
bytes32 predecessor,
|
bytes32 predecessor,
|
||||||
bytes32 salt,
|
bytes32 salt,
|
||||||
uint256 delay
|
uint256 delay
|
||||||
) public virtual onlyRoleOrOpenRole(PROPOSER_ROLE) {
|
) external virtual onlyRoleOrOpenRole(PROPOSER_ROLE) {
|
||||||
require(
|
require(
|
||||||
targets.length == values.length,
|
targets.length == values.length,
|
||||||
"BEMultiSigWallet: length mismatch"
|
"BEMultiSigWallet: length mismatch"
|
||||||
@ -229,7 +229,9 @@ contract BEMultiSigWallet is AccessControlEnumerable {
|
|||||||
*
|
*
|
||||||
* - the caller must have the 'proposer' role.
|
* - the caller must have the 'proposer' role.
|
||||||
*/
|
*/
|
||||||
function cancel(bytes32 id) public virtual onlyRoleOrOpenRole(PROPOSER_ROLE) {
|
function cancel(
|
||||||
|
bytes32 id
|
||||||
|
) external virtual onlyRoleOrOpenRole(PROPOSER_ROLE) {
|
||||||
require(
|
require(
|
||||||
isOperationPending(id),
|
isOperationPending(id),
|
||||||
"BEMultiSigWallet: operation cannot be cancelled"
|
"BEMultiSigWallet: operation cannot be cancelled"
|
||||||
@ -244,7 +246,7 @@ contract BEMultiSigWallet is AccessControlEnumerable {
|
|||||||
*/
|
*/
|
||||||
function confirmTransaction(
|
function confirmTransaction(
|
||||||
bytes32[] calldata ids
|
bytes32[] calldata ids
|
||||||
) public onlyRoleOrOpenRole(CONFIRM_ROLE) {
|
) external onlyRoleOrOpenRole(CONFIRM_ROLE) {
|
||||||
for (uint256 i = 0; i < ids.length; ++i) {
|
for (uint256 i = 0; i < ids.length; ++i) {
|
||||||
require(
|
require(
|
||||||
isOperationPending(ids[i]),
|
isOperationPending(ids[i]),
|
||||||
@ -260,7 +262,7 @@ contract BEMultiSigWallet is AccessControlEnumerable {
|
|||||||
*/
|
*/
|
||||||
function revokeConfirmation(
|
function revokeConfirmation(
|
||||||
bytes32[] calldata ids
|
bytes32[] calldata ids
|
||||||
) public onlyRoleOrOpenRole(CONFIRM_ROLE) {
|
) external onlyRoleOrOpenRole(CONFIRM_ROLE) {
|
||||||
for (uint256 i = 0; i < ids.length; ++i) {
|
for (uint256 i = 0; i < ids.length; ++i) {
|
||||||
require(
|
require(
|
||||||
isOperationPending(ids[i]),
|
isOperationPending(ids[i]),
|
||||||
@ -286,7 +288,7 @@ contract BEMultiSigWallet is AccessControlEnumerable {
|
|||||||
bytes[] calldata datas,
|
bytes[] calldata datas,
|
||||||
bytes32 predecessor,
|
bytes32 predecessor,
|
||||||
bytes32 salt
|
bytes32 salt
|
||||||
) public payable virtual onlyRoleOrOpenRole(EXECUTOR_ROLE) {
|
) external payable virtual onlyRoleOrOpenRole(EXECUTOR_ROLE) {
|
||||||
require(
|
require(
|
||||||
targets.length == values.length,
|
targets.length == values.length,
|
||||||
"BEMultiSigWallet: length mismatch"
|
"BEMultiSigWallet: length mismatch"
|
||||||
|
Loading…
x
Reference in New Issue
Block a user