Project

General

Profile

Actions

Bug #90949

open

persistence.storagePid ignores startingpoint (pages), when it is a stdWrap

Added by Christian Futterlieb almost 4 years ago. Updated 22 days ago.

Status:
New
Priority:
Should have
Assignee:
-
Category:
Extbase
Target version:
Start date:
2020-04-04
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

Consider following situation:

TypoScript setup:

plugin.tx_myext.persistence {
    storagePid.ifEmpty.data = TSFE:id
}

ContentObject:

Array (
    [data] => Array (
        [CType] => list
        [list_type] => myext_myplugin
        [pages] => 123
    )
)

Now, one could expect, that the definitive storagePid would be 123, but it is the value of TSFE:id. This is because ArrayUtility::mergeRecursiveWithOverrule() does not override the storagePid FrontendConfigurationManager::overrideStoragePidIfStartingPointIsSet(), when the storagePid is an array. Which is true, when a config like the one above is used. This leads to the situation, where the value argument passed to stdWrap() would always be the one from TypoScript.

Affected TYPO3 versions: 9.5 and master.

I'll submit a patch in a moment.

Actions #1

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

Actions #2

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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/64074

Actions #3

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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/64074

Actions #4

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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/64074

Actions #5

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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/64074

Actions #6

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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/64074

Actions #7

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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/64074

Actions #8

Updated by Gerrit Code Review over 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/+/64074

Actions #9

Updated by Gerrit Code Review about 2 years ago

Patch set 9 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/+/64074

Actions #10

Updated by Gerrit Code Review 10 months ago

Patch set 10 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/+/64074

Actions #11

Updated by Benni Mack 22 days ago

  • Status changed from Under Review to New
Actions

Also available in: Atom PDF