Fix stupid behavior that we copied for GovernorBravo

Amxx · Amxx · commit 2ef4b150b321 · 2021-09-06T19:32:30.000+02:00
diff --git a/contracts/governance/extensions/GovernorTimelockCompound.sol b/contracts/governance/extensions/GovernorTimelockCompound.sol
@@ -178,7 +178,7 @@ abstract contract GovernorTimelockCompound is IGovernorTimelock, Governor {
         uint256 eta = proposalEta(proposalId);
         require(eta > 0, "GovernorTimelockCompound: proposal not yet queued");
         for (uint256 i = 0; i < targets.length; ++i) {
-            _timelock.executeTransaction{value: values[i]}(targets[i], values[i], "", calldatas[i], eta);
+            _timelock.executeTransaction{value: i == 0 ? msg.value : 0}(targets[i], values[i], "", calldatas[i], eta);
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ abstract contract GovernorTimelockCompound is IGovernorTimelock, Governor {`
`178`	`178`	`uint256 eta = proposalEta(proposalId);`
`179`	`179`	`require(eta > 0, "GovernorTimelockCompound: proposal not yet queued");`
`180`	`180`	`for (uint256 i = 0; i < targets.length; ++i) {`
`181`		`- _timelock.executeTransaction{value: values[i]}(targets[i], values[i], "", calldatas[i], eta);`
	`181`	`+ _timelock.executeTransaction{value: i == 0 ? msg.value : 0}(targets[i], values[i], "", calldatas[i], eta);`
`182`	`182`	`}`
`183`	`183`	`}`
`184`	`184`