Fix for ordering ammo bins for repair

[Krashner]

This is a fix for #1185 which now allows the player to order ammo bins for repair. Previously, when attempting to order a new ammo bin, the player would instead receive ammunition.

[NickAragua]

One general comment, I would avoid updating history.txt with the PR, as that file gets updated pretty frequently and results in a lot of unnecessary merges. Usually, the person merging the PR updates that file.

[Krashner]

Sorry about that, I've closed it out and I'll fix it.

[Krashner]

Alright, it should all should be square now. I've learned that Github's web interface apparently lacks a way to easily revert changes.

[NickAragua]

Sorry about the delay, I've been a little out of the loop for a while. Does ordering ammo still work properly after this fix?

[Krashner]

Yea, ammo ordering still works, I just tested it again just in case. getNewPart seems to handle the ordering of ammunition, whereas getAcquisitionWork now handles replacing the bin.

However, I have noticed a new issue, ammo bins cost 0 c-bills.

if (getShotsPerTon() <= 0) { return Money.zero(); }

Since the ammo bin is empty, when it checks for cost it returns 0.

[NickAragua]

I always thought ammo bins cost 0, since they're basically just an imaginary component made up for the purposes of MekHQ.

[Krashner]

I have no idea, that might be the intended behavior. It could be changed so that replacing the bin also orders an ammo refill as well. That way you're not technically just buying a free part and you don't have to order ammunition separately.

[NickAragua]

Yeah, I checked and it is that way. Let's keep the ammo bin and ammo ordering separate, since there are multiple possible ammo types that can go into a bin and we don't really know which one we should be ordering.

[Krashner]

Alright, that sounds good to me.