Bug #53826
closed
JavaScript error when using foldertree (for files) selector popup in RTE, and icons not showing
100%
Description
When using the RTE to link to a file or folder in the filesystem, or when using the Element Browser for linking a header, a Javascript error occurs when navigating with the folder tree.
In both dialogs the error "Uncaught ReferenceError: jumpTo is not defined" is thrown to the console, when clicking on one of the newly visible folders after expanding a formerly closed tree node.
Additionally in the RTE popup the icons for the newly expanded subtree are not shown due to a wrong path.
The error occurs because the tree implementation is not responding correctly to which scope it is rendered to.
Screencast: http://screencast.com/t/FQBwOxeEP
Files
Updated by Markus Klein over 10 years ago
- Status changed from New to Needs Feedback
- Assignee deleted (
Jan-Erik Revsbech)
Hi Jan-Erik!
Can you please test if the related issue fixes this for you?
The patch will be included in 6.1.6.
Updated by Jan-Erik Revsbech over 10 years ago
- Status changed from Needs Feedback to New
- Assignee set to Jan-Erik Revsbech
Hi Markus, it does not. We tried it, but it still breaks. We have a patch that we will submit very shortly.
Updated by Gerrit Code Review over 10 years ago
- Status changed from New 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/25580
Updated by Jan-Erik Revsbech over 10 years ago
- Status changed from Under Review to New
It appears that the exact same error is present when linking to files using the "Link" popup for links. Example the links field of the Header has the same error, and the patch set does not fix this. Will try to make a new patchset that fixes this.
Updated by Gerrit Code Review over 10 years ago
- Status changed from New 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 https://review.typo3.org/25633
Updated by Markus Klein over 10 years ago
What is the exact error that is produced.
I'm having troubles reproducing this. Could you please give a step by step instruction here.
Updated by Gerrit Code Review over 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 https://review.typo3.org/25633
Updated by Gerrit Code Review over 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 https://review.typo3.org/25633
Updated by Markus Klein over 10 years ago
- Priority changed from Should have to Must have
Updated by Gerrit Code Review over 10 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25633
Updated by Gerrit Code Review over 10 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/25633
Updated by Gerrit Code Review over 10 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/25633
Updated by Gerrit Code Review over 10 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/25633
Updated by Gerrit Code Review over 10 years ago
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/26816
Updated by Gerrit Code Review over 10 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/26817
Updated by Anonymous over 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset f3d324da4c2b382441f3ec080d27b4cf5be0032a.
Updated by 