#25776 closed defect (bug) (fixed)
$is_nginx global not defined in vars.php
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| 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.phpis included in the global scope and the only place that references the$is_nginxglobal directly (options-permalink.php) is also included in the global scope, this does not cause any problems.got_url_rewriteuses$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.phpis only included directly viawp-settings.phpin the global scope, meaning all variables invars.phpare considered global anyways.Striving for consistency, would be nice to see it added. Additionally, the doc-block at the top of
vars.phpshould be updated to include the nginx check.