Bug #38652
closedRTE spell checker is flooding directory typo3temp on Windows
0%
Description
As the spell checker in TYPO3 creates one *.tmp file for every single word of a text the typo3temp directory is crowded of files. The temporary files should be deleted after spell checking was completed.
Updated by Stanislas Rolland over 12 years ago
- Category set to RTE (rtehtmlarea + ckeditor)
Updated by Stanislas Rolland over 12 years ago
The files are indeed deleted with t3lib_div::unlink_tempfile()...
I am unable to reproduce this issue.
Maybe on Windows the file is not deleted if not closed...
Updated by Stanislas Rolland over 12 years ago
- Subject changed from Spell checker is flooding directory typo3temp to RTE spell checker is flooding directory typo3temp on Windows
- Assignee set to Stanislas Rolland
- Target version set to 4.5.18
- PHP Version set to 5.3
Updated by Gerrit Code Review over 12 years ago
- Status changed from New to Under Review
Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/12643
Updated by Stanislas Rolland over 12 years ago
Please test this change on a Windows server: http://review.typo3.org/12643
Updated by Gerhard Rupp over 12 years ago
Even with the patched version the files remain in the temp directory.
Updated by Gerrit Code Review over 12 years ago
Patch set 2 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/12643
Updated by Stanislas Rolland over 12 years ago
Please test the second patch set.
Will be running out of ideas: http://ca2.php.net/manual/en/function.unlink.php
Updated by Gerhard Rupp over 12 years ago
The problem is related to windows path names containing backslashes.
Replacing the backslashes by adding
$tmpFileName = str_replace('\\', '/', $tmpFileName);
does the job.
But I think this replacement should be done in the unlink_tempfile function.
Updated by Stanislas Rolland over 12 years ago
Please test this change and give your vote in gerrit: http://review.typo3.org/12684
Updated by Stanislas Rolland over 12 years ago
Gerhard Rupp wrote:
Should this work under 4.5?
I think so.
Updated by Gerhard Rupp over 12 years ago
I can´t get it running under 4.5. If I only replace the function unlink_tempfile everything is fine.
Updated by Stanislas Rolland over 12 years ago
Gerhard Rupp wrote:
I can´t get it running under 4.5. If I only replace the function unlink_tempfile everything is fine.
Here is the same change for TYPO3 4.5: http://review.typo3.org/12754
Updated by Gerhard Rupp over 12 years ago
I can confirm that this patch works on a windows server on TYPO3 4.5.18.
Updated by Stanislas Rolland about 12 years ago
- Target version changed from 4.5.18 to 4.5.21
Updated by Stanislas Rolland about 12 years ago
- Status changed from Under Review to Resolved
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed