Skip to content

Commit

Permalink
add sell/retrieve check and seller in DomainSold ev
Browse files Browse the repository at this point in the history
  • Loading branch information
alessandroscifoni committed Jan 12, 2024
1 parent 19894fe commit c86ee50
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions smart_contract/GethDomain.sol
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ contract DomainMarketplace is ERC721Royalty, Ownable {

// Evento emesso quando un dominio è stato venduto per aggiornare la vista dei domini in vendita
// Non ho messo il prezzo perchè non interessa più non essendo appunto in vendita ** TEORICAMENTE POTREBBE ESSERE INTERESSANTE SAPERE A QUANTO E' STATO VENDUTO UN DOMINIO
event DomainSold(address indexed buyer, bytes indexed domain);
event DomainSold(address seller, address indexed buyer, bytes indexed domain);

// Evento emesso quando un dominio viene acquistato e le royalties vengono inviate all'acquirente originale
event RoyaltiesPaid(address indexed originalOwner, address indexed buyer, bytes indexed domain, uint256 royaltiesAmount);
Expand Down Expand Up @@ -127,7 +127,7 @@ contract DomainMarketplace is ERC721Royalty, Ownable {
domains[domain].price = 0;

// Emetti l'evento per ricordare di aggiornare la lista dei domini in vendita SERVE???
emit DomainSold(msg.sender, domain);
emit DomainSold(owner, msg.sender, domain);

}

Expand All @@ -140,6 +140,7 @@ contract DomainMarketplace is ERC721Royalty, Ownable {

// Funzione per mettere un dominio in vendita
function sellDomain(bytes calldata domain, uint32 price) external onlyDomainOwner(domain) returns (uint256 prezzo){
require(domains[domain].price>0, "Domain already in sale");
// uint256 id = uint256(keccak256(abi.encodePacked(domain)));
// prezzo maggiore di zero
require(price > 0, "Domain are not free :(");
Expand All @@ -154,6 +155,7 @@ contract DomainMarketplace is ERC721Royalty, Ownable {

function retrieveDomain(bytes calldata domain) external onlyDomainOwner(domain){
// uint256 id = uint256(keccak256(abi.encodePacked(domain)));
require(domains[domain].price==0, "Domain not in sale");

domains[domain].price = 0;

Expand Down

0 comments on commit c86ee50

Please sign in to comment.