Context Navigation

← Previous Ticket
Next Ticket →

#12197 closed defect (bug) (fixed)

edit_post_link() - "Edit Post" shows as link title no matter what $link is entered

Reported by:	sc0ttkclark	Owned by:
Priority:	normal	Milestone:	3.1
Component:	Template	Version:	2.9.1
Severity:	normal	Keywords:
Cc:		Focuses:

Description

Currently when this is used:

edit_post_link('Edit this Blah Blah');

Then this is output:

<a class="post-edit-link" href="link-to-wp-admin-edit" title="Edit Post">Edit this Blah Blah</a>

Of course, that's an i18n "Edit Post" but still, this should be overwritten by the $link parameter (if provided). And even then, if none is provided the Title should be the same as the Anchor text.

See below for my proposed bug fix:

/**
 * Display edit post link for post.
 *
 * @since 1.0.0
 *
 * @param string $link Optional. Anchor text.
 * @param string $before Optional. Display before edit link.
 * @param string $after Optional. Display after edit link.
 * @param int $id Optional. Post ID.
 */
function edit_post_link( $link = null, $before = '', $after = '', $id = 0 ) {
	if ( !$post = &get_post( $id ) )
		return;

	if ( !$url = get_edit_post_link( $post->ID ) )
		return;

	if ( null === $link )
		$link = __('Edit This');

	$link = '<a class="post-edit-link" href="' . $url . '" title="' . esc_attr( $link ) . '">' . $link . '</a>';
	echo $before . apply_filters( 'edit_post_link', $link, $post->ID ) . $after;
}

Attachments (1)

link-template-patch.diff (572 bytes ) - added by wojtekszkutnik 16 years ago.: patch

Download all attachments as: .zip

Change History (5)

@wojtekszkutnik
16 years ago

Attachment link-template-patch.diff added

patch

#1 @aaroncampbell
16 years ago

Cc aaron@… added

Actually, you have to assume that $link is already translated, so you shouldn't be wrapping it in __()

#2 @sc0ttkclark
16 years ago

Looks like this is fixed in 3.0, however it's not using $link - it's using the $post_type_obj->labels->edit_item which isn't too bad ;)

#3 @sc0ttkclark
16 years ago

Resolution → fixed
Status new → closed

#4 @nacin
16 years ago

Milestone Future Release → 3.1

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core