#25776 closed defect (bug) (fixed)
$is_nginx global not defined in vars.php
Reported by: | johnjamesjacoby | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.8 | Priority: | normal |
Severity: | minor | Version: | 3.7 |
Component: | General | Keywords: | has-patch |
Focuses: | Cc: |
Description (last modified by )
Attachments (1)
Change History (6)
Note: See
TracTickets for help on using
tickets.
Because
vars.php
is included in the global scope and the only place that references the$is_nginx
global directly (options-permalink.php
) is also included in the global scope, this does not cause any problems.got_url_rewrite
uses$GLOBALS['is_nginx']
rather than define the global, which is how it gets around it also.Arguably, we could delete the definitions completely at the top of the file, since
vars.php
is only included directly viawp-settings.php
in the global scope, meaning all variables invars.php
are considered global anyways.Striving for consistency, would be nice to see it added. Additionally, the doc-block at the top of
vars.php
should be updated to include the nginx check.