[Request] Adding ->getReferer() method #2121
Conversation
There was a problem hiding this comment.
Referrer is written with two Rs. The HTTP header, in its valid form, has a typo, but we shouldn't reproduce that everywhere imo :)
There was a problem hiding this comment.
Hm, I agree, the benefits of this class is mostly to forget about the "$_SERVER's" entries so it would be better with the two Rs
There was a problem hiding this comment.
The benefit of this class isn't really to forget how to write PHP I'd say :) The main thing is that this provides encapsulation, allows you to run many request/responses within one process (for testing, subrequests, app server in php etc).
|
I'm -1 on adding this method. You can already access it via the headers and as this information should not be relied upon, I don't think we need to make it easier to access it. |
|
@fabpot, it's not reliable but can be useful for statistics, and why wouldn't you provide a method for this since you provide one for almost each properties of the request ? |
|
@Gregwar there is no methods for all the headers. these one are retrieved from the HeaderBag: |
|
Ok, this was just an idea |
4 participants
Adding ->getReferer() for obtaining the server's HTTP_REFERER entry