Skip to:

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#5724 closed enhancement (duplicate)

bp_has_notifications to support all notifications

Reported by: sgr33n's profile SGr33n Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Toolbar & Notifications Keywords:



Is it possible to modify bp_has_notifications to return all notifications? is_new argument is a boolean, so looks like it's only possible to request read or unread only.

Would be useful also to have an "All" link.

Thanks :)

Change History (2)

#1 @r-a-y
10 years ago

  • Component changed from Core to Notifications
  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

I'd say this is a duplicate of #5558.

Give the patch in that ticket a test and post a reply on that thread.

Update: I misread your ticket, SGr33n.

My patch does not address querying all unread and read notification together in bp_has_notifications().

It's supposed to work when passing 'is_new' as the string 'both'. (See #5315.) But, you're right about the boolean restriction.

Since my patch in #5558 covers more use-cases and I don't want to refresh that patch, I'll try to add these fixes in #5558.

Last edited 10 years ago by r-a-y (previous) (diff)

#2 @SGr33n
10 years ago

Thanks r-a-y :)

Note: See TracTickets for help on using tickets.