search

Giveth / vaultcontroller

Vault Architecture
GNU General Public License v3.0
0 stars 3 forks source link

Removing non-existent recipient does not fail #34

Closed adria0 closed 7 years ago

adria0 commented 7 years ago 
vaultController.authorizeSpender("SPENDER", spender, 10, 2, 3, 7, 6 )
vaultController.removeAuthorizedRecipient(spender,noAddedYetRecipient) // does not throw

Consider to throw since recipient was not authorized before

jbaylina commented 7 years ago

I do not throw because the action is done. After this function I guaranty that the spender is not authorized.