Actions
Bug #92568
closed
PopulatePageSlugs upgrade wizard migrates wrong RealURL path data entry
Status:
Closed
Priority:
Must have
Assignee:
Category:
Link Handling, Site Handling & Routing
Target version:
Start date:
2020-10-15
Due date:
% Done:
100%
Estimated time:
TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
Description
The PopulatePageSlugs upgrade wizard allows to migrate entries from tx_realurl_pathdata. However, due to an additional DESC sorting, the wrong entry is used in case there is more than one for a page. This can actually happen often for $reasons.
Given the following two entries in tx_realurl_pathdata:
| uid | ... | pagepath | expire |
|---|---|---|---|
| 1 | ... | first/path | 0 |
| 2 | ... | second/path | 0 |
RealURL will use the first/path, but the PopulatePageSlugs upgrade wizard will use the second/path. This leads to changed URLs after the upgrade.
RealURL itself does not apply any explicit sorting:
https://github.com/dmitryd/typo3-realurl/blob/2.6.2/Classes/Cache/DatabaseCache.php#L286
$row = $this->databaseConnection->exec_SELECTgetSingleRow('*', 'tx_realurl_pathdata',
'pid=' . (int)$pageId .
' AND language_id=' . (int)$languageId .
' AND rootpage_id=' . (int)$rootPageId .
' AND mpvar=' . ($mpVar ? $this->databaseConnection->fullQuoteStr($mpVar, 'tx_realurl_pathdata') : '\'\'') .
' AND expire=0'
);
Thus in this case ASC is used by default.
Updated by 
Updated by 
Updated by 
Updated by
Actions