Bug #26133
closed
Fix broken string replacement for media/ folder
0%
Description
The function setAbsRefPrefix() of the class tslib_fe includes the following string replacement, which is obsolete and should be removed.
$this->content = str_replace('"media/', '"'.t3lib_extMgm::siteRelPath('cms').'tslib/media/', $this->content);
Updated by Mr. Hudson over 13 years ago
Patch set 1 of change Ie9736d815308c20f609f58da2cda735dcdc0a037 has been pushed to the review server.
It is available at http://review.typo3.org/2363
Updated by Susanne Moog over 13 years ago
- Status changed from New to Under Review
Updated by Patrick Rodacker over 13 years ago
From Gerrit:
Susanne Moog May 24 Patch Set 1: I would prefer that you didn't submit this As far as I can see we should not remove the replacement as TYPO3 core still allows the media/ folder as valid place for storing scripts and media files (see t3lib_tstemplate for example) - if someone is still using that directory for file inclusion this patch will break paths if absRefPrefix is used. We should deprecate the inclusion of files in the media folder first.
How do we deprecate the inclusion from the media/ folder first?
I read in another post, that a htaccess file with an AJAX call to write the log could be used.
Updated by Steffen Gebert almost 13 years ago
- Status changed from Under Review to Rejected
Updated by Steffen Gebert almost 13 years ago
- Status changed from Rejected to Under Review
Updated by Chris topher over 12 years ago
Now this folder has been removed from the TYPO3 Core. The replacement, which you quoted above, is still there and will not work anymore. Instead the files from statictemplates/media should be used.
Updated by Chris topher over 12 years ago
- Subject changed from remove obsolete string replacement for media/ folder in tslib_fe::setAbsRefPrefix to Fix broken string replacement for media/ folder
- Target version set to 6.0.0-pre-beta1
Updated by Alexander Opitz over 11 years ago
- Status changed from Under Review to New
- Target version deleted (
6.0.0-pre-beta1)
Patch was abandoned
Updated by Riccardo De Contardi almost 10 years ago
the line reported is still present in 4.5.39, but does not exists in 6.2.9
Updated by Wouter Wolters almost 10 years ago
- Status changed from New to Closed
- Is Regression set to No
Hi, as we removed the complete media stuff I'll close this ticket.