[buddypress-trac] [BuddyPress] #5176: Add get_content() to bp message threads
buddypress-trac
noreply at wordpress.org
Wed Sep 18 00:34:18 UTC 2013
#5176: Add get_content() to bp message threads
-------------------------------------------------------+------------------
Reporter: trishasalas | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 1.9
Component: Template Pack | Version:
Severity: normal | Resolution:
Keywords: reporter-feedback has-patch needs-testing |
-------------------------------------------------------+------------------
Changes (by boonebgorges):
* keywords: needs-patch reporter-feedback has-patch needs-testing =>
reporter-feedback has-patch needs-testing
* milestone: Awaiting Review => 1.9
Comment:
5176.02.patch is a cleaned up version of trishasalas's original patch
(which has what appears to be an accidental deletion in it).
I think this is close to what you want. However, it's worth noting the
specific way that the `last_message_*` stuff is calculated, both in the
`_excerpt()` function and this new function. Consider the following
exchange:
Boone to Trisha: Hi
Trisha to Boone: Yo
Boone to Trisha: Cool
a) When viewing your Inbox, the `last_message` is the most recent message
in that thread of which you are *not* the author. So
`bp_get_message_thread_content()` returns 'Yo' when I am viewing my own
inbox.
b) When viewing your Sentbox, `last_message` is the most recent message in
that thread of which you *are* the member.
`bp_get_message_thread_content()` returns 'Cool' when I'm viewing my
Sentbox.
I can see how this makes sense in a way, but in any case, I want to make
sure that everyone understands it and agrees that this is how the new
message templates are supposed to work.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5176#comment:12>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list