Skip to content
This repository has been archived by the owner on Nov 7, 2023. It is now read-only.

Crash on Orlin's gate rendering #316

Open
slava110 opened this issue May 28, 2021 · 9 comments
Open

Crash on Orlin's gate rendering #316

slava110 opened this issue May 28, 2021 · 9 comments
Labels
Alpha This occurs in an Alpha version of the mod bug Something isn't working crash Severe bug crashing the game fixed-in-dev Issue is fixed in development branch (or pull request) and waiting for merge

Comments

@slava110
Copy link
Contributor

Crash log: https://paste.ee/p/1Iof4
Cause: mrjake.aunis.block.stargate.StargateAbstractMemberBlock#L111

@slava110 slava110 added bug Something isn't working crash Severe bug crashing the game Alpha This occurs in an Alpha version of the mod labels May 28, 2021
@slava110
Copy link
Contributor Author

Should be moved to StargateClassicMemberBlock probably as there's no camouflage for abstract stargate at all

@MrJake222
Copy link
Owner

Also probably when rendering block it should return something different than UNDEFINED

@slava110
Copy link
Contributor Author

Available values

@MrJake222
Copy link
Owner

Probably we should somehow get the default FaceShape of the stock block used, which is probably going to be always solid

@slava110
Copy link
Contributor Author

slava110 commented May 28, 2021

Why? If you have SOLID faceshape people will be able to place torches, redstone on this side of your block. It will look kinda weird on stargate.
We can return SOLID for bottom of Orlin's gate tho (because it looks like full block IIRC)

@MrJake222
Copy link
Owner

when rendering block

When stargate base/member is not merged it is rendered as ordinary block. Then it should probably be able to support snow/torches/etc.

@slava110
Copy link
Contributor Author

slava110 commented May 28, 2021

When stargate base/member is not merged it is rendered as ordinary block. Then it should probably be able to support snow/torches/etc.

Then you will need to update all stargate blocks when stargate will be merged. Otherwise torches/snow will stay on stargate after merging

@MrJake222
Copy link
Owner

Hmm. There is already a field in members whether a gate is merged or not, so what's the problem?

@slava110
Copy link
Contributor Author

No problem, just saying that you'll need to update block too. Via world.scheduleBlockUpdate or something like that. Otherwise snow/torches will stay on the block because they will not check if BlockFaceShape has been changed

@slava110 slava110 added the fixed-in-dev Issue is fixed in development branch (or pull request) and waiting for merge label Jun 8, 2021
slava110 added a commit that referenced this issue Aug 19, 2021
- Moved camouflage logic in `getBlockFaceShape` method from `StargateAbstractMemberBlock` to `StargateClassicMemberBlock`
- `BlockFaceShape.SOLID` will be returned if member block isn't merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Alpha This occurs in an Alpha version of the mod bug Something isn't working crash Severe bug crashing the game fixed-in-dev Issue is fixed in development branch (or pull request) and waiting for merge
Projects
None yet
Development

No branches or pull requests

2 participants