Task #89463

Epic #88528: Remove switchable controller action functionality

Deprecate switchable controller actions

Added by Alexander Schnitzler almost 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Category:
Extbase
Target version:
-
Start date:
2019-10-21
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Sprint Focus:

Related issues

Related to TYPO3 Core - Bug #89922: Correct code-block directiveClosed2019-12-11

Actions
#1

Updated by Gerrit Code Review almost 2 years ago

  • Status changed from New to Under Review

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/c/Packages/TYPO3.CMS/+/61511

#2

Updated by Gerrit Code Review almost 2 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/c/Packages/TYPO3.CMS/+/61511

#3

Updated by Gerrit Code Review almost 2 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/c/Packages/TYPO3.CMS/+/61511

#4

Updated by Gerrit Code Review almost 2 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/c/Packages/TYPO3.CMS/+/61511

#5

Updated by Gerrit Code Review almost 2 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61511

#6

Updated by Gerrit Code Review almost 2 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61511

#7

Updated by Gerrit Code Review almost 2 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61511

#8

Updated by Gerrit Code Review almost 2 years ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61511

#9

Updated by Anonymous almost 2 years ago

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

Updated by Chris Müller almost 2 years ago

  • Related to Bug #89922: Correct code-block directive added
#11

Updated by Benni Mack almost 2 years ago

  • Status changed from Resolved to Closed
#12

Updated by Dmitry Dulepov over 1 year ago

Benni Mack

Suppose we have a news plugin with 12 custom display modes. This is a real life scenario. Currently the plugin uses switchableControllerActions. This deprecation means that instead of having one plugin with 12 selectable modes we now need 12 separate plugins, right? Does that really sounds well thought and reasonable to have a plugin selector go beyond horizon vertically?

#13

Updated by Raphael Zschorsch over 1 year ago

I must say, I agree with Dmitry Dulepov here. A custom plugin for each view of each extension could potentially result in around 20-30 plugins in the content element selector, and I imagine this would be quite confusing for editors and admins alike.

#14

Updated by Alexander Schnitzler over 1 year ago

Please have a look at my post, explaining a few things in detail:
https://gist.github.com/alexanderschnitzler/c685218feea1a8956cc3f915f7a08d0b

TLDR;
No, you don't need to create a single plugin for each controller/action combination, only for those entrypoints where controller and action params should be hidden in the url.

Also available in: Atom PDF