Bug #36616

TYPO3 4.7 sectionIndex not working correctly on multilanguage site

Added by Helmuth Antholzer about 1 year ago. Updated 4 months ago.

Status:Resolved Start date:2012-04-26
Priority:Must have Due date:
Assignee:Stefan Galinski % Done:

100%
Category:-
Target version:-
TYPO3 Version:4.7 Complexity:
PHP Version:
Votes: 1 (View)

Description

I have set up a site with two languages. I added a content element sitemap (section index). The sectionIndex does not respect sys_language_uid and and all elements are shown, also the ones from alternative language.

Changing the following in function sectionIndex in file class.tslib_menu.php should resolve the problem.

From

if (!$pid) {
  $pid = $this->id;
  if ($GLOBALS['TSFE']->sys_language_uid && count($this->sys_page->getPageOverlay($pid))) {
    $where .= ' AND sys_language_uid=' . intval($GLOBALS['TSFE']->sys_language_uid);
  } else {
    $where .= ' AND sys_language_uid=0';
  }
}

to

if (!$pid) {
  $pid = $this->id;
}                
if ($GLOBALS['TSFE']->sys_language_uid && count($this->sys_page->getPageOverlay($pid))) {
  $where .= ' AND sys_language_uid=' . intval($GLOBALS['TSFE']->sys_language_uid);
} else {
  $where .= ' AND sys_language_uid=0';
}

Related issues

related to Core - Bug #37449: Section Index incorrect output Closed 2012-05-23
related to Core - Bug #17354: fallback for menus. "content_fallback;1,0" has no affect ... Under Review 2007-07-22

Associated revisions

Revision 34545f88
Added by Stefan Galinski 10 months ago

[BUGFIX] sectionIndex menu is not i18n ready

Change-Id: I34281d801f292530a1618d8b38eafac22b02b511
Fixes: #36616
Releases: 6.0, 4.7
Reviewed-on: http://review.typo3.org/11249
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Markus Klein
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Revision ae9d18c9
Added by Stefan Galinski 10 months ago

[BUGFIX] sectionIndex menu is not i18n ready

Change-Id: I34281d801f292530a1618d8b38eafac22b02b511
Fixes: #36616
Releases: 6.0, 4.7
Reviewed-on: http://review.typo3.org/13506
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Revision a9449a04
Added by Stefan Galinski 10 months ago

[BUGFIX] sectionIndex menu is not i18n ready

Change-Id: I34281d801f292530a1618d8b38eafac22b02b511
Fixes: #36616
Releases: 6.0, 4.7
Reviewed-on: http://review.typo3.org/11249
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Markus Klein
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Revision 0e85336c
Added by Stefan Galinski 9 months ago

[BUGFIX] sectionIndex menu is not i18n ready

Change-Id: I34281d801f292530a1618d8b38eafac22b02b511
Fixes: #36616
Releases: 6.0, 4.7
Reviewed-on: http://review.typo3.org/11249
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Markus Klein
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1
Updated by Helmuth Antholzer about 1 year ago

  • Project changed from Base Package (aka typo3_src) to Core

#2
Updated by Stefan Galinski about 1 year ago

  • Status changed from New to Accepted
  • TYPO3 Version set to 4.5

#3
Updated by Stefan Galinski about 1 year ago

  • Assignee set to Stefan Galinski
  • TYPO3 Version changed from 4.5 to 4.7

#4
Updated by Gerrit Code Review about 1 year ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#5
Updated by Gerrit Code Review about 1 year ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#6
Updated by Gerrit Code Review about 1 year ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#7
Updated by Gerrit Code Review 12 months ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#8
Updated by Gerrit Code Review 10 months ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#9
Updated by Gerrit Code Review 10 months ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#10
Updated by Gerrit Code Review 10 months ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#11
Updated by Gerrit Code Review 10 months ago

Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#12
Updated by Gerrit Code Review 10 months ago

Patch set 9 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#13
Updated by Gerrit Code Review 10 months ago

Patch set 10 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11249

#14
Updated by Gerrit Code Review 10 months ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/13506

#15
Updated by Stefan Galinski 10 months ago

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

#16
Updated by Kurt Kunig 4 months ago

The above mentioned fix is no longer in the function "sectionIndex" of TYPO3 V4.7.7 but:

in line 1648 you'll find this:

        $selectSetup = array(
            'pidInList' => $pid,
            'orderBy' => $altSortField,
            'languageField' => 'sys_language_uid',
            'where' => 'colPos=0'
        );

so change 

     'where' => 'colPos=0'

to
     'where' => ''

Also available in: Atom PDF