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

Portable Pump Attach #7637

Closed
linax3-genuine opened this issue Dec 5, 2021 · 6 comments
Closed

Portable Pump Attach #7637

linax3-genuine opened this issue Dec 5, 2021 · 6 comments
Labels
Code Programming task Design Design-related task

Comments

@linax3-genuine
Copy link

  • [x ] I have searched the issue tracker to check if the issue has already been reported.

Description
it appears that only the mechanic can properly reattach portable pumps.
if done by another, it fails and gives error message when trying to reattach:
too many of this item type placed on this submarine x/0

Steps To Reproduce
create 2 portable pumps;
attach 2 portable pumps;
detach 1 portable pump;
try to reattach 1 portable pump with non-mechanic;
maybe, an error message: too many of this item type placed on this submarine 1/0;

Version
v0.15.13.0.
windows

Additional information
done some draining and wiring, humpback submarine, inserted fulgurium batteries, tried all other crew professions, only mechanic works properly; singleplayer campaign; docked to station.

@MaaiG
Copy link

MaaiG commented Dec 5, 2021

I haven’t been able to install (none of the characters) no one portble pumps (submarine editor)!
Image 001_02-13 (1) (1) (1) (1) (1) (1)

@Regalis11 Regalis11 added Bug Something isn't working Code Programming task labels Dec 6, 2021
@Rokvach
Copy link
Collaborator

Rokvach commented Dec 6, 2021

I encountered the issue of not being able to install portable pumps as well in singleplayer round where I spawned in a portable pump and tried to place it on a wall.

@linax3-genuine linax3-genuine changed the title Portable Pump Detach Portable Pump Attach Dec 6, 2021
@Regalis11 Regalis11 added Design Design-related task and removed Bug Something isn't working labels Dec 7, 2021
@M61Dev
Copy link

M61Dev commented Dec 7, 2021

It is intentional that only the Mechanic with the Let It Drain talent can place the pumps, because otherwise you could place as many pumps in a submarine as you wanted, which would trivialize the game (among other things).

However, the item message is misleading and looks like a programming error. For this reason, I changed the logic to display a special message when you try to place a pump without the talent in https://github.com/Regalis11/Barotrauma-development/commit/2e521d0e4a7ee6004fee3913b4ed7f087f3c3aef

@linax3-genuine
Copy link
Author

interesting. that needs less-ambiguous design.
you should allow all to attach them and tie them(the restriction) to the submarine, as the design intention seems to be.
its the single item that does not conform to the item usage mechanics (i cant think of another).

ill have to wait and see this new message, maybe its good enough.

other than that, the restriction is nonsense, but i understand the need for it, for such an overpowered item.

im new here and i see this "close with comment" button. i donk know who is to decide and close those issues.
i think this is resolved for now, with the mentioned change.

@Regalis11
Copy link
Collaborator

im new here and i see this "close with comment" button. i donk know who is to decide and close those issues.

We generally keep the issues open until @Rokvach has tested and verified that the fix is working as intended. :) So no need to close it yourself.

@Rokvach
Copy link
Collaborator

Rokvach commented Jan 4, 2022

Tested, working correctly. Closing.

@Rokvach Rokvach closed this as completed Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Programming task Design Design-related task
Projects
None yet
Development

No branches or pull requests

5 participants