#20519 closed defect (bug) (fixed)
PHP Notice in get_single_template()
| Reported by: | markjaquith | Owned by: | ryan |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | Warnings/Notices | Version: | 3.0 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: |
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
@
14 years ago
- Keywords has-patch added
- Milestone Awaiting Review → 3.5
This patch has basic validation for the passing of vars to template file names
#4
@
14 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";
}
#7
@
13 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Related: #17662, #18614