[wp-trac] Re: [WordPress Trac] #3996: MAGPIE_USER_AGENT lack of wp
version
WordPress Trac
wp-trac at lists.automattic.com
Mon Jun 18 17:23:34 GMT 2007
#3996: MAGPIE_USER_AGENT lack of wp version
-------------------------------+--------------------------------------------
Reporter: hakre | Owner: anonymous
Type: enhancement | Status: closed
Priority: normal | Milestone: 2.4 (future)
Component: Administration | Version: 2.1.2
Severity: normal | Resolution: invalid
Keywords: rss magpie snoopy |
-------------------------------+--------------------------------------------
Changes (by Otto42):
* status: reopened => closed
* resolution: => invalid
Comment:
I'm also unable to reproduce your problem.
hakre: I think the issue is that you are wrong about the ordering of the
files loading.
wp-blog-header.php loads wp-config.php which then loads wp-settings.php.
After all this is done, and you make it back to wp-blog-header.php, then
it goes into the template-loader.php.
wp-settings.php (and thus version.php) is loaded well before anything in
the theme is. What's more, version.php is loaded before any plugins,
before the my-hacks file, before the theme's functions.php file...
basically before any normally user-modifiable code.
In order for what you're saying to be true, you'd have to include rss.php
well before any of that stuff, but you're saying that you're loading it
via "a widget within my template" which doesn't work. wp-version is
already set by that point.
Now, you are correct that the function init() in rss.php is busted because
it does not declare $wp_version as global, however the define at the top
of the rss.php file prevents that from making any difference, since it
prevents that code from executing anyway.
Anyway, unless you can show a specific case that we can duplicate where
$wp_version is not set by the time you require_once("rss.php"), please
don't reopen this ticket.
--
Ticket URL: <http://trac.wordpress.org/ticket/3996#comment:5>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list