Skip to:
Content

bbPress.org

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#2224 closed enhancement (fixed)

Add The User ID/Name to The Author Metabox

Reported by: alex-ye's profile alex-ye Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.4 Priority: normal
Severity: normal Version: 2.2.3
Component: Back-end Keywords: has-patch needs-testing commit
Cc: nashwan.doaqan@…, wordpress@…

Description

Hi , The current author meta box only shows the author IP ,

I think we should also add author ID or user-login .

Attachments (3)

metaboxes.php.patch​ (1.9 KB) - added by MZAWeb 10 years ago.
2224.patch​ (5.8 KB) - added by alex-ye 10 years ago.
First Draft
2224.2.patch​ (5.7 KB) - added by alex-ye 10 years ago.
User printf() insted of "echo sprintf()"

Download all attachments as: .zip

Change History (13)

#1 @alex-ye
10 years ago

  • Type changed from defect to enhancement

#2 @alex-ye
10 years ago

  • Cc nashwan.doaqan@… added

#3 @johnjamesjacoby
10 years ago

  • Milestone changed from Awaiting Review to 2.4

#4 @MZAWeb
10 years ago

  • Cc wordpress@… added
  • Keywords has-patch added; needs-patch removed

Added patch

@MZAWeb
10 years ago

#5 @johnjamesjacoby
10 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 4902:

Add authordata to topic/reply metabox. Props mzaweb. Fixes #2224.

#6 @alex-ye
10 years ago

Happy that this solved, but I think we should look more deeper in this, My point was to add the ability to change the topic/reply author.

In also adding support for WordPress author meta-box slowdown the page, especially when the forums have many users, and WordPress list all users in a dropdown which make it hard to select one user !

Can we do this by more smarter way... like the topic ID input in the reply edit screen ?!

Last edited 10 years ago by alex-ye (previous) (diff)

@alex-ye
10 years ago

First Draft

#7 @alex-ye
10 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#8 @johnjamesjacoby
10 years ago

  • Keywords needs-testing commit added

Patch looks okay for 2.4.

@alex-ye
10 years ago

User printf() insted of "echo sprintf()"

#9 @johnjamesjacoby
10 years ago

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

In 5015:

Add user Suggest support for post author overrides for topics and replies. Props alex-ye. Fixes #2224.

#10 @johnjamesjacoby
10 years ago

In 5016:

Clean-up admin ajax JS for Suggest methods. Fixes #2224.

Note: See TracTickets for help on using tickets.