Require statements without messages in Factory

[code423n4]

Handle

pants

Vulnerability details

The functions Factory.setOwnerSplit(), Factory.proposeBasketLicense() and Factory.createBasket() contain a bunch of require statement without messages:

require(newOwnerSplit <= 2e17); // 20%

require(licenseFee >= minLicenseFee);

require(bProposal.basket == address(0));

Impact

When transactions revert, the users won't receive error messages indicating the cause of the failure.

Tool Used

Manual code review.

Recommended Mitigation Steps

Add informative error messages to these require statements.

[GalloDaSballo]

Personally I disagree that this would be a finding at all, that said I can rationalize these being informational findings, hence non-critical