My fallback function can not get msg.value from a sent transaction

Question

There is a mapping in my code as mapping(address => uint) private Payment;. This mapping is used to see who send some ether and how much the sent value is. Its key is an address that stores address of sender and its value is uint that stores the sent value. I need this mapping's records be evaluated in fallback function. So, i write the followed fallback:

function() external payable{
        Payment[msg.sender]=msg.value;
    }

But, whenever i pay some ethers to the contract with Metamask, the transaction will be failed. What should i do?

Answer 1

The error is likely that you don't allocate sufficient gas to the transaction. In fact you have to pay for the ETH transfer (21000 gas) + for the storage operation done inside the fallback function.

Therefore try to increase the transaction gas in Metamask options.

Note that the fallback function implementation has been changed in the 0.6 Solidity release (https://solidity.ethereum.org/2020/03/26/fallback-receive-split/).