Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plutonium reloading broken #16741

Closed
Coolthulhu opened this issue May 18, 2016 · 8 comments

Comments

Projects
None yet
5 participants
@Coolthulhu
Copy link
Contributor

commented May 18, 2016

Affects native plutonium items (RM13) and atomic battery mods.
When reloaded, those items consume a single plutonium cell charge and gain number of charges equal to size of the stack.

For example, a stack of 10 cells used to reload a RM13 will grant the RM13 10 charges and become a stack of 9 cells.

Additionally, atomic battery mod applied to items with batteries in them will turn it unreloadable. It can be fixed by applying a screwdriver to the modded item, loading it with batteries, unloading the batteries and reapplying the mod.

@Coolthulhu Coolthulhu added the <Bug> label May 18, 2016

@karols146

This comment has been minimized.

Copy link
Contributor

commented May 18, 2016

#16273 Similar

@illi-kun

This comment has been minimized.

Copy link
Member

commented May 22, 2016

@player0981234 your endless complains are quite annoying.

@mugling

This comment has been minimized.

Copy link
Contributor

commented Nov 27, 2016

Lots of recent changes to related code. Confirmation anyone?

@BorkBorkGoesTheCode

This comment has been minimized.

Copy link
Contributor

commented Nov 27, 2016

Currently, reloading a RM-13 armor will remove 1 plutonium cell from inventory and add 500 charges to the armor.

EDIT: The stack size is 20.

@mugling

This comment has been minimized.

Copy link
Contributor

commented Nov 27, 2016

That sounds correct

@Coolthulhu

This comment has been minimized.

Copy link
Contributor Author

commented Nov 27, 2016

All other ammo types reload all at once. Unless RM-13 has the RELOAD_ONE flag, it shouldn't act like that.

@mugling

This comment has been minimized.

Copy link
Contributor

commented Nov 27, 2016

Unless RM-13 has the RELOAD_ONE flag

It doesn't and #19502 doesn't change anything in that repsect

@mugling mugling self-assigned this Nov 27, 2016

@mugling

This comment has been minimized.

Copy link
Contributor

commented Nov 27, 2016

Bug is probably in item::reload_option::qty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.