payable

[fulldecent]

Currently transfer functions & approve do not allow payable function implementations.

This is okay but it should be documented that this is more restrictive than the standard.

https://github.com/0xcert/ethereum-erc721/blob/master/contracts/tokens/NFToken.sol

[MoMannn]

#154

[fulldecent]

Confirmed fixed in latest branch

[fulldecent]

This documentation should also be updated in the interfaces.

[MoMannn]

I would add to @dev: This function can be changed to payable.
Since if anyones wants to make it payable he can do that by himself. I would then add to readme that this implementation does not support payable out of the box.

@fulldecent You ok with this?

[fulldecent]

This is good. But should also specifically say there "this implementation is more restrictive than the ERC-721 standard requires" for the avoidance of doubt.

[MoMannn]

#272