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
Doors: Fix missing node parameter passing. #1189
Conversation
This fixes the issue for me. |
👍 |
end, | ||
state = function(self) | ||
return node_name:sub(-5) == "_open" | ||
return minetest.get_node(self.pos).name:sub(-5) == "_open" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only bit i don't understand
@sofar is this ok?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've essentially reverted my previous removal here to force a new node lookup on self.pos
.
This way, there is less chance for mods requesting an old state (e.g. after you closed or opened the door).
Removing this before was bad judgment on my part.
Anyway this is an urgent fix so will merge later. |
👍, untested but it should fix the bug from what I see. |
This fix works for me. |
Ping |
see #1188.