Skip to content

Commit a793506

Browse files
authored
Merge pull request #34 from ProjectBabbage/fix/amount-0
Do not transfer on amount 0
2 parents fa63579 + e45b39f commit a793506

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/WeSplit.sol

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,8 @@ contract WeSplit is UUPSUpgradeable, WeSplitStructure {
120120
for (uint256 i; i < sParticipants.length; i++) weightsSum += split.weights[i];
121121
for (uint256 i; i < sParticipants.length; i++) {
122122
uint256 shareOfAmount = (sAmount * split.weights[i]).divUp(weightsSum);
123-
IERC20(sToken).transferFrom(sParticipants[i], address(this), shareOfAmount);
123+
if (shareOfAmount != 0)
124+
IERC20(sToken).transferFrom(sParticipants[i], address(this), shareOfAmount);
124125
}
125126
IERC20(sToken).transfer(split.receiver, sAmount);
126127
split.amount = 0;

0 commit comments

Comments
 (0)