Bug #93251
Updated by Sven Wappler almost 4 years ago
The \TYPO3\CMS\Extbase\Configuration\BackendConfigurationManager getCurrentPageId() getCurrentPageId function may not work correctly in MultiSite systems because the functions it contains do not work in a MultiSite environment. * *$this->getCurrentPageIdFromGetPostData()* Often the id GET/POST parameter is missing, for example at editing a content record. A separate pid parameter could be introduced because the id parameter is too universal. Or even better, to prevent manipulation: When retrieving the record data, the pid would have to be written to an environment variable. * *$this->getCurrentPageIdFromCurrentSiteRoot()* This function cannot work correctly because it chooses a random root page. This can lead to TypoScript values being read from another site. Possibly a security issue. To be able to choose the correct root page, it would also have to know the current page ID. And then it would be superfluous, because the page ID would be already known. * *$this->getCurrentPageIdFromRootTemplate()* The same as getCurrentPageIdFromCurrentSiteRoot(). Cannot work correctly. * *constant self::DEFAULT_BACKEND_STORAGE_PID* Is only read, but never set.