weechat / scripts

Official scripts for WeeChat.
https://weechat.org/scripts/
231 stars 220 forks source link

colorize_lines.pl: Multi-line messages do not display #534

Closed superjamie closed 1 year ago

superjamie commented 1 year ago

Script / WeeChat

This bug has been reported to the script author? No

Response or reason: I thought this was the correct place to report script bugs? I will email the author too.

Bug summary

Multi-line messages do not displayed properly. I can reproduce this with wee-slack by sending myself a multi-line message. The notification plugin displays the message properly. Removing colorize_lines.pl and restarting WeeChat displays the message properly.

This used to work fine on WeeChat 3.x, it's only broken with the change to WeeChat 4.0.

Steps to reproduce

1. Install WeeChat 4.0.0

  1. Install wee-slack - https://github.com/wee-slack/wee-slack
  2. Connect to a Slack team, you can connect to your own team for $0
  3. Send a multi-line message to yourself with line1<Alt+enter>line2

Current behavior

Only the first line of the message line1 is displayed

Expected behavior

Both lines of the message line1 then line2 are displayed

Suggested solutions

Additional information

flashcode commented 1 year ago

For your information, this is not specific to wee-slack, I can reproduce on an IRC channel with multiline enabled, steps to reproduce:

  1. Install script colorize_lines.pl
  2. Add ergo server: /server add ergo testnet.ergo.chat
  3. /connect ergo (capability draft/multiline should be automatically enabled)
  4. /join #testchan
  5. Send a message with 2 lines

Actual result: only first line is displayed.