Hi,
ich denke ich hab jetzt verstanden was das Problem ist. Leeres Alias bedeutet im Frontend einen solchen Link:
Code:
<a href="" ...>Erste Seite</a>
Daher hab ich nun erst mal meine letzte Änderung rückgängig gemacht, so dass checkInitialPage() nun wieder so aussieht:
PHP-Code:
/**
* Hook function for generateFrontendUrl()
*/
public function checkInitialPage($arrPage, $strParams, $strUrl)
{
if (!isset($arrPage['initialPage']))
{
$objPage = $this->Database->prepare("SELECT * FROM tl_page WHERE id=?")->execute($arrPage['id']);
if ($objPage->numRows)
{
$arrPage['initialPage'] = $objPage->initialPage;
}
}
if ($arrPage['initialPage'] && !strlen($strParams))
{
return TL_PATH . '/';
}
return $strUrl;
}
Dafür mussten allerdings 2 andere Methoden unter's Messer.
1. Opfer war getPageOptions(), denn beim Versuch das 2. Opfer zu flicken ist mir aufgefallen, dass zunächst dieser Patient behandelt werden muss. Die Änderung sieht folgendermaßen aus:
Alt
Code:
/**
* Returns an array of all regular pages including their sitemap settings
*
* Based on Backend::getSearchablePages() from TYPOlight 2.6.5
*/
protected function getPageOptions($pid=0, $domain='', $blnHook=true)
{
$time = time();
// Get published pages
$objPages = $this->Database->prepare("SELECT * FROM tl_page WHERE pid=? AND (start='' OR start<?) AND (stop='' OR stop>?) AND published=1 ORDER BY sorting")
->execute($pid, $time, $time);
if ($objPages->numRows < 1)
{
return array();
}
// Fallback domain
if (!strlen($domain))
{
$domain = $this->Environment->base;
}
$arrPages = array();
// Recursively walk through all subpages
while($objPages->next())
{
// Set domain
if ($objPages->type == 'root')
{
if (strlen($objPages->dns))
{
$domain = ($this->Environment->ssl ? 'https://' : 'http://') . $objPages->dns . TL_PATH . '/';
}
else
{
$domain = $this->Environment->base;
}
}
// Add regular pages
elseif ($objPages->type == 'regular')
{
// Searchable and not protected
if (!$objPages->noSearch && (!$objPages->protected || $GLOBALS['TL_CONFIG']['indexProtected']))
{
// Published
if ($objPages->published && (!$objPages->start || $objPages->start < $time) && (!$objPages->stop || $objPages->stop > $time))
{
// Start page marker
if ($objPages->initialPage)
{
$arrPages[$domain] = $objPages->row();
}
else
{
$arrPages[$domain . $this->generateFrontendUrl($objPages->row())] = $objPages->row();
}
// Get articles with teaser
$objArticle = $this->Database->prepare("SELECT * FROM tl_article WHERE pid=? AND (start='' OR start<?) AND (stop='' OR stop>?) AND published=1 AND showTeaser=1 ORDER BY sorting")
->execute($objPages->id, $time, $time);
while ($objArticle->next())
{
$arrPages[$domain . $this->generateFrontendUrl($objPages->row(), '/articles/' . ((strlen($objArticle->alias) && !$GLOBALS['TL_CONFIG']['disableAlias']) ? $objArticle->alias : $objArticle->id))] = $objPages->row();
}
}
}
}
// Get subpages
if ((!$objPages->protected || $GLOBALS['TL_CONFIG']['indexProtected']))
{
$arrPages = array_merge($arrPages, $this->getPageOptions($objPages->id, $domain, false));
}
}
// HOOK: get additional page options
if ($blnHook && isset($GLOBALS['TL_HOOKS']['getSitemapOptions']) && is_array($GLOBALS['TL_HOOKS']['getSitemapOptions']))
{
foreach ($GLOBALS['TL_HOOKS']['getSitemapOptions'] as $callback)
{
$this->import($callback[0]);
$arrPages = $this->$callback[0]->$callback[1]($arrPages, $domain);
}
}
return $arrPages;
}
Neu
Code:
/**
* Returns an array of all regular pages including their sitemap settings
*
* Based on Backend::getSearchablePages() from TYPOlight 2.6.5
*/
protected function getPageOptions($pid=0, $domain='', $blnHook=true)
{
$time = time();
// Get published pages
$objPages = $this->Database->prepare("SELECT * FROM tl_page WHERE pid=? AND (start='' OR start<?) AND (stop='' OR stop>?) AND published=1 ORDER BY sorting")
->execute($pid, $time, $time);
if ($objPages->numRows < 1)
{
return array();
}
// Fallback domain
if (!strlen($domain))
{
$domain = $this->Environment->base;
}
$arrPages = array();
// Recursively walk through all subpages
while($objPages->next())
{
// Set domain
if ($objPages->type == 'root')
{
if (strlen($objPages->dns))
{
$domain = ($this->Environment->ssl ? 'https://' : 'http://') . $objPages->dns . TL_PATH . '/';
}
else
{
$domain = $this->Environment->base;
}
}
// Add regular pages
elseif ($objPages->type == 'regular')
{
// Searchable and not protected
if (!$objPages->noSearch && (!$objPages->protected || $GLOBALS['TL_CONFIG']['indexProtected']))
{
// Published
if ($objPages->published && (!$objPages->start || $objPages->start < $time) && (!$objPages->stop || $objPages->stop > $time))
{
// Start page marker
$arrPages[$domain . $this->generateFrontendUrl($objPages->row())] = $objPages->row();
// Get articles with teaser
$objArticle = $this->Database->prepare("SELECT * FROM tl_article WHERE pid=? AND (start='' OR start<?) AND (stop='' OR stop>?) AND published=1 AND showTeaser=1 ORDER BY sorting")
->execute($objPages->id, $time, $time);
while ($objArticle->next())
{
$arrPages[$domain . $this->generateFrontendUrl($objPages->row(), '/articles/' . ((strlen($objArticle->alias) && !$GLOBALS['TL_CONFIG']['disableAlias']) ? $objArticle->alias : $objArticle->id))] = $objPages->row();
}
}
}
}
// Get subpages
if ((!$objPages->protected || $GLOBALS['TL_CONFIG']['indexProtected']))
{
$arrPages = array_merge($arrPages, $this->getPageOptions($objPages->id, $domain, false));
}
}
// HOOK: get additional page options
if ($blnHook && isset($GLOBALS['TL_HOOKS']['getSitemapOptions']) && is_array($GLOBALS['TL_HOOKS']['getSitemapOptions']))
{
foreach ($GLOBALS['TL_HOOKS']['getSitemapOptions'] as $callback)
{
$this->import($callback[0]);
$arrPages = $this->$callback[0]->$callback[1]($arrPages, $domain);
}
}
return $arrPages;
}
Grund hierfür ist der, dass in der createSitemap()-Methode zunächst das $arrPages-Array mit der jeweiligen URL als Key aufgebaut wird mittels findSearchablePages(). Diese Methode benutzt aber bereits die BaseURL + generateFrontendUrl() inkl. vorhandene Hooks zur Generierung der URLs, so dass der Index für die "Erste Seite" hier also $arrPages['http://meineseite.de/subfolder//subfolder/']. Die Tatsache, dass hier auch schon der "Fehler" aus der Sitemap auftaucht, ist erst mal egal. Das Problem ist nun aber, dass durch den bisherigen Code in getPageOptions() der Index für das $arrPageOptions-Array nicht nach gleichem Muster generiert wurde, sondern für die erste Seite eben nur $domain als Key gesetzt wird, also folglich $arrPageOptions['http://meineseite.de/subfolder/'].
Das hat nun Auswirkungen auf createSitemap(), denn für die erste Seite wird also nie eiein Options-Array gefunden werden, denn
Code:
// Add pages
foreach ($arrPages as $strUrl)
{
// Options for this page, if available
$arrOptions = $arrPageOptions[$strUrl];
...
if (!is_array($arrOptions) || (!strlen($arrOptions['sitemap_lastmod']) && !strlen($arrOptions['sitemap_changefreq']) && !strlen($arrOptions['sitemap_priority'])))
{
$objFile->append(' <url><loc>' . $strUrl . '</loc></url>');
}
...
Da dies aber durch o.g. Fix behoben ist, können wir nun folgendes in createSitemap() ändern, damit die <loc> für die erste Seite richtig ins XML-File geschrieben wird:
Code:
...
if (!is_array($arrOptions) || (!strlen($arrOptions['sitemap_lastmod']) && !strlen($arrOptions['sitemap_changefreq']) && !strlen($arrOptions['sitemap_priority'])))
{
$objFile->append(' <url><loc>' . $strUrl . '</loc></url>');
}
else
{
if ($arrOptions['sitemap_ignore'])
continue;
if ($arrOptions['initialPage'])
{
$objFile->append(" <url>\n <loc>" . $strDomain . "</loc>");
}
else
{
$objFile->append(" <url>\n <loc>" . $strUrl . "</loc>");
}
...
Damit hab ich nun die gleiche Funktionalität hergestellt wie vorher, allerdings sieht der Frontend-Link nun so aus:
Code:
<a href="/subfolder/" ...>Erste Seite</a>
Wäre schön wenn du das mal testen/nachvollziehen könntest, aber ich glaube es müsste nun passen oder? 
Grüße,
Fabio
Zitieren
Zitat von andreas.schempp
Lesezeichen