Unable to update state variables of inherited contracts

Question

I'm trying to write an extension of OpenZeppelin's CappedCrowdsale to allow for a hard cap in USD, as I think this is more human-readable than having everything in wei.

However, I'm having issues updating some of the inherited state variables (Crowdsale's rate and CappedCrowdsale's cap). I try to do this in the updateEthPrice function, which I know is getting called because the ethPriceInDollars variable gets updated correctly, but cap and rate always return 1... (which is what they were initialized with)

At first I thought that maybe the inherited constructors were firing after the base constructor (and overriding whatever I set in the base constructor), but even if I call updateEthPrice once the contract has been deployed, I'm still not seeing any changes to cap and rate

Been banging my head on this all day, any help is appreciated!

pragma solidity ^0.4.19;

import "zeppelin-solidity/contracts/crowdsale/CappedCrowdsale.sol";


/**
 * @title DollarCappedCrowdsale
 * @author Gianni Settino
 * @dev Extension of CappedCrowdsale with a cap in USD instead of wei.
 * This means that Crowdsale's rate and CappedCrowdsale's cap now depend
 * on ETH's USD price, which can be changed up until the start of the token sale.
 */
contract DollarCappedCrowdsale is CappedCrowdsale(1), Ownable {
  uint256 public ethPriceInDollars;
  uint256 public hardCapInDollars;
  uint256 public tokenPriceInCents;

  // @dev We temporarily init Crowdsale's rate and CappedCrowdsale's cap to 1.
  // Reason: their "real" values are subsequently calculated in updateEthPrice()
  function DollarCappedCrowdsale(
    uint256 _startTime,
    uint256 _endTime,
    address _wallet,
    uint256 _ethPriceInDollars,
    uint256 _hardCapInDollars,
    uint256 _tokenPriceInCents
    )
    public
    Crowdsale(_startTime, _endTime, 1, _wallet)
  {
    require(_hardCapInDollars > 0);
    require(_tokenPriceInCents > 0);
    hardCapInDollars = _hardCapInDollars;
    tokenPriceInCents = _tokenPriceInCents;
    updateEthPrice(_ethPriceInDollars);
  }

  function updateEthPrice(uint256 _ethPriceInDollars) public onlyOwner {
    require(now < startTime);
    require(_ethPriceInDollars > 0);
    ethPriceInDollars = _ethPriceInDollars;
    cap = (hardCapInDollars / ethPriceInDollars).mul(1 ether);
    rate = (ethPriceInDollars.mul(100)) / tokenPriceInCents;
  }
}

Answer 1

I've tried your contractin remix and it works as expected. The modified version to be used with latest version of OpenZeppelin contracts.

pragma solidity ^0.4.19;

import "github.com/OpenZeppelin/zeppelin-solidity/contracts/crowdsale/CappedCrowdsale.sol";

contract DollarCappedCrowdsale is CappedCrowdsale(1), Ownable {
  uint256 public ethPriceInDollars;
  uint256 public hardCapInDollars;
  uint256 public tokenPriceInCents;

  function DollarCappedCrowdsale(
    uint256 _startTime,
    uint256 _endTime,
    address _wallet,
    uint256 _ethPriceInDollars,
    uint256 _hardCapInDollars,
    uint256 _tokenPriceInCents
    )
    public
    Crowdsale(_startTime, _endTime, 1, _wallet, MintableToken(1))
  {
    require(_hardCapInDollars > 0);
    require(_tokenPriceInCents > 0);
    hardCapInDollars = _hardCapInDollars;
    tokenPriceInCents = _tokenPriceInCents;
    updateEthPrice(_ethPriceInDollars);
  }

  function updateEthPrice(uint256 _ethPriceInDollars) public onlyOwner {
    require(now < startTime);
    require(_ethPriceInDollars > 0);
    ethPriceInDollars = _ethPriceInDollars;
    cap = (hardCapInDollars / ethPriceInDollars).mul(1 ether);
    rate = (ethPriceInDollars.mul(100)) / tokenPriceInCents;
  }
}

Creation parameters

1518412209,1518413209,"0x01","870","40000000","10"

Resulting parameters:

• rate: 8700
• tokenPriceInCents: 10
• hardCapInDollars: 40000000
• ethPriceInDollars: 870
• cap: 45977000000000000000000

Answer 2

As I suspected, you're inheriting the CappedCrowdsale Contract incorrectly. See the answer below and let me know how that works out.

contract SampleCrowdsale is CappedCrowdsale, RefundableCrowdsale {

    function SampleCrowdsale(uint256 _startTime, uint256 _endTime, uint256 _rate, uint256 _goal, uint256 _cap, address _wallet, MintableToken _token) public
        CappedCrowdsale(_cap)
        FinalizableCrowdsale()
        RefundableCrowdsale(_goal)
        Crowdsale(_startTime, _endTime, _rate, _wallet, _token)
    {
    //As goal needs to be met for a successful crowdsale
    //the value needs to less or equal than a cap which is limit for accepted funds
        require(_goal <= _cap);
    }
}