Bug 294702 - Backlog messages do not go through message proccessor
Summary: Backlog messages do not go through message proccessor
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: text-ui (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: Future
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-23 17:00 UTC by David Edmundson
Modified: 2012-08-16 02:11 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description David Edmundson 2012-02-23 17:00:55 UTC 
and as such have no emotions or links.
Comment 1 Daniele E. Domenichelli 2012-02-23 17:29:35 UTC 
I suggest that we do it with 2 sets of filters, because we must show links and emoticons, but I wouldn't show, for example, a youtube video in the backlog
Comment 2 David Edmundson 2012-02-25 04:48:05 UTC 
@drdanz, I agree with that completely, but right now we only do links and emoticons so I'm happy for current code to go through the same set as it's simpler and would be identical. Then we split it out later when more plugins come (Hopefully a GSOC project so new development on this is currently paused.)

Also when we implement this, I don't want to see multiple instances of the filters themselves (wasteful) or of MessageProcessor (as it's a singleton), we simply pass the message through varying different chains of MessageProcessor::m_filters.