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

Redstone Alloy Wires checking if wrong side of blocks can connect redstone #1693

Closed
dannydjdk opened this issue Nov 6, 2021 · 1 comment
Closed
Labels

Comments

@dannydjdk
Copy link

Describe the bug
Redstone Alloy Wires will only connect to a block if the opposite side of that block can connect redstone.

To Reproduce
Steps to reproduce the behavior:

  1. Place a redstone alloy wire next to a block from another mod that can connect redstone only on specific sides of the block such as Tiny Redstone's redstone panels or Redstone Pen's redstone dust.
  2. See that the alloy wire will not connect to a side unless the opposite side of that block can connect to redstone.

2021-11-05_18 54 06
2021-11-05_18 54 24
The redstone panel from Tiny Redstone reports that it can connect to redstone on any side that contains a component. Project Red's wires are consistently connecting to the sides opposite those with components.

2021-11-05_19 26 06
2021-11-05_19 26 21
Redstone Alloy is not connecting to the Redstone dust placed by Redstone Pen in the pictures above unless the opposite side is activated.

Expected behavior
2021-11-05_18 54 49
The picture above shows that vanilla redstone properly connects to a redstone panel from Tiny Redstone only on the sides with components.

I haven't check the code, but it is almost certainly the case that the wrong Direction is being passed to the canConnectRedstone method. The direction parameter expects the direction from the querying block to the block being queried, not the other way around.

Versions
Include versions of the dependencies. And since you're looking at it, make sure ProjectRed is up to date. Don't submit bugs for old versions, as they could have been fixed.

  • CCL Version: 4.0.3.434
  • FMP Version: 1.16.5
  • Scorge: 3.1.3
  • ProjectRed Version: 4.10.0

Screenshots/Videos
See above.

@MrTJP
Copy link
Owner

MrTJP commented Jul 6, 2023

This was fixed at some point

@MrTJP MrTJP closed this as completed Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants