Remove transfer with permit

[MikeHathaway]

Description of change

High level

• Removes unnecessary transferFromWithPermit method. We weren't using Permit as intended, as interactions with permit would normally occur in external contracts where the spender is the actual initiator of the transaction. There's no vulnerability, but I believe it's better to simply remove this function to simplify the code.
  • Removes unnecessary function and updates tests. Implements new contractNFTSpender utility contract to handle permit interactions.

Description of bug or vulnerability and solution

• No vulnerability, but not useful either.