Reihenfolge: Laden der localconfig vs. Laden der config.php der einzelnen Module

[oetzi]

Hi,

mir ist aufgefallen, dass sich die Reihenfolge des Ladens der localconfig.php und der config.phps der einzelnen Module in Contao 3.0 geändert hat.

Contao 2.11.x

include(TL_ROOT . '/system/config/config.php');

if (file_exists(TL_ROOT . '/system/config/localconfig.php'))
{
    $this->blnHasLcf = true;
    include(TL_ROOT . '/system/config/localconfig.php');
}

// Load the agent file
include(TL_ROOT . '/system/config/agents.php');
// Get the module configuration files
foreach ($this->getActiveModules() as $strModule)
{
    $strFile = TL_ROOT . '/system/modules/' . $strModule . '/config/config.php';
    if (file_exists($strFile))
    {
        include($strFile);
    }
} 

Contao 3.0

// Load the default files
include TL_ROOT . '/system/config/default.php';
include TL_ROOT . '/system/config/agents.php';

// Get the module configuration files
foreach ($this->getActiveModules() as $strModule)
{
    $strFile = TL_ROOT . '/system/modules/' . $strModule . '/config/config.php';

    if (file_exists($strFile))
    {
        include $strFile;
    }
}

// Return if there is no local configuration file yet
if (!file_exists(TL_ROOT . '/system/config/localconfig.php'))
{
    return;
}

$this->blnHasLcf = true;
include TL_ROOT . '/system/config/localconfig.php'; 

Gibt es dafür einen Grund? Problem ist, dass man nun, wenn man in der config.php auf die Werte des TL_CONFIG-Arrays zugreift nur die Standardwerte aus der default.php und der agents.php hat, nicht aber auf die Werte der localconfig.php.

lg
oetzi

[tril]

Ich glaube der Hintergrund war, dass auf diese Weise die localconfig.php nur 1 mal geladen wird. In dem Fall muss man sich denke ich einfach als Entwickler anpassen ^^

[andreas.schempp]

Ja das war der Grund. Ich sehe allerdings auch den grossen Nachteil... ich frag mal im GitHub Ticket.

[oetzi]

Hi,

schon mal danke für das Feedback. Auch wenn ich es nicht ganz blicke. Vielleicht steh ich da aber auch einfach auf dem Schlauch.

Es hat sich doch nur die Reihenfolge geändert, oder sehe ich das falsch?

vorher:

1. einmaliges Laden der localconfig.php
2. je aktivem Modul die config.php laden

nachher:

1. je aktivem Modul die config.php laden
2. einmaliges Laden der localconfig.php

Ich befürchte jetzt nämlich, dass der (evtl. faule) Modulentwickler in seiner config.php einfach folgendes reinschreibt:

if (file_exists(TL_ROOT . '/system/config/localconfig.php')) 
{ 
    include(TL_ROOT . '/system/config/localconfig.php'); 
} 

Und dann würde sie wirklich mehrfach geladen werden.

lg
oetzi

[andreas.schempp]

Nein bisher wurde die Localconfig zweimal geladen. Vor und nach den config.php files.

[oetzi]

ah ja, wer weiter liest ist klar im Vorteil.

Mmh, gab es dafür einen Grund, warum die Datei unter 2.11.x einmal vorher und einmal nachher eingelesen wurde?

[andreas.schempp]

Genau deshalb. Damit die config-Daten zu Verfügung stehen, aber sicherlich als letztes nochmals gelten. Ich hab das Ticket leider noch nicht gefunden

[oetzi]

nur keinen Stress. So eilig ist es dann doch nicht.