Bug #41413
closedlink title got urlencoded in link-wizard
100%
Description
When opening the link-wizard form a link field, the title got urlencoded.
- Create a new Content-Element, e.g. "Text only"
- find a "Link" field and enter:
1 _top - "foo bar"
- (Title is wrapped by "" and contains a space)
- open the link-wizard by clicking the "chain with pencil"-icon on the right
- see the title got "urlencoded" and transformed to "foo%20bar"
See attached screenshot
Occurred in 4.5 and later versions.
Files
Updated by Stefan Galinski about 12 years ago
- Status changed from New to Accepted
Updated by Mario Hirschfeld about 12 years ago
- File linkproblem.jpg linkproblem.jpg added
Same here but with the link url:
f.e.: fileadmin%2Fuser_upload%2F
The link changed to urlencode after changing title or targetwindow with linkwizard.
Updated by Gerrit Code Review over 11 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/20034
Updated by Gerrit Code Review over 11 years ago
Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/20034
Updated by Gerrit Code Review about 11 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23740
Updated by Gerrit Code Review about 11 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23740
Updated by Gerrit Code Review about 11 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23740
Updated by Gerrit Code Review about 11 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23740
Updated by Chris topher about 11 years ago
- Subject changed from link title got urlencoded n link-wizard to link title got urlencoded in link-wizard
- Is Regression set to No
Updated by Ernesto Baschny about 11 years ago
Maybe this problem #51943 could be tackled at the same time - as it is related.
Updated by Markus Klein almost 11 years ago
- Priority changed from Should have to Must have
Updated by Markus Klein almost 11 years ago
- Target version set to next-patchlevel
- Complexity set to medium
The "additional parameters" field is affected as well!
Affected versions are 4.5 to 6.2
Updated by Gerrit Code Review almost 11 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23740
Updated by Gerrit Code Review almost 11 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23740
Updated by Gerrit Code Review almost 11 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/23740
Updated by Gerrit Code Review almost 11 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/23740
Updated by Gerrit Code Review almost 11 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/23740
Updated by Helmut Hummel almost 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset b44e8af67e681febef7798e66c5ff8e2528bef54.
Updated by Gerrit Code Review almost 11 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27853
Updated by Gerrit Code Review almost 11 years ago
Patch set 2 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27853
Updated by Tobias Wollender almost 11 years ago
With this Patch applied on 6.2 master, the linkbrowser for input fields (e. g. header_link) doesn't work correctly anymore. The tab "Page" gets displayed, but a click on any other tab results in a white page in the linkbrowser. The linkbrowser in the RTE works fine.
Updated by Markus Klein almost 11 years ago
Updated by Tobias Wollender almost 11 years ago
I'm already on latest master, but the problem is as described in comment #21. I cleared all caches, but nothing changes. And unfortunately there is no output in the error_log I can post here.
Updated by Thomas Sperling almost 11 years ago
Same Problem: I think the error is in function jumpToUrl. The URL looks like this: typo3/browse_links.php?PM=0_0_0_browsePages&act=&mode=wizard%27&P[itemName]=data%5Btt_cont ...
Attention to wizard%27 there is one ' (=%27) to much, also at the end of the URL.
Updated by Markus Klein almost 11 years ago
Got it. Fixed in https://review.typo3.org/27924
Please test and vote. Thank you.
Updated by Gerrit Code Review almost 11 years ago
Patch set 3 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27853
Updated by Gerrit Code Review almost 11 years ago
Patch set 1 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27925
Updated by Helmut Hummel almost 11 years ago
- Status changed from Under Review to Resolved
Applied in changeset 1474e2cf09101a73b5f83c58bbbb3463e3dfd216.