Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#28766 closed enhancement (invalid)

Permalinks Options/Notices on nginx + HHVM incorrect

Reported by: crxgames's profile crxgames Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.9
Component: Permalinks Keywords: has-patch needs-testing
Focuses: Cc:

Description

The permalink settings page in the admin panel is incorrect when running Wordpress under HHVM + nginx. index.php is always included in the permalink samples and the notice messages are those from Apache.

This patch adds a new global, $is_hhvm, in the same fashion nginx is tested for, and updates the permalinks settings page accordingly.

Attachments (1)

hhvm_permalink_fix.diff (3.2 KB) - added by crxgames 10 years ago.

Download all attachments as: .zip

Change History (5)

#1 @SergeyBiryukov
10 years ago

  • Component changed from General to Permalinks

#2 @johnbillion
10 years ago

  • Focuses administration removed
  • Keywords has-patch needs-testing added
  • Milestone changed from Awaiting Review to Future Release
  • Type changed from defect (bug) to enhancement

Thanks for the report crxgames.

Interesting that HHVM doesn't pass the web server name through to $_SERVER['SERVER_SOFTWARE']. I would have expected it to pass through 'nginx' if I'm running Nginx.

What happens if you're running HHVM behind Apache instead of Nginx? The return value of got_url_rewrite() will return true even if got_mod_rewrite() returns false. (Granted, if you're running HHVM you're unlikely to be running Apache without mod_rewrite, but the point still stands.)

#3 @crxgames
10 years ago

So, should we report this as an HHVM issue instead of include a work around for it perhaps?

#4 @johnbillion
10 years ago

  • Milestone Future Release deleted
  • Resolution set to invalid
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.