Rocket.chat: Emote detection fails around linebreaks

Created on 1 Jan 2019  路  3Comments  路  Source: RocketChat/Rocket.Chat

Description:

Emote detection fails around linebreaks.

Steps to reproduce:

  1. Open a chat, for example a direct message to yourself.
  2. Send the following single, multi-line message, which contains no trailing or leading whitespace:

    :D foo :D bar :D baz :D quux :D
    :D this :D is :D a simple :D test :D
    without :D leading :D or :D trailing :D emote
    :D hello :D world :D

Expected behavior:

The message is sent and all occurrences of :D get replaced by 馃槂.

Actual behavior:

The message is sent and MOST occurrences of :D get replaced by 馃槂, EXCEPT those around the linebreaks.

Server Setup Information:

  • Version of Rocket.Chat Server: 0.70.0
  • Operating System: dunno, shouldn't be relevant
  • Deployment Method: dunno
  • Number of Running Instances: 1, I guess, shouldn't be relevant
  • DB Replicaset Oplog: dunno, shouldn't be relevant
  • NodeJS Version: dunno
  • MongoDB Version: dunno

Additional context

I found this issue while using a private instance. I'm just the end user, and have been told to direct any bug reports upstream.

Relevant logs:

JS console log: Loading: false (nothing else; seems irrelevant)

Server log: unavailable; this is a private instance. Shouldn't be relevant.

message uux bug

All 3 comments

@tassoevan Could I work on this issue?

@savish28 Of course!

@tassoevan Please review PR #13447. It solves this issue.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

mddvul22 picture mddvul22  路  3Comments

amayer5125 picture amayer5125  路  3Comments

Kiran-Rao picture Kiran-Rao  路  3Comments

Buzzele picture Buzzele  路  3Comments

antn89 picture antn89  路  3Comments