Wallet.sol: change to onlymanyowners modifier breaks administrative functions

[brynbellomy]

I think the most recent change to wallet.sol broke all of the administrative functions (i.e., those with the onlymanyowners modifier).

See relevant commit + line: 2849dab#diff-ed99a3039a35a9961cc9c6735a7099baR49

confirmAndCheck returns either true or false depending on whether or not the function call has been approved by M-of-N owners. However, regardless of return value, it makes important state changes that count the confirmations received thus far. The new implementation of onlymanyowners reverts state changes (via require) any time confirmAndCheck returns false, making it impossible to hit the confirmation threshold for any function. No tallying ever happens.

This makes it impossible to change any of the wallet's properties such as ownership, required confirmations, etc.

[brynbellomy]

I can PR a fix if someone on your team can confirm the bug

[illuzen]

You guys don't have any unit tests?...