profanity icon indicating copy to clipboard operation
profanity copied to clipboard

Bug Fix #1938

Open OdenDillenkoffer opened this issue 10 months ago • 1 comments

For the bug reported in #1938 there seems to be an issue of not grabbing the right time stamp. This is similar to the bug in #1190 of not grabbing the proper delay for MAM messages. To fix this, instead of discarding the original timestamp and trying to find the oldest delay, which may have been lost to a server quirk, we preserve the timestamp until an older delay is found. Then, and only then, will the timestamp update. Of course if it still can't find a delay and there is no timestamp, the fall back of grabbing the current time is still available. This should fix the bug experienced in #1938 and #1190 , given they look like they stem from a similar issue.

  • [ ] I ran valgrind when using my new feature

How to test the functionality

  • Step 1 - Profanity is not connected.
  • Step 2 - Receive messages in a MUC *Step 3 - Connect Profanity Now the timestamps should be correct for the MUC.

Previous to this there were issues of inaccurate time stamps, but now that should be mitigated.

OdenDillenkoffer avatar Feb 09 '25 20:02 OdenDillenkoffer

Ping

jubalh avatar Aug 22 '25 07:08 jubalh