Task #70477

Deprecate SpriteIcon classes

Added by Frank Naegler almost 4 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Backend API
Target version:
Start date:
2015-10-07
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Sprint Focus:
Remote Sprint

Description

The following classes should be deprecated and removed in CMS8:

  • SpriteManager
  • AbstractSpriteHandler
  • SimpleSpriteHandler
  • SpriteBuildingHandler

Related issues

Related to TYPO3 Core - Task #24840: spritemanager and clear cache Closed 2011-01-26
Related to TYPO3 Core - Bug #72992: No action icons in the workspace module anymore Closed 2016-01-28
Related to TYPO3 Core - Bug #76996: Language flags (css classes and sprites) got removed in 7.6.0 Closed 2016-07-09

Associated revisions

Revision 58c46d75 (diff)
Added by Frank Naegler almost 4 years ago

[TASK] Deprecate SpriteIcon classes

This patch deprecates all SpriteIcon classes:

  • \TYPO3\CMS\Backend\Sprite\SpriteManager
  • \TYPO3\CMS\Backend\Sprite\AbstractSpriteHandler
  • \TYPO3\CMS\Backend\Sprite\SimpleSpriteHandler
  • \TYPO3\CMS\Backend\Sprite\SpriteBuildingHandler

Resolves: #70477
Releases: master
Change-Id: Ib71e0651c17c8de6cac7c3dcd602c06356e823fa
Reviewed-on: http://review.typo3.org/43897
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision 3c47cec7 (diff)
Added by Frank Naegler almost 3 years ago

[BUGFIX] Fix broken language flags via IconUtility::getSpriteIcon

With #70477 the SpriteIcon stuff has been marked as deprecated.
The patch removed the flags support. This patch fix the problem by checking
the IconRegisty and return the icon by using the IconFactory.

Resolves: #76996
Related: #70477
Releases: 7.6
Change-Id: I0323c6bc132eb9a8049294ee66ac3cf37d10f9fd
Reviewed-on: https://review.typo3.org/49161
Reviewed-by: Thomas Mayer <>
Tested-by: Thomas Mayer <>
Reviewed-by: Wouter Wolters <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review almost 4 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43897

#2 Updated by Simon Schaufelberger almost 4 years ago

Alternatives? Documentation?

#3 Updated by Gerrit Code Review almost 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43897

#4 Updated by Gerrit Code Review almost 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43897

#5 Updated by Gerrit Code Review almost 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43897

#6 Updated by Gerrit Code Review almost 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43897

#7 Updated by Gerrit Code Review almost 4 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43897

#8 Updated by Gerrit Code Review almost 4 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43897

#9 Updated by Gerrit Code Review almost 4 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43897

#10 Updated by Frank Naegler almost 4 years ago

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

#11 Updated by Riccardo De Contardi over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF