Bug #46644

Layermenu code is duplicated

Added by Michael Stucki over 2 years ago. Updated over 2 years ago.

Status:Resolved Start date:2013-03-25
Priority:Should have Due date:
Assigned To:Michael Stucki % Done:

100%

Category:Frontend Spent time: -
Target version:-
TYPO3 Version:4.7 Is Regression:
PHP Version: Sprint Focus:
Complexity:

Description

The code which is used by GMENU_LAYERS/TMENU_LAYERS exists in "cms" as well as in "statictemplates".
This is even worse as a few issues have been fixed only in one of the versions.


Related issues

related to Core - Task #28024: Remove tslib/media, cglify statictemplates/media Resolved 2011-07-09

Associated revisions

Revision 7c536c7d
Added by Michael Stucki over 2 years ago

[BUGFIX] Remove redundant layermenu files

These files were maintained as two individual version. A number of
bugfixes were only applied in one version bot not in the other.

This change adds all missing fixes to the files in "statictemplates" and
removes all code from the files in "cms".

Resolves: #46644
Releases: 4.5, 4.7
Change-Id: I56ce401c73933eda381ce52b4d906a12b6ed963c
Reviewed-on: https://review.typo3.org/19259
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Revision 27df6d87
Added by Michael Stucki over 2 years ago

[BUGFIX] Remove redundant layermenu files

These files were maintained as two individual version. A number of
bugfixes were only applied in one version bot not in the other.

This change adds all missing fixes to the files in "statictemplates" and
removes all code from the files in "cms".

Resolves: #46644
Releases: 4.5, 4.7
Change-Id: I56ce401c73933eda381ce52b4d906a12b6ed963c
Reviewed-on: https://review.typo3.org/19261
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Michael Stucki over 2 years ago

This is almost the same problem that was already described and solved in #28024. However, the change was not applied to pre-6.0 releases yet.

#2 Updated by Michael Stucki over 2 years ago

  • Status changed from New to Accepted
  • Assigned To set to Michael Stucki

#3 Updated by Gerrit Code Review over 2 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19259

#4 Updated by Gerrit Code Review over 2 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at https://review.typo3.org/19260

#5 Updated by Gerrit Code Review over 2 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/19261

#6 Updated by Gerrit Code Review over 2 years ago

Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19259

#7 Updated by Gerrit Code Review over 2 years ago

Patch set 2 for branch TYPO3_4-6 has been pushed to the review server.
It is available at https://review.typo3.org/19260

#8 Updated by Gerrit Code Review over 2 years ago

Patch set 2 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/19261

#9 Updated by Gerrit Code Review over 2 years ago

Patch set 3 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19259

#10 Updated by Gerrit Code Review over 2 years ago

Patch set 4 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19259

#11 Updated by Gerrit Code Review over 2 years ago

Patch set 3 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/19261

#12 Updated by Michael Stucki over 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF