error NotOwner(); - Expected ';' but got '('

[czilauto0r1]

Hi, so after first problem I go back to some quick JavaScript course pass that and now I'm back to solidity. I did everything from begin no errors everything fine now, again I have some stupid issue, so my code is exactly like in code lesson but I'm getting weird error.

My code is:
//SPDX-License-Identifier: MIT

pragma solidity ^0.8.0; //Get funds from users // Withdraw funds // set a minimum funding value in usd

// contract cost 819,290 gas import "./PriceConverter.sol"; import "@chainlink/contracts/src/v0.8/interfaces/AggregatorV3Interface.sol";

error NotOwner();

contract FundMe {

and when I'm trying compile I get error from solidity:
ParserError: Expected ';' but got '(' --> contracts/FundMe.sol:12:15: | 12 | error NotOwner(); | ^

I was reading solidity by example, checking on internet, comparing code with code from lesson, I run out of ideas.

[Rishi860]

Can you paste the whole FundMe.sol file here?

[czilauto0r1]

//SPDX-License-Identifier: MIT

pragma solidity ^0.8.0;
//Get funds from users
// Withdraw funds
// set a minimum funding value in usd

// contract cost 819,290 gas
import "./PriceConverter.sol";
import "@chainlink/contracts/src/v0.8/interfaces/AggregatorV3Interface.sol";

error NotOwner();

contract FundMe {
using PriceConverter for uint256;

uint256 public constant MINIMUM_USD = 50 * 1e18;

address[] public funders;
mapping(address => uint256) public addressToAmountFunded;
address public immutable i_owner;
constructor() {
    i_owner = msg.sender;
}

function fund() public payable {
    //We want to be able to set minimum funding value
    // 1. How do we send eth to this contract
    require(msg.value.getConversionRate() >= MINIMUM_USD, "Didnt send enough"); // 1e18 == 1 * 10 ** 18 == 1000000000000000000 = 1 eth
    // if you need to require something with revert use require
    funders.push(msg.sender);
    addressToAmountFunded[msg.sender] = msg.value;
}

function withdraw() public onlyOwner {
    // require(msg.sender == owner, "Sender is not owner");
    // for loop
    // [1, 2, 3, 4]
    //  0, 1, 2, 3  <- indexes
    // for(/* starting index, ending index, step amount */ )
    // 0, 10, 2 <- from 0 to 10 by 2 so: 0,2,4,6,8,10
    for (uint funderIndex = 0; funderIndex < funders.length; funderIndex++) {
        address funder = funders[funderIndex];
        addressToAmountFunded[funder] = 0;
    }
    // reset array
    funders = new address[](0);
    //actually withdraw the funds
    (bool callSuccess, ) = payable(msg.sender).call{value: address(this).balance}("");
    require(callSuccess, "Call failed");
    // transfer method
    // msg.sender == address
    // payable(msg.sender) = payable
    // payable msg.sender.transfer(address(this).balance);
    // send method
    // bool sendSuccess = payable(msg.sender).send(address(this).balance);
    // require(sendSuccess, "Send failed");
    // call method <- RECOMMENDED METHOD
    // (bool callSuccess, ) = payable(msg.sender).call{value: address(this).balance}("");
    // require(callSuccess, "Call failed");
}

modifier onlyOwner {
    if(msg.sender != i_owner) {revert NotOwner();}
    // require(msg.sender == i_owner, "Sender is not owner"); // <- do this first
    _; // <- rest of code
}

}

[Rishi860]

I see no error in error NotOwner();
Try compiling again

[czilauto0r1]

keep getting that error... // refreshed remix still that error showing

[czilauto0r1]

Problem is: I call that error NotOwner but should be notOwner. Now it's working. Sorry for troubles.

[Rishi860]

Nice, but that was not the error your compiles warn you about. Besides I did NotOwner() and it works for me. Infact I copied your code and compiled it and that works too :)

[czilauto0r1]

yeah I don't know still what was making that error, just changed on small letter and its working so I keep going maybe in future I will find answer :)

[mattjaf]

that's a common syntactical error for missing a ; or { , } or some thing like that