Fix use menu denial message and invoking items for comestibles without charges

[mqrause]

Summary

None

Purpose of change

Fixes #67330

Describe the solution

Exit early when checking item::ammo_sufficient for comestibles, so the menu and invoking items accepts comestibles as having the required charge.

Describe alternatives you've considered

This whole chain of item::ammo_sufficient > item::shots_remaining > item::ammo_required > item::ammo_remaining is really ugly, because it shouldn't even have anything to do with comestibles. But it's still checking them, expects comestibles to use 1 charge, but now obviously fails for uncharged comestibles. This really should get a rework to properly split things up, but that requires quite a bit more work.

Testing

Activated a caffein pill from the use menu.

Additional context

[mqrause]

Looks like this won't work, unfortunately.

Actually similar enough solution to justify reopening this.

[Hirmuolio]

I feel like adding extra checks for this in ammo_sufficient is not good. I think it would be better to have that check somewhere around here

Cataclysm-DDA/src/game_inventory.cpp

Line 1105 in b4c5469

!it.ammo_sufficient( &you, uses.begin()->first ) ) {

[mqrause]

I don't like the whole setup either, but it's what ammo_sufficient actually checks and it does so at multiple places. Before the charge removal, it'd go deeper into the chain and check if charges is larger than qty. Which is always true for comestibles if I'm not mistaken because qty will always be 1 and charges will always be >= 1 if the item actually exists.

It'd be great to clean this all up, but I'm currently not willing to dive into that. Most important is making the use menu functional with comestibles and actually allow comestibles with use actions to be actually used.