Task #28024

Remove tslib/media, cglify statictemplates/media

Added by Dimitri Koenig about 4 years ago. Updated about 3 years ago.

Status:Resolved Start date:2011-07-09
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:- Spent time: -
Target version:-
TYPO3 Version:4.6 Complexity:
PHP Version: Sprint Focus:

Description

One of them should be removed. But there are small changes in some files in both directories and subdirectories so one has to decide what to remove and what to modify and so on...


Related issues

related to Core - Bug #46644: Layermenu code is duplicated Resolved 2013-03-25
related to TypoScript Reference (TSref) - Task #26638: The folder "cms/tslib/media" has been removed Closed 2011-05-05
related to Core - Bug #26133: Fix broken string replacement for media/ folder Closed 2011-04-19
duplicated by Core - Bug #25028: Remove cms/tslib/media folder to TER extension Closed 2011-02-14

Associated revisions

Revision ff9e6a88
Added by Steffen Gebert about 4 years ago

[BUGFIX][INSTALL] Show description for enableDeprecationLog as <dl>

Currently it's listed inline in the text, make it a definition list.

Change-Id: I18c2df10a0e18dbc167b40465192909cf30278e6
Resolves: #28024
Reviewed-on: http://review.typo3.org/3174
Reviewed-by: Philipp Gampe
Reviewed-by: Stefan Neufeind
Reviewed-by: Xavier Perseguers
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack

Revision 5685f098
Added by Susanne Moog about 3 years ago

[TASK] Remove tslib/media, cglify statictemplates/media

The media folder in the TYPO3 core is duplicated: It resides once
in tslib/media and once in statictemplates/media. It is only used
in statictemplates.

This patch moves the already "cglfixed" files from tslib/media to
statictemplates and removes the superfluous media folder.

Change-Id: I4c740849fd0d147590cf28efd6ae0680fc66b779
Resolves: #28024
Releases: 6.0
Reviewed-on: http://review.typo3.org/12334
Reviewed-by: Markus Klein
Reviewed-by: Wouter Wolters
Tested-by: Markus Klein
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

Revision 5048ff73
Added by Susanne Moog about 3 years ago

[TASK] Remove tslib/media, cglify statictemplates/media

The media folder in the TYPO3 core is duplicated: It resides once
in tslib/media and once in statictemplates/media. It is only used
in statictemplates.

This patch moves the already "cglfixed" files from tslib/media to
statictemplates and removes the superfluous media folder.

Change-Id: I4c740849fd0d147590cf28efd6ae0680fc66b779
Resolves: #28024
Releases: 6.0
Reviewed-on: http://review.typo3.org/12334
Reviewed-by: Markus Klein
Reviewed-by: Wouter Wolters
Tested-by: Markus Klein
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

History

#1 Updated by Mr. Hudson about 4 years ago

Patch set 1 of change I18c2df10a0e18dbc167b40465192909cf30278e6 has been pushed to the review server.
It is available at http://review.typo3.org/3174

#2 Updated by Dimitri Koenig about 4 years ago

something went wrong. hudson has a bug!

#3 Updated by Steffen Gebert about 4 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

#4 Updated by Dimitri Koenig about 4 years ago

Wrong Association. Still Bug in Hudson?

#5 Updated by Steffen Gebert about 4 years ago

  • Status changed from Resolved to New

No, sorry. My fault!

I referenced the wrong issue in the commit message!

Resolves: #28024

should be
Resolves: #28025

#6 Updated by Mr. Hudson about 4 years ago

Patch set 1 of change I18c2df10a0e18dbc167b40465192909cf30278e6 has been pushed to the review server.
It is available at http://review.typo3.org/3658

#7 Updated by Gerrit Code Review about 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12334

#8 Updated by Chris topher about 3 years ago

  • Subject changed from sysext/statictemplates/media and sysext/cms/tslib/media are almost the same to Remove tslib/media, cglify statictemplates/media

#9 Updated by Gerrit Code Review about 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12334

#10 Updated by Gerrit Code Review about 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12334

#11 Updated by Gerrit Code Review about 3 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12334

#12 Updated by Susanne Moog about 3 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF