On 4/21/2016 11:12 AM, Christina Fu wrote:
In general, the idea is good to use something more descriptive than "OP_DO_TOKEN" in activity report, however, I think "OP_DO_TOKEN" was intended for any activities that were performed on the token records, so perhaps "OP_TOKEN_*" (e.g. OP_TOKEN_MODIFY) instead of "OP_*" would be more appropriate?
Also, in TPSTokendb.java, where the three activities report that revocation is not permitted, why do they have "success". Shouldn't they be "failure" since the op was not allowed to happen? I know you didn't touch that code, but I'm just wondering.
Christina
The patch has been revised as described, ACKed over IRC (thanks!), and pushed to master. The activity logs are indeed incorrect and they will be fixed in subsequent patches. -- Endi S. Dewata