Serban Simu

The exact use case we ran into was an extension of the ERC721 that calls `_isApprovedOrOwner()` with the result of `ecrecover()` which returns 0 for a non-matching signature, thus the...

Yes agreed. I still think it is worth adding a prerequisites or requirements section to the function comment when there is potential for mistakes like this but this is up...

I verified the fix on the main branch - works well, thank you! (and been patiently waiting for that tag :) )