Permalink
Browse files

Changed Server variable to REQUEST_URI from REDIRECT_URL

  • Loading branch information...
Stardog committed Mar 27, 2017
1 parent 85ff463 commit e9150c10cf4a851b00fc57cf2741da8f7a6fb502
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/Server.php
View
@@ -185,7 +185,7 @@ public static function getCurrentUrl($relative = true, $use_redirect = true)
$self = & $_SERVER['PHP_SELF'];
- if ($use_redirect && isset($_SERVER['REDIRECT_URL'])) {
+ if ($use_redirect && isset($_SERVER['REQUEST_URI'])) {
// some users reported problems using redirect_url so parsing uri instead
if ($_SERVER['REQUEST_URI'] != '/') {
$root_url = substr($self, 0, strrpos($self, '/'));

4 comments on commit e9150c1

@trf000

This comment has been minimized.

Show comment
Hide comment
@trf000

trf000 Mar 28, 2017

Collaborator

This causes an issue with link showing as active in menu.

Collaborator

trf000 replied Mar 28, 2017

This causes an issue with link showing as active in menu.

@Stardog

This comment has been minimized.

Show comment
Hide comment
@Stardog

Stardog Mar 30, 2017

Member

We are looking into it.

Member

Stardog replied Mar 30, 2017

We are looking into it.

@eberhardtm

This comment has been minimized.

Show comment
Hide comment
@eberhardtm

eberhardtm Mar 30, 2017

Contributor

I had this changed so we could get phpwebsite branches working with nginx. I tested both on apache and nginx but can't reproduce what your seeing.

Contributor

eberhardtm replied Mar 30, 2017

I had this changed so we could get phpwebsite branches working with nginx. I tested both on apache and nginx but can't reproduce what your seeing.

@trf000

This comment has been minimized.

Show comment
Hide comment
@trf000

trf000 Mar 30, 2017

Collaborator

could be our setup, so i wouldn't worry about it, but I'll send Stardog our server setup maybe that will help track it down.

Collaborator

trf000 replied Mar 30, 2017

could be our setup, so i wouldn't worry about it, but I'll send Stardog our server setup maybe that will help track it down.

Please sign in to comment.