Update Borrow function in NFT Docs

[DappCoderr]

In the Collection resource borrowNFT_ function gives an error.
NonFungibleToken type cast to &{NFT}? is redundant.

Solution: To remove this error add Force Unwrap Operator (!).

[vercel]

@DappCoderr is attempting to deploy a commit to the Flow Team on Vercel.

A member of the Team first needs to authorize it.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 19, 2024 2:40pm

[sisyphusSmiling]

Good catch, thanks for the correction!

[joshuahannan]

I don't think this is correct. This function is supposed to return an optional, so we should not be force unwrapping it here. See ExampleNFT for the correct implementation.

@DappCoderr Can you explain what you're trying to do here? Maybe I'm not understanding

[DappCoderr]

I don't think this is correct. This function is supposed to return an optional, so we should not be force unwrapping it here. See ExampleNFT for the correct implementation.

@DappCoderr Can you explain what you're trying to do here? Maybe I'm not understanding

I understand that the value should be returned as optional. However, when I use it, I encounter the compilation error "cast to &{NFT}? is redundant."

To resolve this, I've added the force unwrap operator (!).

[joshuahannan]

That isn't an error, it is just a warning, so the contract will still deploy if you ignore it.

That is just pointing out that the entire cast is redundant, not the ?, if you just remove the cast completely, it will work properly.

        access(all) view fun borrowNFT(_ id: UInt64): &{NonFungibleToken.NFT}? {
            return &self.ownedNFTs[id]
        }

Can you update the PR to just have that? Thank you!

[DappCoderr]

Thanks for the clarification, @joshuahannan
I'll remove the redundant cast and push the updated PR.