Skip to content
This repository has been archived by the owner on May 25, 2024. It is now read-only.

LEG rotation fix #249

Merged
merged 1 commit into from
May 6, 2024
Merged

Conversation

AbdielKavash
Copy link
Member

@AbdielKavash AbdielKavash commented Apr 6, 2024

Previously, the Large Essentia Generator could only be rotated to two possible orientations: either with the main structure below the controller (if the face of the controller was pointed towards any cardinal direction), or with the structure on the south side of the controller (if the face of the controller was pointing up or down). This was caused by the LEG attempting to do its own rotation logic (??) instead of inheriting the one from GT_MetaTileEntity_EnhancedMultiblockBase and reimplementing it poorly.

This PR makes it so that the LEG can be rotated to any direction, allowing for more varied and compact setups. Flipping the LEG is still allowed, although it makes no functional difference (since the structure is symmetrical).

@AbdielKavash AbdielKavash requested a review from a team April 6, 2024 12:55
Copy link

@HoleFish HoleFish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I bet the original intention was to disallow up & down directions, but it overrode the wrong method and forgot to restrict the orientation setting on placing. That is the real issue that needs to be fixed.

@Dream-Master Dream-Master requested a review from a team April 22, 2024 10:39
@Dream-Master Dream-Master added ongoing freeze - don't merge Not just a bug fix and thus affected by a current freeze for a upcoming version and removed ongoing freeze - don't merge Not just a bug fix and thus affected by a current freeze for a upcoming version labels Apr 22, 2024
@Dream-Master Dream-Master requested a review from HoleFish May 5, 2024 17:55
@HoleFish HoleFish dismissed their stale review May 5, 2024 18:08

Discussed with Dream

@Dream-Master Dream-Master merged commit 88d2c27 into GTNewHorizons:master May 6, 2024
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants