TYPO3 Core

Hi,

sys_catgory doesn't feature a slug field by default (by core). This must be added by a 3rd party extension. I just tested it with latest ext:news on latest 9.5 and couldn't reproduce it.

Could you please provide the configuration for your slug field in sys_category. Also some information about your installation like where do you store the categories and which languages are in use. If possible also Stack trace would help to narrow this down.

Editing a tt_address record throws the same error.

We are running:

tt_address 5.1.2
news 8.4.0
typo3 9.5.21

Hi,

thanks Daniel for the stack trace. As I already thought the problem is the usage of the \TYPO3\CMS\Core\Compatibility\PseudoSiteTcaDisplayCondition::isInPseudoSite() for the slug fields introduced by 3rd party e.g. ext:news or ext:tt_address.

Since https://review.typo3.org/c/Packages/TYPO3.CMS/+/65066 this now breaks.

The reason is that the used display condition is only intended to be used for pages and is furthermore marked as internal. Using it for other record types was never intended and could have been broken already before.

See the PHPDoc:

/**
 * A display condition that returns true if the page we are dealing
 * with is in a page tree that is represented by a PseudoSite object.
 *
 * This is used to suppress the 'slug' field in pseudo site page trees
 * when editing page records and to show the alias field.
 *
 * Both "Pseudo sites" and "alias" db field will bite the dust in TYPO3 v10.0,
 * so this is a temporary display condition for v9 only and thus marked internal.
 *
 * @internal Implementation and class will probably vanish in TYPO3 v10.0 without further notice
 */

Prior to the relevant core patch, $defaultLanguagePageId fell back to 0 by coincidence (for default records).

$defaultLanguagePageId = (int)($parameters['record']['t3ver_oid'] ?: $parameters['record']['uid']);
if (is_array($parameters['record']['l10n_parent'])) {
    // But if the page is a localized page, we take the l10n_parent as uid for the sitematcher
    // This is 0 if the page in a default language page
    $defaultLanguagePageId = (int)$parameters['record']['l10n_parent'][0];
}

This is not longer the case for default language records as the second condition is not matched anymore.

&& (int)$parameters['record']['sys_language_uid'][0] > 0

Side note: It still "works" for records which have the same uid as a pages record. It also "works" for L>0 records because they still fall back to 0.

However, this is not a core bug but a misuse of the internal DisplayCond by 3rd party extensions.

E.g. in the latest tt_address this is already fixed, see: https://github.com/FriendsOfTYPO3/tt_address/commit/4501d1f30f3381a235e447567018e6d5646ec069

Please report the problem to the corresponding extension authors. I will close this issue for now. If you feel there is still something not working in the core, please let me know and I'll re-open the issue or create a new one.