Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 13 years ago

Closed 13 years ago

#23744 closed defect (bug) (fixed)

PHP notice on a custom post type archive with default permalinks enabled

Reported by:	ericlewis	Owned by:	SergeyBiryukov
Priority:	normal	Milestone:	3.6
Component:	Warnings/Notices	Version:	3.1
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

Notice: Undefined index: feeds in /Users/ericlewis/Sites/wordpress-trunk/wp-includes/link-template.php on line 871

Looks like something to do with the way the post type object is set up by default with default permalinks enabled.

Attachments (2)

23744.patch (589 bytes ) - added by ericlewis 13 years ago.: Change boolean check to empty()
23744.2.patch (721 bytes ) - added by SergeyBiryukov 13 years ago.

Download all attachments as: .zip

Change History (5)

@ericlewis
13 years ago

Attachment 23744.patch added

Change boolean check to empty()

@SergeyBiryukov
13 years ago

Attachment 23744.2.patch added

#1 @SergeyBiryukov
13 years ago

Component Feeds → Warnings/Notices
Milestone Awaiting Review → 3.6
Version trunk → 3.1

#2 @SergeyBiryukov
13 years ago

23744.2.patch reorders the conditions in the same way as in get_post_type_archive_link():
http://core.trac.wordpress.org/browser/tags/3.5.1/wp-includes/link-template.php#L840

#3 @SergeyBiryukov
13 years ago

Owner set to SergeyBiryukov
Resolution → fixed
Status new → closed

Avoid an undefined index notice in get_post_type_archive_feed_link(). props ericlewis. fixes #23744.

Note: See TracTickets for help on using tickets.

Download in other formats: