Project

General

Profile

Actions

Bug #63047

closed

TreeView with non pages isInWebMount wrong parameter because of Bugfix #18797

Added by Andreas Allacher over 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
-
Target version:
Start date:
2014-11-18
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Stabilization Sprint

Description

Hi,

because of the bug fix für #18797 which results in this commit:
https://git.typo3.org/Packages/TYPO3.CMS.git/commitdiff/c59d89f809898784aaedd507db61a4d380bc27a8

This part is only correct IF the records are pages, will result in problems if the records are not pages.

757                         if (!$GLOBALS['BE_USER']->isInWebMount($row['uid'])) {
758                                 // Current record is not within web mount => skip it
759                                 continue;
760                         }

Which means we have to modify it to something like this:

class ... {
        ....
    public $fieldArray = array('uid', 'pid', 'title');
        ....
}

getTree Method:
$pageUid = ($this->table === 'pages') ?> $row['uid'] : $row['pid'];
      757                         if (!$GLOBALS['BE_USER']->isInWebMount(pageUid)) {
758                                 // Current record is not within web mount => skip it
759                                 continue;
760                         }


Files

patch-63047.diff (675 Bytes) patch-63047.diff Franz Holzinger, 2014-11-19 19:03

Related issues 1 (0 open1 closed)

Blocked by TYPO3 Core - Bug #18797: "New page" wizard might disclose existence of pages outside DB mountClosedNicole Cordes2008-05-15

Actions
Actions #1

Updated by Franz Holzinger over 9 years ago

This bug is also with TYPO3 6.1.

Actions #2

Updated by Gerrit Code Review about 9 years ago

  • Status changed from New to Under Review

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/37034

Actions #3

Updated by Gerrit Code Review about 9 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/37034

Actions #4

Updated by Gerrit Code Review about 9 years ago

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

Actions #5

Updated by Gerrit Code Review about 9 years ago

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

Actions #6

Updated by Nicole Cordes about 9 years ago

  • Target version set to 7.2 (Frontend)
  • Sprint Focus set to Stabilization Sprint
Actions #7

Updated by Gerrit Code Review about 9 years ago

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

Actions #8

Updated by Gerrit Code Review about 9 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/38848

Actions #9

Updated by Andreas Allacher about 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 10 to 100
Actions #10

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF