Create: Steam 'n' Rails

Create: Steam 'n' Rails

Mods
20M Downloads

Use tag for Conductor-usable blocks

dhouck opened this issue · 3 comments

commented

Describe the Suggestion

Conductors [can use] (

Railway/common/src/main/java/com/railwayteam/railways/content/conductor/ConductorEntity.java

Line 1096 in 9f316c0

return state.is(BlockTags.BUTTONS) || state.is(Blocks.LEVER) || state.getBlock() instanceof TrackSwitchBlock;
) levers, correctly-tagged buttons, and track switches. It would be nice to instead have this be data-based so people could add or remove blocks from the category. Ideally it should just be a new tag which by default contains those blocks, so blocks could also be removed from the list.

Screenshots and Videos

No response

Additional Context

This is a general issue but I thought of it because of Infinity Buttons; see also TeamDiopside/InfinityButtons#37; even if that is fixed they might not apply it to that mod’s levers, or worse they might have a technical reason to not fix it in which case a #railways:conductor_usable tag would let the two mods work together anyway.

commented

Yeah this is a pretty simple thing, we could do this change :)

commented

I will try to do this myself this weekend if nobody gets to it first, but thereʼs an 80% chance Gradle will throw an error nobody I talk to has ever heard of, and I have time to make and debug a really simple fix like this only if I can get things building; I donʼt have time to debug build systems that hate me.

(I donʼt know why I have so many java build system issues; yes most instructions are for Windows and I use Linux, but I think it goes beyond that.)

commented

I was confused when I made this issue; I was mixing up conductors using a block themselves and possessing a conductor to use a block. I wanted the latter, and it apparently already existed with the tag #railways:conductor_spy_usable. Iʼm not actually sure how to activate the former (looking at the code and comments it involves looking at the conductor?), but should it maybe use the same tag?