[wp-trac] [WordPress Trac] #12145: wp-activate.php creates many notices on load due to theme hackery
WordPress Trac
wp-trac at lists.automattic.com
Sat Feb 6 07:10:14 UTC 2010
#12145: wp-activate.php creates many notices on load due to theme hackery
------------------------------+---------------------------------------------
Reporter: dd32 | Owner: westi
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.0
Component: Warnings/Notices | Version:
Severity: normal | Keywords: multisite
------------------------------+---------------------------------------------
an example to start:
{{{
Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes
\comment-template.php on line 777
Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes
\comment-template.php on line 794
Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes
\general-template.php on line 1588
Notice: Trying to get property of non-object in G:\www\ho.st\wp-
includes\query.php on line 2600
Notice: Trying to get property of non-object in G:\www\ho.st\wp-
includes\query.php on line 2600
Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes
\post-template.php on line 431
Notice: Trying to get property of non-object in G:\www\ho.st\wp-
includes\query.php on line 2600
Notice: Trying to get property of non-object in G:\www\ho.st\wp-includes
\post-template.php on line 1172
class="page page-id- page-parent logged-in">
}}}
the problem is, that wp-activate.php attempts to simulate a normal
WordPress page, the problem arrises that when the header is included
and/or custom code in the theme runs on the header, that the no-existance
of the current post id (NULL) causes most of the WordPress functions to
complain, as they're expecting a correct post object to be returned.
2 options
1. Update all API function to move to {{{ if ( ! $_post = get_post(..) )
return false; - This should probably be done anyway
2. Improve the hackery to allow displaying custom pages such as wp-
activate.php more transparently.
Furthur to #2, this might involve creating a page in the database which
the API functions can access directly, the activate site page would then
become a real page in the eyes of WordPress.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/12145>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list