Escher contest - 0x4non's results

LOWS / QA / NC

LOWS

[L-0] Missing address(0) checks

Escher721.sol#L64-L69: Missing address(0) check to receiver LPDAFactory.sol#L46-L48: Missing address(0) check to fees OpenEditionFactory.sol#L42-L44: Missing address(0) check to fees FixedPriceFactory.sol#L42-L44: Missing address(0) check to fees Escher721Factory.sol#L17-L21: on _escher

[L-1] Unsafe downcasting uint256 to uint48

Downcasting, or converting a larger integer type to a smaller one, can potentially lead to loss of data if the value of the larger integer is too big to fit into the smaller type. This can happen in Solidity when using the uint48 type, which can only hold values up to 281474976710655 (or 2^48-1), but is often used to store values that are initially declared as uint256, which has a much larger range of possible values.

In order to prevent data loss, it is important to carefully check the value of the uint256 variable before downcasting it to uint48. This can be done using an if statement and the require function, which will throw an error if the condition is not met.

This happens on:

• LPDA.sol#L59: downcasting _amount
• FixedPrice.sol#L62: downcasting _amount

[L-2] Royalty Fee cant be 100%

On Escher721.sol#L64-L69 it seems that feeNumerator can be 100_00 bps, add a threashold if you intent to fix a max royalty percentage.

[L-3] Use _safeMint instead of _mint for minting ERC721

OpenZeppelin recommends the usage of _safeMint() instead of _mint(). If the recipient is a contract,_safeMint() checks whether they can handle ERC721 tokens.

