search

minetest-mods / MoreMesecons

More mesecons items
https://forum.minetest.net/viewtopic.php?f=11&t=13150
Mozilla Public License 2.0
19 stars 12 forks source link

moremesecons_luacontroller_tool has issues with more than 1 luacontroller. #8

Closed oiaohm closed 6 years ago

oiaohm commented 6 years ago

I have found when I was attempting to program 2 luacontrollers at the same time using the templates I was having the nightmare where the first one I opened was being overwritten and the one was pointing at was having nothing happen.

The patch includes is a really simple delete a piece of code and it now seams to work as expected as in programming what ever lua controller I happen to be pointing at.

moremesecons_luaprogrammerfix.diff.txt

paly2 commented 6 years ago

Thank you for this bug report. I applied you fix (with a small change to keep the old behaviour of remembering the current open template between luacontroller tool uses) in https://github.com/minetest-mods/MoreMesecons/commit/8a56370ce8efd68d4ca54fde0bd6b5c37d35f601.

oiaohm commented 6 years ago

Thanks I knew my patch had to breaking something. Its not like someone codes a remembering system for no reason. I was just not seeing what. I had not noticed that template remembering was gone.