Skip to:
Content

BuddyPress.org

Opened 8 years ago

Closed 7 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 7 years ago.

Download all attachments as: .zip

Change History (11)

#1 @DJPaul
8 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
8 years ago

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

#3 @DJPaul
8 years ago

  • Type changed from task to enhancement

#4 @tw2113
8 years ago

We now support only 4.1+ for WordPress.

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

#5 @hnla
8 years ago

Yes needs to be actioned then.

#6 @tw2113
7 years ago

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

#7 @hnla
7 years ago

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

#8 @tw2113
7 years ago

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

#9 @hnla
7 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.

Ignore above, the issue was an empty link not no link, so lets commit!

Last edited 7 years ago by hnla (previous) (diff)

#10 @tw2113
7 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.