execute purchase if price is lower instead of revert

todo[bot] commented 2 years ago

https://github.com/MoonSHRD/UniversalNFTMarketplace/blob/0dbe8e1bb09586a6f440ee81f5a3a96852ba3b21/contracts/721/singleton/MetaMarketplace.sol#L295-L298

This issue was generated by todo based on a `TODO` comment in 0dbe8e1bb09586a6f440ee81f5a3a96852ba3b21 when #100 was merged. cc @MoonSHRD.

JackBekket commented 2 years ago

execute purchase in case of higher price, not lower

JackBekket commented 2 years ago

https://github.com/MoonSHRD/UniversalNFTMarketplace/blob/0dbe8e1bb09586a6f440ee81f5a3a96852ba3b21/contracts/721/singleton/MetaMarketplace.sol#L293

заменить строго больше (>) на больше или равно (>=) по этой строке

JackBekket commented 2 years ago

https://github.com/MoonSHRD/UniversalNFTMarketplace/blob/0dbe8e1bb09586a6f440ee81f5a3a96852ba3b21/contracts/721/singleton/MetaMarketplace.sol#L295

добавить дополнительное условие если цена ставки строго равна цене оффера, который выставляет продавец шо то типа if ((bid_price_ == metainfo.activeSellOffers[tokenId].minPrice[currency_]) { purchase(....) }

при этом, по идее, если срабатывает purchase (цена ставки покупателя строго равна указаной цене продавца), то должен срабатывать только purchase, и не выставляться buyOffer. то есть по идее возможно где-то здесь нужно поставить break

MoonSHRD / UniversalNFTMarketplace