#20519 closed defect (bug) (fixed)
PHP Notice in get_single_template()
Reported by: | markjaquith | Owned by: | ryan |
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | normal | Version: | 3.0 |
Component: | Warnings/Notices | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
PHP Notice: Trying to get property of non-object in wp-includes/template.php on line 283
These get_*_template() functions make a lot of assumptions about the current state of the environment.
Attachments (2)
Change History (9)
#3
@
12 years ago
- Keywords has-patch added
- Milestone changed from Awaiting Review to 3.5
This patch has basic validation for the passing of vars to template file names
#4
@
12 years ago
I think in most cases it should be enough to validate we have an object. So rather than:
if ( isset( $author->user_nicename ) ) $templates[] = "author-{$author->user_nicename}.php"; if ( isset( $author->ID ) ) $templates[] = "author-{$author->ID}.php";
We can do:
if ( $author ) { $templates[] = "author-{$author->user_nicename}.php"; $templates[] = "author-{$author->ID}.php"; }
#6
@
12 years ago
- Owner set to ryan
- Resolution set to fixed
- Status changed from new to closed
In [22293]:
#7
@
11 years ago
I believe this still happens, and I opened a new ticket here on it: http://core.trac.wordpress.org/ticket/25398
Note: See
TracTickets for help on using
tickets.
Related: #17662, #18614