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

Can install multiple alternators on a single engine #39578

Closed
OddSkollar opened this issue Apr 15, 2020 · 2 comments
Closed

Can install multiple alternators on a single engine #39578

OddSkollar opened this issue Apr 15, 2020 · 2 comments

Comments

@OddSkollar
Copy link
Contributor

Describe the bug

Player is able to attach multiple alternators to an engine with the E_ALTERNATOR flag.

Steps To Reproduce

Install multiple alternators.

Expected behavior

Should have an error message similar to that if a player attempts to install multiple turrets on the same slot.

Screenshots

image

Versions and configuration

  • OS: Windows
    • OS Version: 10.0 1903
  • Game Version: 0.E-439-gdf00ef1ba6-dirty [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    TESTING DATA [test_data]
    ]

Additional context

Currently addressing this myself as part of implementation of a new feature which will be PRed soon, just want a paper trail in case I mess it up horribly.

OddSkollar added a commit to OddSkollar/Cataclysm-DDA that referenced this issue Apr 15, 2020
add check for extant alternator on attempted install slot, hopefully addressing CleverRaven#39578
add vehicle_part::is_alternator()
add VPFLAG_POWERED_TURRET_MOUNT to C++ code
fix relevant JSON lexer issues for new content
@l29ah
Copy link
Contributor

l29ah commented Apr 15, 2020

I don't see how is this unbalanced or unrealistic.

@mlangsdorf
Copy link
Contributor

Not a bug, working as intended. You can run multiple alternators off different drive belts driven by the same drive shaft.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants