[TYPO3-core] RFC #13279: defaults are not applied using "page after" in "new element wizard"
Martin Kutschker
masi-no at spam-typo3.org
Mon Jan 18 11:43:36 CET 2010
Steffen Kamper schrieb:
> Hi Masi,
>
> Martin Kutschker schrieb:
>> Steffen Kamper schrieb:
>>> Hi,
>>>
>>> wait a moment.
>>>
>>> If there is a negative pid this means "after" abs(pid). But reading
>>> PageTS has to be done from parent instead from pid. Example:
>>> // Negative PID values is pointing to a page on the same level as
>>> the current.
>>> if ($positionPid < 0) {
>>> $pidRow = t3lib_BEfunc::getRecordWSOL('pages',
>>> abs($positionPid), 'pid');
>>> $parentPageId = $pidRow['pid'];
>>> } else {
>>> $parentPageId = $positionPid;
>>> }
>>>
>>> this is the equivalent in TV, so we need the same check here.
>>
>> Does this info change anything in my original patch? If anything it
>> shows that the decision to abuse
>> the sign of the pid for something completely unrelated was a bad
>> decision.
>>
>
> yes it does, same chack has to be done there too.
But in a different file, right?
Masi
More information about the TYPO3-team-core
mailing list