Skip to:

Opened 3 years ago

Closed 3 years ago

#8489 closed enhancement (fixed)

Mark a thread as (un)read to a specific user

Reported by: espellcaste's profile espellcaste Owned by: espellcaste's profile espellcaste
Milestone: 9.0.0 Priority: normal
Severity: normal Version: 8.0.0
Component: Messages Keywords: has-patch has-unit-tests commit


Currently, messages_mark_thread_read and messages_mark_thread_unread grab the id of the user from the logged in user or from the displayed user id.

From an API perspective, that's not ideal since you can't set a specific user id directly.

So I'm suggesting an improvement so that we can use those functions in the BuddyPress APIs (REST API and CLI) by setting a user ID directly without relying in the forementioned user IDs.

Attachments (2)

8489.diff (7.0 KB) - added by espellcaste 3 years ago. (7.0 KB) - added by imath 3 years ago.

Download all attachments as: .zip

Change History (5)

3 years ago

#1 @imath
3 years ago

Thanks for your work on this, I will test it asap 👌

#2 @imath
3 years ago

  • Keywords commit added

Just tested, It's good to go. I've just --no-prefix - ed the patch.

#3 @espellcaste
3 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 12984:

Adding a user ID parameter to the messages_mark_thread_read and messages_mark_thread_unread functions.

Allowing those functions to be used in other BuddyPress API.

Props imath
Fixes #8489 (trunk)

Note: See TracTickets for help on using tickets.