-
Notifications
You must be signed in to change notification settings - Fork 33
Closed
Milestone
Description
The purpose of _status is unclear. All functions based on this variable seem to have no end-user value.
Recommendation: First consider if this is needed at all. Please note that messages cannot be sent by address(0) so currently, all status check is redundant.
Review if _activateOperator, _deactivateOperator, _transferOperator, _renounceOperator, hasActiveOperator, isActiveOperator are necessary.
If keeping, consider switching _status to an enum or adding documentation.
If removing, just check whether the operator is address(0)
References:
Metadata
Metadata
Assignees
Labels
No labels