Ich wollte mich hier schon ganz zu beginn mit meinen ersten Gedanken zu Wort melden, habe dann aber gemerkt dass Leo einiges mehr bedacht hatte, als ich im ersten Moment 
Nun habe ich trotzdem/noch folgende Anmerkungen/Fragen zu Rev. 526:
1: Frontend.php, Zeile 132 sollte doch "OR fallback=1" auch gefragt werden, wenn der Browser keine Sprachen übergibt? Das erübrigt sich aber mit Punkt 3.
2: Das parsen der Sprachen könnte doch mit Regex enorm vereinfacht werden? Zumindest array_values ist nicht nötig, weil array_slice die Keys sowieso neu generiert.
PHP-Code:
protected function httpAcceptLanguage()
{
preg_match_all('/([a-z]{2})[^,]*/', strtolower($_SERVER['HTTP_ACCEPT_LANGUAGE']), $matches);
return array_slice(array_unique($matches[1]), 0, 8);
}
3: Danach habe ich mir noch die getRootIdFromUrl angeschaut. Ich hatte bereits mal das Problem dass ich die Sortierung eines WHERE ... IN realisieren musste. Hier meine Variante der Funktion 
PHP-Code:
protected function getRootIdFromUrl()
{
$host = $this->Environment->host;
$accept_language = $this->Environment->httpAcceptLanguage;
$time = time();
$objRootPage = $this->Database->prepare("SELECT id, dns, language, fallback FROM tl_page WHERE type='root' AND (dns=? OR dns=? OR dns='')" . ((count($accept_language) > 0) ? " AND (language IN('". implode("','", $accept_language) ."') OR fallback=1)" : " AND fallback=1") . (!BE_USER_LOGGED_IN ? " AND (start='' OR start<$time) AND (stop='' OR stop>$time) AND published=1" : "") . " ORDER BY dns DESC" . ((count($accept_language) > 0) ? ", language='". implode("' DESC, language='", $accept_language) ."' DESC" : ""))
->limit(1)
->execute($host, 'www.'.$host);
return intval($objRootPage->id);
}
Als Anmerkung: Ja, ich habe alles implementiert und getestet... Bei Punk 3 habe ich alle 4 Varianten geprüft und es hat immer funktioniert.
Zitat von FloB
Zitieren
.
Wenn wir testen sollen, was sehr gut ist, dann müssen wir schon verstehen was. 
(und auf die 1-2µs kommt es nun auch nicht an, die man evtl. gewinnen könnte).
). Die Anzahl der Festplattenzugriffe habe ich trotzdem reduziert.
CONTAO_THEME is not defined
Lesezeichen