Skip to:
Content

BuddyPress.org

Opened 5 years ago

Closed 5 years ago

#8489 closed enhancement (fixed)

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

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

Description

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 5 years ago.
8489.no-prefix.patch​ (7.0 KB ) - added by imath 5 years ago.

Download all attachments as: .zip

Change History (5)

@espellcaste
5 years ago

#1 @imath
5 years ago

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

@imath
5 years ago

#2 @imath
5 years ago

  • Keywords commit added

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

#3 @espellcaste
5 years ago

  • Resolution → fixed
  • Status assigned → 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.