-
Notifications
You must be signed in to change notification settings - Fork 88
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: L-01 Ownership of Land and PolygonLand
- Loading branch information
Andres Adjimann
committed
Jul 10, 2024
1 parent
3911872
commit 5bbcc5e
Showing
1 changed file
with
6 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,7 +20,9 @@ import {LandBaseToken} from "./LandBaseToken.sol"; | |
/// @author The Sandbox | ||
/// @custom:security-contact [email protected] | ||
/// @notice LAND contract | ||
/// @dev LAND contract implements ERC721, quad and marketplace filtering functionalities | ||
/// @dev LAND contract implements ERC721, quads, metadata, royalties and marketplace filtering functionalities. | ||
/// @dev The contract also implements EIP173 because it is needed by some marketplaces. The owner() doesn't have | ||
/// @dev any privileged roles within the contract. It can be is set by the admin to any value. | ||
abstract contract LandBase is | ||
LandBaseToken, | ||
Initializable, | ||
|
@@ -98,6 +100,9 @@ abstract contract LandBase is | |
|
||
/// @notice Set the address of the new owner of the contract | ||
/// @param newOwner address of new owner | ||
/// @dev This owner doesn't have any privileged role within this contract | ||
/// @dev It is set by the admin to comply with EIP173 which is needed by some marketplaces | ||
/// @dev Even when set to address(0) ownership is never permanently renounced the admin can always set any value | ||
function transferOwnership(address newOwner) external onlyAdmin { | ||
_transferOwnership(newOwner); | ||
} | ||
|
5bbcc5e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coverage for this commit
Coverage Report