Task #92583

Remove option that BackendUtility::wrapClickMenuOnIcon() can return an array

Added by Daniel Goerz 6 days ago. Updated 4 days ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2020-10-17
Due date:
% Done:

100%

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

Associated revisions

Revision 3dc97cfd (diff)
Added by Daniel Goerz 4 days ago

[TASK] Deprecate last arguments of wrapClickMenuOnIcon()

BackendUtility::wrapClickMenuOnIcon() has a boolean flag
to let the method return an array with tag parameters
instead of a fully build HTML tag as string.

As this are two completely different things and cause
problems when analysing return types it should not be
done in the same method.

Calling BackendUtility::wrapClickMenuOnIcon() with the
7th and last argument $returnTagParameters set to true
has been deprecated alongside the 5th and 6th arguments
that are already unused.

Resolves: #92583
Releases: master
Change-Id: Ia536aa3a52085e4ebe7956997b505047340102e7
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/66161
Tested-by: TYPO3com <>
Tested-by: Christian Kuhn <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Oliver Klee <>
Reviewed-by: Christian Kuhn <>
Reviewed-by: Anja Leichsenring <>

History

#1 Updated by Gerrit Code Review 6 days 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/c/Packages/TYPO3.CMS/+/66161

#2 Updated by Gerrit Code Review 6 days 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/c/Packages/TYPO3.CMS/+/66161

#3 Updated by Gerrit Code Review 6 days 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/c/Packages/TYPO3.CMS/+/66161

#4 Updated by Gerrit Code Review 6 days 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/c/Packages/TYPO3.CMS/+/66161

#5 Updated by Gerrit Code Review 6 days 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/c/Packages/TYPO3.CMS/+/66161

#6 Updated by Gerrit Code Review 6 days 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/+/66161

#7 Updated by Gerrit Code Review 6 days 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/+/66161

#8 Updated by Gerrit Code Review 5 days 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/+/66161

#9 Updated by Daniel Goerz 4 days ago

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

Also available in: Atom PDF