On (Escher721.sol#L52)[https://github.com/code-423n4/2022-12-escher/blob/main/src/Escher721.sol#L52]mint(address to, uint256 tokenId) you are using _mint function to mint the NTFs. However, this will not trigger the expected callback after minting the NFT if the buyer is a contract. This could cause issues with the implementation of the ERC721 standard and may result in stuck or malfunctioning contracts.

So if a contract buys a NFT in any of this;

• https://github.com/code-423n4/2022-12-escher/blob/main/src/minters/FixedPrice.sol#L66
• https://github.com/code-423n4/2022-12-escher/blob/main/src/minters/LPDA.sol#L74
• https://github.com/code-423n4/2022-12-escher/blob/main/src/minters/OpenEdition.sol#L67

It could get stuck or dont work as the ERC721 standard expect to work.

The recomendation is to use _safeMint instead of _mint, take in consideration that _safeMint will trigger a callback to the receiver, so stick to check - effects - itterarion pattern or add a nonReentrant modifier to minters/FixedPrice.sol#L66, minters/LPDA.sol#L74 & minters/OpenEdition.sol#L67

In OZ ERC721 docs on _mint method;

Usage of this method is discouraged, use _safeMint whenever possible

Reference: https://docs.openzeppelin.com/contracts/4.x/api/token/erc721#ERC721-_mint-address-uint256-

[L-4] Reentrancy issue

If minting would work as expected (if the minter is a contract he should receive a callback because of _safeMint), this functions dont respect the check - effects - itterarion pattern or add a nonReentrant modifier.

• https://github.com/code-423n4/2022-12-escher/blob/main/src/minters/FixedPrice.sol#L66
• https://github.com/code-423n4/2022-12-escher/blob/main/src/minters/LPDA.sol#L74
• https://github.com/code-423n4/2022-12-escher/blob/main/src/minters/OpenEdition.sol#L67

Non critical

[NC-0] Floating pragma

Floating pragma may cause unexpected compilation time behaviour and introduce unintended bugs. Is recommend to use only on interfaces and librearies Change pragma solidity ^0.8.17; to pragma solidity 0.8.17; on;

• src/uris/Base.sol:2:pragma solidity ^0.8.17;
• src/uris/Generative.sol:2:pragma solidity ^0.8.17;
• src/uris/Unique.sol:2:pragma solidity ^0.8.17;
• src/minters/OpenEdition.sol:2:pragma solidity ^0.8.17;
• src/minters/FixedPrice.sol:2:pragma solidity ^0.8.17;
• src/minters/LPDAFactory.sol:2:pragma solidity ^0.8.17;
• src/minters/OpenEditionFactory.sol:2:pragma solidity ^0.8.17;
• src/minters/FixedPriceFactory.sol:2:pragma solidity ^0.8.17;
• src/minters/LPDA.sol:2:pragma solidity ^0.8.17;
• src/Escher.sol:2:pragma solidity ^0.8.17;
• src/Escher721Factory.sol:2:pragma solidity ^0.8.17;
• src/Escher721.sol:2:pragma solidity ^0.8.17;

[Q-0] Missing event emission on critical functions

Event emission is a critical aspect of smart contract development in Solidity, as it allows external contracts and applications to listen for and react to changes in the state of a contract. Failure to emit events on key functions can prevent external contracts and applications from functioning properly, and can also make it difficult to debug and track changes to the contract.

• Function updateURIDelegate on Escher721.sol#L57-L59 should emit an event
• Function setDefaultRoyalty on Escher721.sol#L64-L69 should emit an event
• Function resetDefaultRoyalty on Escher721.sol#L72-L74 should emit an event
• Function setFeeReceiver on OpenEditionFactory.sol#L42-L44 should emit an event
• Function setFeeReceiver on LPDAFactory.sol#L46-L48 should emit an event
• Function setFeeReceiver on FixedPriceFactory.sol#L42 should emit an event

[Q-1] struct should be defined before contract states vars

On OpenEdition.sol#L11 you have state variable an then the struct. Change it and first declare the structs.

On LPDA.sol#L11 you have state variable an then the struct. Change it and first declare the structs.

[Q-2] Use _disableInitializers on Escher721 so Escher721Factory dont have to intialize implementation

diff --git a/src/Escher721.sol b/src/Escher721.sol
index c22429b..f9b9fd5 100644
--- a/src/Escher721.sol
+++ b/src/Escher721.sol
@@ -22,7 +22,9 @@ contract Escher721 is
     address public tokenUriDelegate;
 
     /// @custom:oz-upgrades-unsafe-allow constructor
-    constructor() {}
+    constructor() {
+        _disableInitializers();
+    }
 
     /// @notice initialize the proxy contract
     /// @param _creator the initial admin for the contract
diff --git a/src/Escher721Factory.sol b/src/Escher721Factory.sol
index 9eb2771..b1bae91 100644
--- a/src/Escher721Factory.sol
+++ b/src/Escher721Factory.sol
@@ -17,7 +17,6 @@ contract Escher721Factory {
     constructor(address _escher) {
         escher = Escher(_escher);
         implementation = address(new Escher721());
-        Escher721(implementation).initialize(address(this), address(0), "Implementation", "IMPL");
     }
 
     /// @notice create a new escher unique contract

Gas optimizations

Use uint40 for representing time to pack structs

If you use uint40, which is considere safe for representing unix timstamp you could pack the structs and save lot of gas;

Snapshot

test_LPDA() (gas: -24987 (-3.564%)) 
test_RevertsWhenSoldOut_Buy() (gas: -24931 (-3.576%)) 
test_RevertsWhenEnded_Buy() (gas: -24753 (-3.603%)) 
test_SellsOut_Buy() (gas: -24741 (-3.617%)) 
test_RevertsWhenAlreadyRefunded_Refund() (gas: -24750 (-6.225%)) 
test_Refund() (gas: -24616 (-6.243%)) 
test_WhenNotOver_Refund() (gas: -24616 (-6.243%)) 
test_RevertsWhenTooSoon_Buy() (gas: -24546 (-6.244%)) 
test_RevertsWhenNoRefund_Refund() (gas: -24627 (-6.315%)) 
test_RevertsWhenTooLate_Cancel() (gas: -24452 (-6.339%)) 
test_RevertsWhenNotOwner_Cancel() (gas: -24457 (-6.340%)) 
test_RevertsWhenTooLittleValue_Buy() (gas: -24647 (-6.364%)) 
test_Buy() (gas: -24457 (-6.416%)) 
test_WhenEnded_Finalize() (gas: -24517 (-6.958%)) 
test_RevertsWhenTooSoon_Buy() (gas: -24551 (-7.802%)) 
test_RevertsWhenEnded_Buy() (gas: -24561 (-7.806%)) 
test_RevertsTooMuchValue_Buy() (gas: -24563 (-7.811%)) 
test_RevertsWhenNotOwner_TransferOwnership() (gas: -24462 (-7.930%)) 
test_RevertsWhenNotOwner_Cancel() (gas: -24456 (-7.953%)) 
test_RevertsWhenTooLate_Cancel() (gas: -24468 (-7.958%)) 
test_RevertsWhenNotEnded_Finalize() (gas: -24537 (-7.964%)) 
test_RevertsWhenTooLittleValue_Buy() (gas: -24563 (-7.979%)) 
test_RevertsWhenNotAdmin_CreateSale() (gas: -2095 (-8.064%)) 
test_Buy() (gas: -24456 (-8.073%)) 
test_RevertsWhenNotAdmin_CreateSale() (gas: -2081 (-8.755%)) 
test_Cancel() (gas: -24294 (-9.627%)) 
test_Cancel() (gas: -24468 (-10.826%)) 
test_CreateSale() (gas: -24235 (-11.077%)) 
test_RevertsWhenInitialized_Initialize() (gas: -2128 (-11.795%)) 
test_CreateSale() (gas: -24337 (-12.925%)) 
test_RevertsWhenInitialized_Finalize() (gas: -2062 (-13.762%)) 
test_RevertsWhenInitialized_Buy() (gas: -2095 (-13.959%)) 
Overall gas change: -673443 (-249.656%)

Recommendation

diff --git a/src/minters/LPDA.sol b/src/minters/LPDA.sol
index f52161c..d96efe0 100644
--- a/src/minters/LPDA.sol
+++ b/src/minters/LPDA.sol
@@ -20,10 +20,9 @@ contract LPDA is Initializable, OwnableUpgradeable, ISale {
         uint80 finalPrice;
         uint80 dropPerSecond;
         // slot 3
-        uint96 endTime;
+        uint40 endTime;
         address payable saleReceiver;
-        // slot 4
-        uint96 startTime;
+        uint40 startTime;
     }
 
     struct Receipt {
@@ -46,9 +45,10 @@ contract LPDA is Initializable, OwnableUpgradeable, ISale {
     constructor() {
         factory = msg.sender;
         _disableInitializers();
+        
         uint48 x = type(uint48).max;
         uint80 y = type(uint80).max;
-        uint96 z = type(uint96).max;
+        uint40 z = type(uint40).max;
         address payable i = payable(address(0));
         sale = LPDA.Sale(x, x, i, y, y, y, z, i, z);
     }

diff --git a/src/minters/OpenEdition.sol b/src/minters/OpenEdition.sol
index 004f7fd..2a52f97 100644
--- a/src/minters/OpenEdition.sol
+++ b/src/minters/OpenEdition.sol
@@ -16,10 +16,9 @@ contract OpenEdition is Initializable, OwnableUpgradeable, ISale {
         uint24 currentId;
         address edition;
         // slot 2
-        uint96 startTime;
+        uint40 startTime;
         address payable saleReceiver;
-        // slot 3
-        uint96 endTime;
+        uint40 endTime;
     }
 
     /// @notice sale info for this fixed price edition
@@ -46,9 +45,9 @@ contract OpenEdition is Initializable, OwnableUpgradeable, ISale {
             type(uint72).max,
             type(uint24).max,
             address(0),
-            type(uint96).max,
+            type(uint40).max,
             payable(0),
-            type(uint96).max
+            type(uint40).max
         );
     }

Use assembly to write storage values

contract GasTest is DSTest {
    Contract0 c0;
    Contract1 c1;

    function setUp() public {
        c0 = new Contract0();
        c1 = new Contract1();
    }

    function testGas() public {
        c0.updateOwner(0x158B28A1b1CB1BE12C6bD8f5a646a0e3B2024734);
        c1.assemblyUpdateOwner(0x158B28A1b1CB1BE12C6bD8f5a646a0e3B2024734);
    }
}

contract Contract0 {
    address owner = 0xb4c79daB8f259C7Aee6E5b2Aa729821864227e84;

    function updateOwner(address newOwner) public {
        owner = newOwner;
    }
}

contract Contract1 {
    address owner = 0xb4c79daB8f259C7Aee6E5b2Aa729821864227e84;

    function assemblyUpdateOwner(address newOwner) public {
        assembly {
            sstore(owner.slot, newOwner)
        }
    }
}

Gas Report

╭────────────────────┬─────────────────┬──────┬────────┬──────┬─────────╮
│ Contract0 contract ┆                 ┆      ┆        ┆      ┆         │
╞════════════════════╪═════════════════╪══════╪════════╪══════╪═════════╡
│ Deployment Cost    ┆ Deployment Size ┆      ┆        ┆      ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ 60623              ┆ 261             ┆      ┆        ┆      ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ Function Name      ┆ min             ┆ avg  ┆ median ┆ max  ┆ # calls │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ updateOwner        ┆ 5302            ┆ 5302 ┆ 5302   ┆ 5302 ┆ 1       │
╰────────────────────┴─────────────────┴──────┴────────┴──────┴─────────╯
╭────────────────────┬─────────────────┬──────┬────────┬──────┬─────────╮
│ Contract1 contract ┆                 ┆      ┆        ┆      ┆         │
╞════════════════════╪═════════════════╪══════╪════════╪══════╪═════════╡
│ Deployment Cost    ┆ Deployment Size ┆      ┆        ┆      ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ 54823              ┆ 232             ┆      ┆        ┆      ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ Function Name      ┆ min             ┆ avg  ┆ median ┆ max  ┆ # calls │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ assemblyUpdateOwner┆ 5236            ┆ 5236 ┆ 5236   ┆ 5236 ┆ 1       │
╰────────────────────┴─────────────────┴──────┴────────┴──────┴─────────╯

Lines

• LPDAFactory.sol:24
• LPDAFactory.sol:47
• OpenEditionFactory.sol:24
• OpenEditionFactory.sol:43
• FixedPriceFactory.sol:24
• FixedPriceFactory.sol:43
• Escher721.sol:41
• Escher721.sol:58

Tightly pack storage variables

When defining storage variables, make sure to declare them in ascending order, according to size. When multiple variables are able to fit into one 256 bit slot, this will save storage size and gas during runtime. For example, if you have a bool, uint256 and a bool, instead of defining the variables in the previously mentioned order, defining the two boolean variables first will pack them both into one storage slot since they only take up one byte of storage.

contract GasTest is DSTest {
    Contract0 c0;
    Contract1 c1;

    function setUp() public {
        c0 = new Contract0();
        c1 = new Contract1();
    }

    function testGas() public {
        bool bool0 = true;
        bool bool1 = false;
        uint256 num0 = 200;
        uint256 num1 = 100;
        c0.accessNonTightlyPacked(bool0, bool1, num0, num1);
        c1.accessTightlyPacked(bool0, bool1, num0, num1);
    }
}

contract Contract0 {
    uint256 num0 = 100;
    bool bool0 = false;
    uint256 num1 = 200;
    bool bool1 = true;

    function accessNonTightlyPacked(
        bool _bool0,
        bool _bool1,
        uint256 _num0,
        uint256 _num1
    ) public {
        bool0 = _bool0;
        bool1 = _bool1;
        num0 = _num0;
        num1 = _num1;
    }
}

contract Contract1 {
    bool bool0 = false;
    bool bool1 = true;
    uint256 num0 = 100;
    uint256 num1 = 200;

    function accessTightlyPacked(
        bool _bool0,
        bool _bool1,
        uint256 _num0,
        uint256 _num1
    ) public {
        bool0 = _bool0;
        bool1 = _bool1;
        num0 = _num0;
        num1 = _num1;
    }
}

Gas Report

╭───────────────────────────────────────────┬─────────────────┬───────┬────────┬───────┬─────────╮
│ src/test/GasTest.t.sol:Contract0 contract ┆                 ┆       ┆        ┆       ┆         │
╞═══════════════════════════════════════════╪═════════════════╪═══════╪════════╪═══════╪═════════╡
│ Deployment Cost                           ┆ Deployment Size ┆       ┆        ┆       ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ 122268                                    ┆ 334             ┆       ┆        ┆       ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ Function Name                             ┆ min             ┆ avg   ┆ median ┆ max   ┆ # calls │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ accessNonTightlyPacked                    ┆ 32774           ┆ 32774 ┆ 32774  ┆ 32774 ┆ 1       │
╰───────────────────────────────────────────┴─────────────────┴───────┴────────┴───────┴─────────╯
╭───────────────────────────────────────────┬─────────────────┬───────┬────────┬───────┬─────────╮
│ src/test/GasTest.t.sol:Contract1 contract ┆                 ┆       ┆        ┆       ┆         │
╞═══════════════════════════════════════════╪═════════════════╪═══════╪════════╪═══════╪═════════╡
│ Deployment Cost                           ┆ Deployment Size ┆       ┆        ┆       ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ 126247                                    ┆ 356             ┆       ┆        ┆       ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ Function Name                             ┆ min             ┆ avg   ┆ median ┆ max   ┆ # calls │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ accessTightlyPacked                       ┆ 15476           ┆ 15476 ┆ 15476  ┆ 15476 ┆ 1       │
╰───────────────────────────────────────────┴─────────────────┴───────┴────────┴───────┴─────────╯

Lines

• LPDA.sol:10

Mark functions as payable (with discretion)

You can mark public or external functions as payable to save gas. Functions that are not payable have additional logic to check if there was a value sent with a call, however, making a function payable eliminates this check. This optimization should be carefully considered due to potentially unwanted behavior when a function does not need to accept ether.

contract GasTest is DSTest {
    Contract0 c0;
    Contract1 c1;

    function setUp() public {
        c0 = new Contract0();
        c1 = new Contract1();
    }

    function testGas() public {
        c0.isNotPayable();
        c1.isPayable();
    }
}

contract Contract0 {
    function isNotPayable() public view {
        uint256 val = 0;
        val++;
    }
}

contract Contract1 {
    function isPayable() public payable {
        uint256 val = 0;
        val++;
    }
}

Gas Report

╭────────────────────┬─────────────────┬─────┬────────┬─────┬─────────╮
│ Contract0 contract ┆                 ┆     ┆        ┆     ┆         │
╞════════════════════╪═════════════════╪═════╪════════╪═════╪═════════╡
│ Deployment Cost    ┆ Deployment Size ┆     ┆        ┆     ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ 32081              ┆ 190             ┆     ┆        ┆     ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ Function Name      ┆ min             ┆ avg ┆ median ┆ max ┆ # calls │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ isNotPayable       ┆ 198             ┆ 198 ┆ 198    ┆ 198 ┆ 1       │
╰────────────────────┴─────────────────┴─────┴────────┴─────┴─────────╯
╭────────────────────┬─────────────────┬─────┬────────┬─────┬─────────╮
│ Contract1 contract ┆                 ┆     ┆        ┆     ┆         │
╞════════════════════╪═════════════════╪═════╪════════╪═════╪═════════╡
│ Deployment Cost    ┆ Deployment Size ┆     ┆        ┆     ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ 29681              ┆ 178             ┆     ┆        ┆     ┆         │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ Function Name      ┆ min             ┆ avg ┆ median ┆ max ┆ # calls │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌┼╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌┤
│ isPayable          ┆ 174             ┆ 174 ┆ 174    ┆ 174 ┆ 1       │
╰────────────────────┴─────────────────┴─────┴────────┴─────┴─────────╯

Lines

• OpenEdition.sol:75
• OpenEdition.sol:82
• OpenEdition.sol:89
• OpenEdition.sol:97
• OpenEdition.sol:102
• OpenEdition.sol:107
• OpenEdition.sol:112
• OpenEdition.sol:116
• FixedPrice.sol:50
• FixedPrice.sol:78
• FixedPrice.sol:86
• FixedPrice.sol:91
• FixedPrice.sol:96
• FixedPrice.sol:101
• LPDAFactory.sol:29
• LPDAFactory.sol:46
• OpenEditionFactory.sol:29
• OpenEditionFactory.sol:42
• FixedPriceFactory.sol:29
• FixedPriceFactory.sol:42
• LPDA.sol:92
• LPDA.sol:99
• LPDA.sol:110
• LPDA.sol:117
• LPDA.sol:128
• LPDA.sol:133
• LPDA.sol:138
• LPDA.sol:142
• Escher.sol:21
• Escher.sol:27
• Escher.sol:31
• Escher.sol:38
• Escher.sol:49
• Escher721Factory.sol:27
• Escher721.sol:32
• Escher721.sol:51
• Escher721.sol:57
• Escher721.sol:64
• Escher721.sol:72
• Escher721.sol:78
• Escher721.sol:84