getClientIp() will now only return valid IP addresses, rather than assuming the X_FORWARDED_FOR is the first comma seperated value.
Since getClientIp() no longer takes a parameter, removed that old test

Added a new test for HTTP_X_FORWARDED_FOR that starts with 'unknown, '
[Bridge][Form] Rename some deprecated methods
merged branch nomack84/rename_form_methods (PR #4473)

Commits
-------

cbc9944 [Bridge][Form] Rename some deprecated methods

Discussion
----------

[Bridge][Form] Rename some deprecated methods

This pull request is for renaming some deprecated Form's methods from the Bridge namespace.

---------------------------------------------------------------------------

by travisbot at 2012-05-31T12:20:59Z

This pull request [fails](http://travis-ci.org/symfony/symfony/builds/1486874) (merged cbc99442 into 78747e6c).

---------------------------------------------------------------------------

by nomack84 at 2012-05-31T12:27:29Z

The fail is not my fault.
merged branch neilferreira/master (PR #4472)

Commits
-------

85db221 Since getClientIp() no longer takes a parameter, removed that old test
7b5328f getClientIp() will now only return valid IP addresses, rather than assuming the X_FORWARDED_FOR is the first comma seperated value.

Discussion
----------

getClientIp() will now only return valid IP addresses, rather than assuming the X_FORWARDED_FOR is the first comma seperated value.

Fixes #4471

I'm not sure why an empty string was being returned in the first place, rather than null.  Any ideas?

---------------------------------------------------------------------------

by travisbot at 2012-05-31T08:59:12Z

This pull request [fails](http://travis-ci.org/symfony/symfony/builds/1485615) (merged 68c17e07 into 78747e6c).

---------------------------------------------------------------------------

by travisbot at 2012-05-31T09:02:57Z

This pull request [fails](http://travis-ci.org/symfony/symfony/builds/1485634) (merged 9c1ba1c4 into 78747e6c).

---------------------------------------------------------------------------

by neilferreira at 2012-05-31T09:04:16Z

Sorted, I'm guessing I need to squash the commits?

---------------------------------------------------------------------------

by travisbot at 2012-05-31T09:21:30Z

This pull request [fails](http://travis-ci.org/symfony/symfony/builds/1485732) (merged 7b5328f1 into 78747e6c).

---------------------------------------------------------------------------

by igorw at 2012-05-31T09:38:07Z

With what value did it fail? Can you add a test case for that `HTTP_X_FORWARDED_FOR ` value?

---------------------------------------------------------------------------

by neilferreira at 2012-05-31T10:45:11Z

Anyone have any idea why that function returns an empty string instead of null ?

---------------------------------------------------------------------------

by neilferreira at 2012-05-31T11:34:12Z

@igorw done, I've also removed an old test that should have been removed when getClientIp() started using the 'trust proxy' variable concept.

---------------------------------------------------------------------------

by travisbot at 2012-05-31T11:38:19Z

This pull request [fails](http://travis-ci.org/symfony/symfony/builds/1486534) (merged 85db2213 into 78747e6c).
Revert "merged branch nomack84/rename_form_methods (PR #4473)"

This reverts commit 136b7868f7dfbb5108a3d60aec726950d3430091, reversing
changes made to 78747e6cc5f6d65c27bd315b5627d9265af332f2.
[Form] Removed invalid tests introduced again upon merge
Fixed mistake in UPGRADE file
merged branch bschussek/fix (PR #4478)

Commits
-------

d6e8001 Fixed mistake in UPGRADE file
a1cd8ae [Form] Removed invalid tests introduced again upon merge

Discussion
----------

[Form] Fixed failing tests in master

---------------------------------------------------------------------------

by travisbot at 2012-06-01T06:24:55Z

This pull request [passes](http://travis-ci.org/symfony/symfony/builds/1495799) (merged d6e80012 into e0825289).
[Session] Removed false reliance on session.auto_start

The session must be started explicitly including in test environment.
[HttpFoundation] Added test and fixed docblocks