Bug #86978

TypoScript getText "site" can't retrieve all properties

Added by Josef Glatz 8 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
TypoScript
Target version:
-
Start date:
2018-11-22
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Problem/Description

According to https://docs.typo3.org/typo3cms/extensions/core/Changelog/9.5/Feature-86422-TypoScriptGetTextPropertySite.html (#86422) it's possible to retrieve anything from the site configuration.

Following keys are working for example:

{site:base}
{site:languages.0.title}
{site:rootPageId}

The following key produces exceptions for example:

{site:configuration.rootPageId}
{site:identifier}
Exception

#1341397869 TYPO3\CMS\Core\Utility\Exception\MissingArrayPathException
Segment configuration of path configuration.rootPageId does not exist in array

#1341397869 TYPO3\CMS\Core\Utility\Exception\MissingArrayPathException
Segment identifier of path identifier does not exist in array

Acceptance Criteria

  • According to the RsT, every configuration option must be available.
    • Otherwise it has to be mentioned in the RsT

Related issues

Related to TYPO3 Core - Feature #86422: Allow usage of site in TypoScript gettext Closed 2018-09-28
Related to TYPO3 Core - Task #87434: site:identifier is not available Closed 2019-01-14
Precedes TYPO3 Core - Bug #87036: TypoScript getText "site" can't retrieve nested properties Rejected 2018-11-23

Associated revisions

Revision d01d6158 (diff)
Added by Georg Ringer 8 months ago

[BUGFIX] Catch exception in getData with invalid key for site configuration

Catch the exception which is thrown if provided path does not exist in
site configuration. Especially if the site configuration is extended
and the desired configuration is not yet persisted the frontend must
be still available.

The exception is logged to be able to find the error.

Resolves: #86978
Releases: master
Change-Id: Iad70acc8a76e0f77875c2cfb6deb67e3afea584a
Reviewed-on: https://review.typo3.org/58947
Tested-by: TYPO3com <>
Reviewed-by: Josef Glatz <>
Tested-by: Josef Glatz <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

History

#1 Updated by Josef Glatz 8 months ago

  • Related to Feature #86422: Allow usage of site in TypoScript gettext added

#2 Updated by Georg Ringer 8 months ago

it returns the configuration and not the site itself. however what I fix is throwing the exception

#3 Updated by Gerrit Code Review 8 months 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/58947

#4 Updated by Gerrit Code Review 8 months 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/58947

#5 Updated by Georg Ringer 8 months ago

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

#6 Updated by Josef Glatz 8 months ago

  • Precedes Bug #87036: TypoScript getText "site" can't retrieve nested properties added

#7 Updated by Josef Glatz 6 months ago

  • Related to Task #87434: site:identifier is not available added

#8 Updated by Benni Mack 2 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF