Unreachable code due to checked conditions
```\\nfunction revealBid(\\n    bytes32 auctionID,\\n    uint256 price,\\n    uint256 targetSBTAmount,\\n    uint256 random\\n) public override {\\n    require(\\n        inRevealingValuationPeriod(auctionID),\\n        "it is not the time to reveal the value of bids"\\n    );\\n```\\n\\nHowever, later in the same function, code exists to introduce “Penalties for revealing too early.” This checks to see if the function was called before closing, which should not be possible given the previous check.\\n```\\n/\\*\\*\\n \\* @dev Penalties for revealing too early.\\n \\* Some participants may not follow the rule and publicate their bid information before the reveal process.\\n \\* In such a case, the bid price is overwritten by the bid with the strike price (slightly unfavored price).\\n \\*/\\nuint256 bidPrice = price;\\n\\n/\\*\\*\\n \\* @dev FOR TEST CODE RUNNING: The following if statement in L533 should be replaced by the comment out\\n \\*/\\nif (inAcceptingBidsPeriod(auctionID)) {\\n    // if (false) {\\n    (, , uint256 solidStrikePriceE4, ) = \\_getBondFromAuctionID(auctionID);\\n    bidPrice = \\_exchangeSBT2IDOL(solidStrikePriceE4.mul(10\\*\\*18));\\n}\\n```\\n
Resolution\\nComment from Lien Protocol:\\nDouble-check the logic in these functions. If revealing should be allowed (but penalized in the earlier stage), the first check should be changed. However, based on our understanding, the first check is correct, and the second check for early reveal is redundant and should be removed.
```\\nfunction revealBid(\\n    bytes32 auctionID,\\n    uint256 price,\\n    uint256 targetSBTAmount,\\n    uint256 random\\n) public override {\\n    require(\\n        inRevealingValuationPeriod(auctionID),\\n        "it is not the time to reveal the value of bids"\\n    );\\n```\\n