ERC20 token spec is being violated #7
Description
We did a big no-no and modified the code of the ERC20 token specification to account for some particular situations on our contracts. This apparently is not something that we should be doing (which makes sense, as we can be possibly be introducing vulnerabilities in a well specified spec).
Instead, we should be using the pre-defined methods for interacting with the token via outside contracts (namely, approveAndCall) where we track transfers of tokens in our outside contract and use that to guide our application's flow.