Project

General

Profile

Actions
Copy link

Bug #104217

closed

mispelled function name on TypoScriptFrontendController::isStaticCacheble()

Added by Riccardo De Contardi 5 months ago. Updated about 1 month ago.

Status:
Rejected
Priority:
Could have
Assignee:
-
Category:
Code Cleanup
Target version:
-
Start date:
2024-06-26
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
13
PHP Version:
8.2
Tags:
Complexity:
no-brainer
Is Regression:
Sprint Focus:
On Location Sprint

Description

There is a typo on the function name TypoScriptFrontendController::isStaticCacheble()

I guess it should be:

isStaticCacheable

Occurrences found:

typo3/sysext/install/Configuration/ExtensionScanner/Php/MethodCallMatcher.php
typo3/sysext/core/Documentation/Changelog/13.0/Breaking-102621-MostTSFEMembersMarkedInternalOrRead-only.rst
typo3/sysext/frontend/Classes/Controller/TypoScriptFrontendController.php

Found on TYPO3 12,13 (not tested on 11)

kudos to Mathias Brodala for finding it!

Related issues 1 (0 open — 1 closed)

Related to TYPO3 Core - Task #105230: Declare TypoScriptFrontendController deprecatedClosed2024-10-08

Actions
Actions
Copy link
 #1

Updated by Riccardo De Contardi 5 months ago

  • Subject changed from mispelled function name on to mispelled function name on TypoScriptFrontendController::isStaticCacheble()
Actions
Copy link
 #2

Updated by Georg Ringer 4 months ago

  • Status changed from New to Accepted
  • Priority changed from Should have to Could have
Actions
Copy link
 #3

Updated by Gerrit Code Review 4 months ago

  • Status changed from Accepted to Under Review

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

Actions
Copy link
 #4

Updated by Gerrit Code Review 4 months ago

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

Actions
Copy link
 #5

Updated by Gerrit Code Review 4 months ago

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

Actions
Copy link
 #6

Updated by Gerrit Code Review 4 months ago

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

Actions
Copy link
 #7

Updated by Gerrit Code Review 4 months ago

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

Actions
Copy link
 #8

Updated by Gerrit Code Review 4 months ago

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

Actions
Copy link
 #9

Updated by Gerrit Code Review 4 months ago

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

Actions
Copy link
 #10

Updated by Christian Kuhn about 1 month ago

  • Related to Task #105230: Declare TypoScriptFrontendController deprecated added
Actions
Copy link
 #11

Updated by Christian Kuhn about 1 month ago

  • Status changed from Under Review to Rejected

TypoScriptFrontendController is being fully deprecated. We should not fix a typo together with a migration path in a deprecated class anymore.

Actions
Copy link

Also available in: Atom PDF