Make WordPress Core

Opened 2 years ago

Closed 23 months ago

Last modified 21 months ago

#26249 closed defect (bug) (fixed)

Include Nginx reference in PHPDoc head of /wp-includes/vars.php

Reported by: netweb Owned by:
Milestone: 3.9 Priority: normal
Severity: minor Version: 3.7
Component: Permalinks Keywords: has-patch commit
Focuses: docs Cc:


Add a reference to Nginx as one of three known servers that are detected by WordPress.


Attachments (3)

vars.php.diff (581 bytes) - added by netweb 2 years ago.
26249.diff (775 bytes) - added by DrewAPicture 2 years ago.
26249.2.diff (715 bytes) - added by markjaquith 23 months ago.

Download all attachments as: .zip

Change History (10)

2 years ago

#1 @SergeyBiryukov
2 years ago

Three web servers with known permalink capability.

It's probably worth noting that we don't write or suggest nginx configuration for permalinks (see [25456] for #25098).

#2 @SergeyBiryukov
2 years ago

  • Version changed from trunk to 3.7

2 years ago

#3 @DrewAPicture
2 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.9

26249.diff attempts to incorporate @SergeyBiryukov's feedback from comment:2. Feedback is welcome, otherwise it looks good to me.

#4 follow-up: @markjaquith
23 months ago

Simplified DrewAPicture's addition, referenced Codex which has these rules.

#5 in reply to: ↑ 4 @DrewAPicture
23 months ago

Replying to markjaquith:

Simplified DrewAPicture's addition, referenced Codex which has these rules.

+1 for 26249.2.diff

#6 @DrewAPicture
23 months ago

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [26931]

#7 @DrewAPicture
21 months ago

  • Component changed from Inline Docs to Permalinks
  • Focuses docs added
Note: See TracTickets for help on using tickets.