Project

General

Profile

Actions

Bug #24260

closed

Broken sitemaps

Added by Georg Ringer almost 14 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Must have
Category:
-
Target version:
-
Start date:
2010-12-01
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

It seems there is something broken in using sitemaps.

Testcase:
TS:
---------------------------------------
page = PAGE
page.10 < styles.content.get
---------------------------------------

Demo tree:
Root page
_ Child 1
___ Child 1b
__Child 2

Creating a sitemap with mode "menu of subpages" with page "root page" as "selected pages" works fine

but:
- Having a sitemap with the same settings on page "child 1" shows a menu of subpages of child 1 => "child 1b". but of course it should be a menu of child 1 & child 2!!

hope the t3d works
(issue imported from #M16632)


Files

T3D__01-12-13-59-22-z.t3d (2.9 KB) T3D__01-12-13-59-22-z.t3d Administrator Admin, 2010-12-01 13:58
16632-menu.patch (902 Bytes) 16632-menu.patch Administrator Admin, 2010-12-01 14:25

Related issues 1 (0 open1 closed)

Is duplicate of TYPO3 Core - Bug #24232: tslib_menu and tslib_Xmenu calls stdwrap on $GLOBALS['TSFE']->cObj instead of this->parent_cObjClosedErnesto Baschny2010-11-29

Actions
Actions #1

Updated by Ernesto Baschny almost 14 years ago

Instead of:

- $value = isset($this->conf['special.']['value.'])
- ? $GLOBALS['TSFE']->cObj->stdWrap($this->conf['special.']['value'], $this->conf['special.']['value.'])
- : $this->conf['special.']['value'];
+ $value = $this->parent_cObj->stdWrap($this->conf['special.']['value'], $this->conf['special.']['value.']);

What speaks against keeping the "speedup" condition but just using the corret cObject to do the work?

$value = isset($this->conf['special.']['value.'])
? $this->parent_cObj->stdWrap($this->conf['special.']['value'], $this->conf['special.']['value.'])
: $this->conf['special.']['value'];

Could you test it please?

Actions #2

Updated by Georg Ringer almost 14 years ago

seems to work

Actions #3

Updated by Ernesto Baschny almost 14 years ago

In #24232 this case is also included and several others. Please test, give your +1 in the core list and it can go in! Thanks!

Actions #4

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF