Trick or Treat

Relevant Github Link: https://github.com/Cyfrin/2024-10-trick-or-treat/blob/9cb3955058cad9dd28a24eb5162a96d759bfa842/src/TrickOrTreat.sol#L9

Description: The Ownable contract from OpenZeppelin was inherited incorrectly as shown below:

causing the compiler error:

The error occurs because Ownable does not require any arguments when inherited. Instead, OpenZeppelin’s Ownable automatically sets msg.sender as the contract owner upon deployment.

Impact:

1. This issue causes deployment failure leading to delay in project timelines.

2. It could result in misconfigured access control if not fixed properly before deployment.

Recommended Mitigation:
The recommended fix is to remove msg.sender from the inheritance line, as follows:

This corrects the inheritance, allowing Ownable to initialize msg.sender as the owner upon deployment without requiring additional parameters.