SQL_CACHEDIR hardcodiert über "application_top.php"

[Torsten Riemer]

Mir ist gerade durch Zufall aufgefallen, dass "SQL_CACHEDIR" hardcodiert in den "application_top.php" Dateien steht.

// SQL caching dir
define('SQL_CACHEDIR', DIR_FS_CATALOG.'cache/');

Würde es hier nicht Sinn machen, dass wir entweder folgenden Code benutzen:

// SQL caching dir
define('SQL_CACHEDIR', DIR_FS_CACHE);

Oder wir einen weiteren Wert in die Tabelle "configuration" einfügen, beispielsweise "DIR_FS_SQLCACHE", so dass man unter "Erweiterte Konfiguration" -> "Cache Optionen" auch für den "DB Cache" einen Ordner definieren kann?

[Gerhard Waldemair]

das Verzeichnis wird in der paths.php festgelegt. Eine zusätzliche Einstellung im Admin finde ich nicht sinnvoll, denn hier muss man den absoluten Pfad angeben.

[anonymous]

Was du, GTB, sagst widerspricht der Tatsache, daß auch das SESSION_WRITE_DIRECTORY für den Pfad wo bei file-basiertem Session-Handling die Sessions gespeichert werden im Backend einstellbar ist.

Dazu gibt es dann auch noch eine Unlogik oder einen Bug, wenn man so will.
Denn in /admin/configuration.php wird hard-codiert

case 'delcache':
    $path = DIR_FS_CATALOG.'cache/';

wo doch hier die Konstante SESSION_WRITE_DIRECTORY stehen müsste für $path.

Gruß,
noRiddle

[anonymous]

NACHTRAG:

Okay, sehe gerade, daß ja der DB-Cache und nicht der Pfad für die Sessions gemeint ist mit $path.
Dann jedoch ist der Code aus der login_admin.php nicht korrekt

// reset session write directory
case 'sess_write':
    xtc_db_query('
            UPDATE configuration
            SET    configuration_value = "'.DIR_FS_CATALOG.'cache"
            WHERE  configuration_key   = "SESSION_WRITE_DIRECTORY"
          ');
    die('Report: SESSION_WRITE_DIRECTORY wurde auf das Cache-Verzeichnis zurückgesetzt.');
    break;

denn per Default ist das SESSION_WRITE_DIRECTORY auf /temp gesetzt.

Gruß,
noRiddle

[Torsten Riemer]

Also dann öffne ich das Ticket nochmal...

Ich verstehe es auch nicht, dass wir teilweise benutzen:

"/admin/configuration.php":

DIR_FS_CATALOG.'cache/'

bzw.:

"/inc/xtc_read_cache.inc.php" / "/inc/xtc_write_cache.inc.php":

DIR_FS_CACHE . $filename

Und dann aber wieder:

"/includes/paths.php":

define('SQL_CACHEDIR', DIR_FS_CATALOG . 'cache/');

Sofern die Verbindung zur Datenbank früh genug steht sollte man daraus machen:

define('SQL_CACHEDIR', DIR_FS_CACHE);

[anonymous]

Was ist denn /includes/paths.php, bist du da schon in der 2.0 ?
Jedenfalls besteht ein heilloses Durcheinander was Cache-Verzeichnisse und das Directory für die file-basierte Session-Speicherung.

Gruß,
noRiddle

[Torsten Riemer]

Ja, bei der Datei "/includes/paths.php" bin ich bei Shopversion 2.00.

Ich gebe dir Recht, dass das überarbeitet gehört!