Contract returns default value

Question

I'm trying to create a contract which calls boolean method on another contract. However, for some reason it doesn't work.

Here is an example:

pragma solidity ^0.4.23;

contract Season {
    address[] public requests;

    constructor() public {
        requests.push(new Request());
    }

    function getRequestsByFilter() public view returns(bool) {
        Request r = Request(requests[0]);
        bool ret = r.isMatch();
        return ret;
    }
}

contract Request {  
    function isMatch() public pure returns(bool) {
        return true;
    }    
}

It compiles fine but if you call R.foo from web3.js you get false instead of true.

Here is what I see in UI:

I don't understand why it happens and how can I call this method.

---

Here is smaller example:

pragma solidity ^0.4.23;

contract Season {
    function getRequestsByFilter() public returns(bool) {
        Request r = new Request();
        bool ret = r.isMatch();
        return ret;
    }
}

contract Request {  
    function isMatch() public pure returns(bool) {
        return true;
    }    
}

Answer 1

Here, the Hub deploys Spoke contracts that store a simple string.

To iterate over the strings stored in all the Spokes you 1) get the spokesCount from the Hub with hub.getSpokeCount() 2) get the spokeAddress from the Hub, by row number with hub.spokes(row) 3) instantiate a Spoke with spoke = Spoke.at(spokeAddress), 4) invoke functions in the spokes such as Set with spoke.setString("foo"), and Get with spoke.storedString()

pragma solidity 0.4.23;

contract Owned {

    address public owner;

    constructor() public {
        owner = msg.sender;
    }

    modifier onlyOwner {
        require(msg.sender == owner);
        _;
    }

}

contract Spoke {

    string public storedString;

    event LogSetHubString(address sender, string _string);

    function setString(string _string) public returns(bool success) {
        emit LogSetHubString(msg.sender, _string);
        storedString = _string;
        return true;
    }

}

contract Hub is Owned {

    address[] public spokes;
    mapping(address => bool) public isSpoke;

    event LogNewSpoke(address sender, address spoke);

    constructor() public { owner = msg.sender; }

    function getSpokeCount() public view returns(uint count) { return spokes.length; }

    function newSpoke() public onlyOwner returns(address newContract) {
        Spoke s = new Spoke();
        spokes.push(address(s));
        isSpoke[address(s)] = true;
        emit LogNewSpoke(msg.sender, address(s));
        return address(s);
    }

}

Hope it helps.

Answer 2

It was my blockchain bug. It was resolving by applying byzantium EIPs which fixes this issue, specifically saying: 140, 211, 214, 658. It seems that this behavior is fixed by 214 alone, but others are good too.