Skip to content

Commit

Permalink
minor #20994 Add missing upgrade path for Request::isMethodSafe() dep…
Browse files Browse the repository at this point in the history
…recation (chalasr)

This PR was merged into the 3.2 branch.

Discussion
----------

Add missing upgrade path for Request::isMethodSafe() deprecation

| Q             | A
| ------------- | ---
| Branch?       | 3.2
| Fixed tickets| #20990

Commits
-------

570a76c Add missing upgrade path for Request::isMethodSafe() deprecation
  • Loading branch information
fabpot committed Dec 20, 2016
2 parents 411afbe + 570a76c commit 2de9dd1
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
5 changes: 5 additions & 0 deletions UPGRADE-3.2.md
Expand Up @@ -94,6 +94,11 @@ HttpFoundation
- `isInvalid`/`isSuccessful`/`isRedirection`/`isClientError`/`isServerError`
- `isOk`/`isForbidden`/`isNotFound`/`isRedirect`/`isEmpty`

* Checking only for cacheable HTTP methods with `Request::isMethodSafe()` is deprecated
since version 3.2 and will throw an exception in 4.0. Disable checking only for
cacheable methods by calling the method with `false` as first argument or use
`Request::isMethodCacheable()` instead.

HttpKernel
----------

Expand Down
3 changes: 3 additions & 0 deletions UPGRADE-4.0.md
Expand Up @@ -151,6 +151,9 @@ HttpFoundation
- `isInvalid`/`isSuccessful`/`isRedirection`/`isClientError`/`isServerError`
- `isOk`/`isForbidden`/`isNotFound`/`isRedirect`/`isEmpty`

* The ability to check only for cacheable HTTP methods using `Request::isMethodSafe()` is
not supported anymore, use `Request::isMethodCacheable()` instead.

HttpKernel
----------

Expand Down

0 comments on commit 2de9dd1

Please sign in to comment.