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

Fix an issue with the screwdriver being used on glass. #712

Closed
wants to merge 1 commit into from
Closed

Fix an issue with the screwdriver being used on glass. #712

wants to merge 1 commit into from

Conversation

Amaz1
Copy link
Contributor

@Amaz1 Amaz1 commented Oct 21, 2015

This "fixes" https://forum.minetest.net/viewtopic.php?f=6&t=13438

When glass has a param2 set, then it has a random amount of a random colour inside it, because no "special_tiles" are defined. This fixes this by setting special_tiles to a blank texture.

I don't know if this is a proper fix, or whether it would be better if things were done in the engine, either disabling the feature, or setting the texture to a blank one when no special tiles are given.

@PilzAdam
Copy link
Contributor

A proper fix would be to change the screwdriver to only rotate nodes with paramtype2 = "facedir".

Maybe adding this additionally would be good, though, for other unintentional changes of param2.

@est31
Copy link
Contributor

est31 commented Oct 21, 2015

The only thing what I think the engine has to do is to define a proper paramtype2 for the volume indicator. Once that is there the engine can perhaps also automatically check whether "special_tiles" is defined, and if not, print a warning or stacktrace or so.

@RealBadAngel
Copy link
Contributor

@est31, thats the only way

@Ekdohibs
Copy link
Member

I think that there should be something in the engine too for checking param2, and that the screwdriver mod should be fixed, not the glass definition.

@paramat
Copy link
Contributor

paramat commented Nov 14, 2015

The only thing what I think the engine has to do is to define a proper paramtype2 for the volume indicator.

Maybe, but glasslike nodes should be facedir if wanted, being rotatable is more important than having the liquid tank feature.

@sofar
Copy link
Contributor

sofar commented Dec 24, 2015

Posted a (tested) patch that nop's the screwdriver actions on any node that isn't a facedir paramtype2.

Mods can still make glass "facedir" by changing the nodedef for glasslike nodes, to get screwdriver functionality back.

I'm not sure this is the proper solution, but thought it may help to signal users that the fluid level in glasslike nodes is a "special" thing and not something the screwdriver is supposed to manipulate.

@Amaz1 Amaz1 closed this Dec 24, 2015
paramat pushed a commit that referenced this pull request Dec 25, 2015
This prevents nodes that are not of the "facedir" paramtype2,
as these are generally not expected to be able to rotate. Mods
and other methods may still manipulate param2 of these nodes
as expected.

Addresses #712
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants