Bug #54410: page.config.pageTitleSeparator doesn't work - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #54410

closed

page.config.pageTitleSeparator doesn't work

Added by Bill Dagou almost 11 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

Target version:

Start date:

2013-12-14

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

6.1

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

page = PAGE
page {
config {
pageTitleSeparator = -
}
}

doesn't work. Seems only

config {
pageTitleSeparator = -
}

is working.

Actions

Copy link

Updated by Chris topher almost 11 years ago

Status changed from New to Needs Feedback

config is a top level object, but it should also be available under page.config and the one under page.config should overwrite what is set under config.

Is pageTitleSeparator the only property, for which this is not working or are there also others?

Actions

Copy link

Updated by Bill Dagou almost 11 years ago

Hi Chris,

Not sure. But I checked the core code, TYPO3\CMS\Core\TypoScript\TemplateService::printTitle(), $this->setup['config.']['pageTitleSeparator'] is not existed. So, I think maybe $this->setup['config.'] is not merged with page.config correctly or something. As I'm using page=PAGE, the pageTitleSeparator property is in $this->setup['page.']['config.'].

BTW, I'm using templavoila, so not sure whether it's something with it.

Cheers,
Bill

Actions

Copy link

Updated by Bill Dagou almost 11 years ago

And, seems $GLOBALS['TSFE']->altPageTitle doesn't work any more in extensions.

Actions

Copy link

Updated by Markus Klein almost 11 years ago

Bill please open a separate issue for the altPageTitle thing.

Actions

Copy link

Updated by Markus Klein almost 11 years ago

Status changed from Needs Feedback to Accepted
Priority changed from Should have to Must have

Can confirm this for 6.1. It is independent of templavoila

Actions

Copy link

Updated by Gerrit Code Review almost 10 years ago

Status changed from Accepted 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 http://review.typo3.org/35750

Actions

Copy link

Updated by Gerrit Code Review almost 10 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35750

Actions

Copy link

Updated by Benni Mack almost 10 years ago

How to test:

Add the following TypoScript to your page

config.pageTitleSeparator = common separator

page.config.pageTitleSeparator = type-specific
page.config.pageTitleSeparator.noTrimWrap = | separator|

Actions

Copy link

Updated by Gerrit Code Review almost 10 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35750

Actions

Copy link

#10

Updated by Gerrit Code Review almost 10 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35903

Actions

Copy link

#11