Bug #61960
closedf:widget.link always generates links with cHash
Added by Dmitry Dulepov about 10 years ago. Updated about 6 years ago.
100%
Description
In certain cases links do not need the cHash (for example, search results or uncached action). But there is no parameter to avoid cHash in links made by this viewhelper.
Problematic file is ./typo3/sysext/fluid/Classes/ViewHelpers/Widget/LinkViewHelper.php
Found this while using the pagebrowser widget.
Files
61960-widget-uri-nocachehash.patch (2.11 KB) 61960-widget-uri-nocachehash.patch | noCacheHash parameter for <f:widget.uri/> | Mathias Brodala, 2014-10-09 09:22 |
Updated by Georg Ringer about 10 years ago
- Status changed from New to Needs Feedback
what about noCacheHash="0"
?
Updated by Dmitry Dulepov about 10 years ago
Have you looked to the file?
protected function getWidgetUri() { $uriBuilder = $this->controllerContext->getUriBuilder(); $argumentPrefix = $this->controllerContext->getRequest()->getArgumentPrefix(); $arguments = $this->hasArgument('arguments') ? $this->arguments['arguments'] : array(); if ($this->hasArgument('action')) { $arguments['action'] = $this->arguments['action']; } if ($this->hasArgument('format') && $this->arguments['format'] !== '') { $arguments['format'] = $this->arguments['format']; } if ($this->hasArgument('addQueryStringMethod') && $this->arguments['addQueryStringMethod'] !== '') { $arguments['addQueryStringMethod'] = $this->arguments['addQueryStringMethod']; } return $uriBuilder->reset()->setArguments(array($argumentPrefix => $arguments))->setSection($this->arguments['section'])->setAddQueryString(TRUE)->setAddQueryStringMethod($this->arguments['addQueryStringMethod'])->setArgumentsToBeExcludedFromQueryString(array($argumentPrefix, 'cHash'))->setFormat($this->arguments['format'])->build(); }
Where do you see anything about noCacheHash here? :)
Updated by Georg Ringer about 10 years ago
- Status changed from Needs Feedback to Accepted
sry, true. wanna add a patch?
Updated by Mathias Brodala about 10 years ago
Untested but this patch should do.
Updated by Gerrit Code Review almost 10 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/34689
Updated by Gerrit Code Review almost 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/34689
Updated by Gerrit Code Review almost 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/34689
Updated by Gerrit Code Review over 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/34689
Updated by Gerrit Code Review over 9 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/34689
Updated by Gerrit Code Review about 9 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/34689
Updated by Gerrit Code Review over 8 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/34689
Updated by Gerrit Code Review almost 7 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/34689
Updated by Gerrit Code Review almost 7 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/34689
Updated by Gerrit Code Review almost 7 years ago
Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/34689
Updated by Gerrit Code Review almost 7 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54967
Updated by Benni Mack almost 7 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset b2d34992b2291541c5c168b2281ee182cee76e91.