fix: can_be_replied_to() returns false for LiveLocation events.

[bmarty]

Let can_be_replied_to() returns false for LiveLocation events.

I do not really understand why self.content.is_message() return false for LiveLocation events, because when I read the code it should return true, but it's not the case.

If reviewer agrees with the change, I will spend time to add unit test and changelog.

Closes #6562

• I've documented the public API Changes in the appropriate CHANGELOG.md files.
• This PR was made with the help of AI.

Signed-off-by:

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.95%. Comparing base (f503097) to head (4074ee8).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6563      +/-   ##
==========================================
+ Coverage   89.92%   89.95%   +0.03%     
==========================================
  Files         381      381              
  Lines      107620   107718      +98     
  Branches   107620   107718      +98     
==========================================
+ Hits        96778    96900     +122     
+ Misses       7170     7151      -19     
+ Partials     3672     3667       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[codspeed-hq]

Merging this PR will not alter performance

✅ 50 untouched benchmarks

---

_{Comparing bma/fixCanBeRepliedTo (4074ee8) with main (f503097)}

[andybalaam]

If self.content.is_message() returns false, we don't need this change, right?

[bmarty]

If self.content.is_message() returns false, we don't need this change, right?

No, because self.latest_json().is_some() is also returning true for LLS events.

[andybalaam]

is_message checks the type is MsgLike but also checks that the MsgLikeKind is Message, not LiveLocation, so that is why it returns false for live locations.

    /// Check whether this item's content is a
    /// [`Message`][MsgLikeKind::Message].
    pub fn is_message(&self) -> bool {
        matches!(self, Self::MsgLike(MsgLikeContent { kind: MsgLikeKind::Message(_), .. }))
    }

[andybalaam]

I think it is sensible to say that live location messages can't be replied to, and this looks like a reasonable way to do it.

Along with tests, I would like the comment to explain itself a little more. Maybe something like "Live location messages are not displayed on the timeline individually, so can't be replied to."

[bmarty]

Thanks. I asked AI to add tests.

[andybalaam]

I fixed the terrible test names your AI produced.