zerolawtech / ZAP-Tech

ZeroLaw Augmentation Protocol
GNU General Public License v3.0
27 stars 13 forks source link

NFToken.mint events are showing incorrect recipient #19

Closed iamdefinitelyahuman closed 4 years ago

iamdefinitelyahuman commented 5 years ago

The Issue

Regardless of which address tokens are minted to, the Transfer and TransferRange events that fire will show them as being minted to msg.sender:

https://github.com/iamdefinitelyahuman/ZAP-Tech/blob/62f8434e9033a930f61e3ddb968a5ecd227ef535/contracts/NFToken.sol#L373-L374

How to Fix

Change msg.sender to _owner in the above lines, and maybe add some unit tests to check the correctness of the events being fired.