minor #24586 [HttpKernel] Remove obsolete PHPDoc from UriSigner (emodric)

stof · stof · commit bb8f1534d2cd · 2017-10-17T14:57:01.000+02:00
This PR was merged into the 2.7 branch. Discussion ---------- [HttpKernel] Remove obsolete PHPDoc from UriSigner | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | N/A | License | MIT | Doc PR | N/A `_hash` query param does not have to be at the end of the URL ever since #12574 so this PHPDoc is confusing. I've actually lost couple of hours of work rewriting my URLs to place it at the end before I realized that `UriSigner` doesn't really care. Commits ------- 45ac192 Remove obsolete PHPDoc from UriSigner
diff --git a/src/Symfony/Component/HttpKernel/UriSigner.php b/src/Symfony/Component/HttpKernel/UriSigner.php
@@ -55,10 +55,6 @@ public function sign($uri)
     /**
      * Checks that a URI contains the correct hash.
      *
-     * The _hash query string parameter must be the last one
-     * (as it is generated that way by the sign() method, it should
-     * never be a problem).
-     *
      * @param string $uri A signed URI
      *
      * @return bool True if the URI is signed correctly, false otherwise

Original file line number	Diff line number	Diff line change
`@@ -55,10 +55,6 @@ public function sign($uri)`
`55`	`55`	`/**`
`56`	`56`	`* Checks that a URI contains the correct hash.`
`57`	`57`	`*`
`58`		`- * The _hash query string parameter must be the last one`
`59`		`- * (as it is generated that way by the sign() method, it should`
`60`		`- * never be a problem).`
`61`		`- *`
`62`	`58`	`* @param string $uri A signed URI`
`63`	`59`	`*`
`64`	`60`	`* @return bool True if the URI is signed correctly, false otherwise`