Page 3 of 3

Re: [Mod] Chat plus [2.3] [chatplus] – Now two mods, bug fix

PostPosted: Mon Mar 20, 2017 14:54
by ExeterDad
fireglow wrote:Recent minetest-dev clients can't see their own messages.
Other players can see the messages, but not the player with the dev client.

Example:
Client: 0.4.15-1de08e19
Server: 0.4.15-7b74f04a


I'm seeing this as well. Newer dev clients no longer have chat prediction visible. I thought about overcoming this on mine by sending a copy of sent chat to sender, but then older clients with chat prediction would see double posts.

Re: [Mod] Chat plus [2.3] [chatplus] – Now two mods, bug fix

PostPosted: Mon Mar 20, 2017 15:15
by rubenwardy

Re: [Mod] Chat plus [2.3] [chatplus] – Now two mods, bug fix

PostPosted: Mon Mar 20, 2017 15:18
by ExeterDad
rubenwardy wrote:https://github.com/rubenwardy/chatplus/commit/d0834e2cb31d67645c1840d4cf4ee4ed872d2a85

Nice! And only two lines added? Is it wrong that I hate how easy you make things look? :P

Re: [Mod] Chat plus [2.3] [chatplus] – Now two mods, bug fix

PostPosted: Mon Mar 20, 2017 15:49
by fireglow
Not quite, I'm afraid:
Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.
Code: Select all
ServerError: AsyncErr: ServerThread::run Lua: Runtime error from mod '' in callback on_chat_message(): /srv/minetest/world3/worldmods/chatplus/api.lua:223: attempt to call field 'chat_send_place' (a nil value)
stack traceback:
      /srv/minetest/world3/worldmods/chatplus/api.lua:223: in function 'send'
      /srv/minetest/world3/worldmods/chatplus/api.lua:232: in function </srv/minetest/world3/worldmods/chatplus/api.lua:231>
      ...rvers/0.4.15-3-7b74f04a/bin/../builtin/game/register.lua:412: in function <...rvers/0.4.15-3-7b74f04a/bin/../builtin/game/register.lua:392>

Server: 0.4.15-7b74f04a
Client: 0.4.15-c05dfac

Re: [Mod] Chat plus [2.3] [chatplus] – Now two mods, bug fix

PostPosted: Mon Mar 20, 2017 15:51
by rubenwardy

Re: [Mod] Chat plus [2.3] [chatplus] – Now two mods, bug fix

PostPosted: Mon Mar 20, 2017 15:53
by fireglow
Perfect, thank you very much!

Re: [Mod] Chat plus [2.3] [chatplus] – Now two mods, bug fix

PostPosted: Mon Mar 20, 2017 16:16
by fireglow
I spoke too soon:
Now the release clients get their chat echoed two times.
Is this an EITHER case, where you could either support 0.4.15-release OR 0.4.15-dev, but not both?

Re: [Mod] Chat plus [2.3] [chatplus] – Now two mods, bug fix

PostPosted: Thu Mar 23, 2017 15:28
by rubenwardy
fireglow wrote:I spoke too soon:
Now the release clients get their chat echoed two times.
Is this an EITHER case, where you could either support 0.4.15-release OR 0.4.15-dev, but not both?


Afaik there's no way of detecting whether a client will print the chat message before sending it, so yes: I can only support one or the other :(