Task #73068

Breaking <f:case default="true">

Added by Xavier Perseguers over 4 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2016-02-01
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Due to the switch to Fluid standalone, the following template markup is not supported anymore:

<f:case default="true"> ... </f:case>

It must be changed to read:

<f:defaultCase> ... </f:defaultCase>

This breaking change is properly documented in #69863. However there is no possible compatibility fallback and this means extension authors are unable to ship code compatible with TYPO3 7 and 8 (not even speaking about 6.2 which is still a valid supported LTS version) without adding much logic to change the template in use.

Furthermore, adding this business logic is likely to break the work of integrators who probably adapted the templates to fit their need.

A compatibility layer should really be added.


Related issues

Follows TYPO3 Core - Feature #69863: Use new standalone Fluid as composer dependency Closed 2015-09-16

Associated revisions

Revision 5d4f4a3a (diff)
Added by Claus Due over 4 years ago

[TASK] Compatibility to safely deprecate f:case "default" argument

A compatibility layer can be left in place to support "default" used
on f:case. Internally the compatibility ViewHelper changes the class
name used in the ViewHelperNode which gets reflected in the compiled
template as well as the uncompiled rendering.

ReflectionProperty had to be used since no public API exists for
manipulating ViewHelperNodes in this way.

Change-Id: I80c5c8787f6e88f37dfe129ea1844c4421f77d19
Resolves: #73068
Releases: master
Reviewed-on: https://review.typo3.org/46431
Reviewed-by: Daniel Goerz <>
Tested-by: Daniel Goerz <>
Reviewed-by: Xavier Perseguers <>
Tested-by: Xavier Perseguers <>

History

#1 Updated by Gerrit Code Review over 4 years ago

  • Status changed from New 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 https://review.typo3.org/46431

#2 Updated by Gerrit Code Review over 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 https://review.typo3.org/46431

#3 Updated by Gerrit Code Review over 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 https://review.typo3.org/46431

#4 Updated by Gerrit Code Review over 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 https://review.typo3.org/46431

#5 Updated by Gerrit Code Review over 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 https://review.typo3.org/46431

#6 Updated by Gerrit Code Review over 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 https://review.typo3.org/46431

#7 Updated by Gerrit Code Review over 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 https://review.typo3.org/46431

#8 Updated by Gerrit Code Review over 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 https://review.typo3.org/46431

#9 Updated by Anonymous over 4 years ago

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

#10 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF