Skip to:
Content

BuddyPress.org

Opened 9 years ago

Closed 8 years ago

#6831 closed enhancement (fixed)

Remove remove_node 'edit' in bp-core-adminbar.php

Reported by: hnla's profile hnla Owned by: tw2113's profile tw2113
Milestone: 2.9 Priority: low
Severity: minor Version:
Component: Core Keywords: needs-patch WP4.1
Cc:

Description

In bp-core-adminbar.php we address empty edit links for post/pages being created by removing the 'node' L: 43.

if ( is_buddypress() ) {
 $wp_admin_bar->remove_node( 'edit' );
}

This carries the comment that it ought to be removed when a WP ticket/patch is committed that checks for empty edit link before rendering. This ticket and patch has been committed and closed 17 months ago:
https://core.trac.wordpress.org/ticket/29538

Time to remove?

Attachments (1)

6831-remove-edit-link.diff (586 bytes) - added by tw2113 8 years ago.

Download all attachments as: .zip

Change History (11)

#1 @DJPaul
9 years ago

  • Keywords 4.1 added

Let's wait until our minimum WP version is 4.1. Should be sometime later this year, at current rate. I'll tag this 4.1!

#2 @DJPaul
9 years ago

  • Keywords WP4.1 added; 4.1 removed
  • Milestone changed from Awaiting Review to Future Release

#3 @DJPaul
9 years ago

  • Type changed from task to enhancement

#4 @tw2113
9 years ago

We now support only 4.1+ for WordPress.

Any reason not to close this one now @DJPaul @hnla ?

#5 @hnla
9 years ago

Yes needs to be actioned then.

#6 @tw2113
8 years ago

  • Owner set to tw2113
  • Status changed from new to assigned

#7 @hnla
8 years ago

  • Keywords 2nd-opinion removed
  • Milestone changed from Future Release to 2.9

#8 @tw2113
8 years ago

I suspect it'll be fine, but the attached diff is ready for quick review @hnla

#9 @hnla
8 years ago

@tw2113 Patched but oddly this removal doesn't have desired effect, I see the edit link back in on the main WP adminbar, not sure why at this stage.

Version 0, edited 8 years ago by hnla (next)

#10 @tw2113
8 years ago

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

In 11560:

Re-adds "Edit" node after WordPress core ticket had been patched to check for empty edit links. Fixes #6831.

Note: See TracTickets for help on using tickets.