Bug #60316
Parsing RTE links should not throw an exception if a resource is missing or its path invalid
| Status: | Resolved | Start date: | 2014-07-14 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Helmut Hummel | % Done: | 100% |
|
| Category: | rtehtmlarea | Spent time: | - | |
| Target version: | 7.0 | |||
| TYPO3 Version: | 6.2 | Is Regression: | No | |
| PHP Version: | 5.4 | Sprint Focus: | ||
| Complexity: |
Related issues
Associated revisions
[BUGFIX] Catch all resource exceptions when parsing RTE links
The RteHtmlParser checks for valid links by trying to retrieve
a file or folder object. The FAL API will throw exceptions
if anything goes wrong with the given path.
Catch all these exceptions to ignore the invalid link
and act the same for invalid paths as for not found files/ folders
Releases: 6.2, 6.3
Resolves: #60316
Change-Id: I0752678eb8ed2b596c3a88536d957ec35c172d7b
Reviewed-on: http://review.typo3.org/31619
Reviewed-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
[BUGFIX] Catch all resource exceptions when parsing RTE links
The RteHtmlParser checks for valid links by trying to retrieve
a file or folder object. The FAL API will throw exceptions
if anything goes wrong with the given path.
Catch all these exceptions to ignore the invalid link
and act the same for invalid paths as for not found files/ folders
Releases: 6.2, 6.3
Resolves: #60316
Change-Id: I0752678eb8ed2b596c3a88536d957ec35c172d7b
Reviewed-on: http://review.typo3.org/31745
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
History
#1 Updated by Helmut Hummel about 1 year ago
Steps to reproduce:
- Create tt_content text element
- Disable RTE
- put the following into the bodytext field and save
<link ../fileadmin/foo>bar</link>
- enable RTE and quit
- open this element again
#2 Updated by Helmut Hummel about 1 year ago
- File exception.png added
Steps to reproduce:
- Create tt_content text element
- Disable RTE
- put the following into the bodytext field and save
<link ../fileadmin/foo>bar</link>
- enable RTE and quit
- open this element again
#3 Updated by Gerrit Code Review about 1 year ago
- Status changed from Accepted to Under Review
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/31745
#4 Updated by Helmut Hummel about 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset f9f4b49255d3ac19ee64e1942324466e5d34ec5c.