Skip to:
Content

BuddyPress.org

Opened 8 years ago

Closed 8 years ago

#7031 closed enhancement (fixed)

Emails: Add "Situations" column to dashboard

Reported by: r-a-y's profile r-a-y Owned by: r-a-y's profile r-a-y
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.5.0
Component: Administration Keywords: has-patch commit
Cc:

Description

Would be nice to add a "Situations" column to the Emails admin dashboard.

Attached patch does this.

This is what it looks like:
http://i.cubeupload.com/nvILy6.png

Attachments (1)

7031.01.patch​ (2.4 KB) - added by r-a-y 8 years ago.
Refreshed and switches wp_get_object_terms() with get_the_terms()

Download all attachments as: .zip

Change History (7)

#1 @DJPaul
8 years ago

Idea is great, but -- regarding the patch -- wp_get_object_terms() is an uncached function. Can you use the get_the_terms() instead?

@r-a-y
8 years ago

Refreshed and switches wp_get_object_terms() with get_the_terms()

#2 @r-a-y
8 years ago

  • Owner set to r-a-y
  • Status changed from new to reviewing

regarding the patch -- wp_get_object_terms() is an uncached function. Can you use the get_the_terms() instead?

Thanks for the recommendation! Patch is updated to use get_the_terms() instead.

#3 @r-a-y
8 years ago

  • Status changed from reviewing to accepted

Didn't mean to change the status of the ticket. Stupid mouse!

#4 @dcavins
8 years ago

Nice improvement! I had to look up the behavior of the + operator on arrays, which is pretty interesting/useful.

I think this is ready to commit. :)

#5 @DJPaul
8 years ago

  • Keywords commit added

#6 @r-a-y
8 years ago

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

In 10724:

Emails Admin: Introduce 'Situations' column.

This will allow site admins to easily identify the email situations in the
WordPress post list table.

Fixes #7031.

Note: See TracTickets for help on using tickets.