[ENHANCEMENT] Deprecate HasItem callback

[i-kulgu]

Description

Deprecate HasItem callback since it's not used anymore in any QB script.

Checklist

• [x ] I have personally loaded this code into an updated qbcore project and checked all of its functionality.
• [x ] My code fits the style guidelines.
• [ x] My PR fits the contribution guidelines.

[i-kulgu]

If deprecated, shouldn't it be fully removed anyways? We have logs / backups of every change through this github repository. So no need to comment out the code, it just makes it less readable and such.

I will gladly approve this if you remove the callback instead of commenting it :)

I had let it stay so people who STILL have the HasItem callback could easily uncomment it to use but i have removed as you asked.

[Z3rio]

If deprecated, shouldn't it be fully removed anyways? We have logs / backups of every change through this github repository. So no need to comment out the code, it just makes it less readable and such. I will gladly approve this if you remove the callback instead of commenting it :)

I had let it stay so people who STILL have the HasItem callback could easily uncomment it to use but i have removed as you asked.

Yeah, I understand. I will take a look at this PR soon 👍🏼 Although by the looks of it, qb-houserobbery & qb-tunerchip still use these callbacks, and therefore I would need to change those to the actual export before committing this PR.