Use the request_uri() function instead of $_SERVER['REQUEST_URI '] for Non-Apache servers [#1470262]

I've noticed that webform_submission_page() in /includes/webform.submissions.inc uses $_SERVER['REQUEST_URI'].
IIRC this does not work for some non-Apache servers, and request_uri() should be used

Comment	File	Size	Author
#1	webform-requesturi-1470262-1.patch	703 bytes	bart.hanssens
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

bart.hanssens CreditAttribution: bart.hanssens commented 6 March 2012 at 12:58

Status:

Active

» Needs review

File	Size
webform-requesturi-1470262-1.patch	703 bytes

Small patch attached

Comment #2

quicksketch

he/him

CreditAttribution: quicksketch commented 6 March 2012 at 17:52

Status:

Needs review

» Reviewed & tested by the community

Excellent, thanks! I'll give this a pass in testing then commit next time I'm going through the queue.

Comment #3

quicksketch

he/him

CreditAttribution: quicksketch commented 7 March 2012 at 07:42

Title:	Non-Apache server and REQUEST_URI	» Use the request_uri() function instead of $_SERVER['REQUEST_URI '] for Non-Apache servers
Status:	Reviewed & tested by the community	» Fixed

Thanks! Committed to both branches.