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

No free wiring in log walls #77203

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sparr
Copy link
Member

@sparr sparr commented Oct 20, 2024

Summary

Content "No free wiring in log walls"

Purpose of change

I suspect this flag was added because existing log cabins are expected to have power outlets and light fixtures and such. However, a side effect is that players can build log walls and get free wiring from them.

Describe the solution

Removed the WIRED_WALL flag from t_wall_log

Describe alternatives you've considered

We could add a new log wall type used by existing modern log cabins that includes wiring, and let the player build the kind where they have to add wiring.

Testing

Tried to reveal wall wiring on a log wall.

Additional context

This is the only player-constructable wall that has magical free wiring.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Oct 20, 2024
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 Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant