Skip to content

Allow wearing non-covering items with power armor#60533

Merged
dseguin merged 1 commit intoCleverRaven:masterfrom
n0pen:fix/power-armor-and-non-covering
Sep 1, 2022
Merged

Allow wearing non-covering items with power armor#60533
dseguin merged 1 commit intoCleverRaven:masterfrom
n0pen:fix/power-armor-and-non-covering

Conversation

@n0pen
Copy link
Contributor

@n0pen n0pen commented Aug 28, 2022

Summary

Bugfixes "Allow wearing non-covering items with power armor"

Purpose of change

Non-covering items should be allowed to be worn as they don't obstruct power armor (or any other armor)

Describe the solution

Add an exception when calculating power armor compatible equipment to allow non-covering items

Describe alternatives you've considered

None

Testing

  1. Spawn and get naked
  2. Spawn and equip power armor
  3. Spawn hairpin and check it can be worn along with power armor
  4. Spawn jacket and check it can't be worn as it covers many parts of the body

Additional context

There is currently a power armor rework in the making but this can be a temporary fix

Currently the game will not allow any item incompatible with power armor
Non-covering items should be allowed as they don't obstruct power armor
@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Aug 28, 2022
@dseguin dseguin merged commit ef92ebd into CleverRaven:master Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants