commerce:SEO URL v2.4.2

So, Optimierungsbedarf.

admin/includes/modules/cseo/commerce_seo_url.php, from line 105

$pQuery = xtc_db_query("SELECT products_id FROM " . TABLE_PRODUCTS_DESCRIPTION);


$pA;
$pHash;
//product ids in zwischen array speichern:
while ($pID = xtc_db_fetch_array($pQuery))
    $pA[] = $pID['products_id'];
//hashArray füllen:
for ($k = 0; $k <= sizeof($pA); $k++)
    $pHash[] = array('pid' => $pA[$k]);
$max_width = sizeof($pHash) - 1;
$step = $_GET['max'];
$_count = $_GET['count'];
$start = $_GET['start'];
$limit = $offset + $step;
for ($i = $offset; $i <= $limit; $i++) {
    if ($i >= $max_width) {
        // $commerceSeo->createSeoDBTable();
        $commerceSeo->createSeoDBTable();
        $infotext = urlencode(IMAGE_STEP_INFO . $_count . IMAGE_STEP_INFO_READY);
        // Cache leeren
        $del_array = array(array('templates_c/'),
        array(DIR_FS_CATALOG . 'cache/'),
        array('cache/'),
        array(DIR_FS_CATALOG . 'templates_c/'));


        foreach ($del_array AS $key => $value)
        $response .= $this->delfiles($value[0]);


        xtc_redirect(xtc_href_link(FILENAME_MODULE_SYSTEM, 'set=' . $_GET['set'] . '&module=commerce_seo_url&infotext=' . $infotext)); //FERTIG	
    }
    $commerceSeo->createSeoDBTableProduct($i, $pHash[$i]['pid']);
    //counter erhoehen um alle operationen zu zaehlen:
    $_count += 1;
}

1. Jeder Scriptaufruf holt alle product_id aus products_description. Danach wird pagination durchgeführt ("for ($i = $offset; $i <= $limit; $i++)"). Überlasse doch diese Aufgabe an MySql-Server:
http://dev.mysql.com/doc/refman/5.6…timization.html
"SELECT products_id FROM " . TABLE_PRODUCTS_DESCRIPTION . " ORDER BY products_id LIMIT $offset, $step"
Für ORDER BY products_id wird ein Index für "products_id"-Spalte benötigt.

2. Die Tabelle products_description enthält Lokalisierungen für alle Sprachen, somit kommen ins Abfrageergebnis alle products_id mal Anzahl Sprachen. Später in inc/commerce_seo.inc.php, line 599 werden sowieso alle lokalisierte Namen abgefragt.

$product_query = xtDBquery("SELECT
    products_id,
    language_id,
    products_name,
    products_url_alias,
    url_old_text
    FROM
    " . TABLE_PRODUCTS_DESCRIPTION . "
    WHERE products_id=" . $productID);

Es ergibt sich, dass jedes Produkt genau so viel mal abgearbeitet wird, wie die Anzahl Sprachen. Denkbar wäre noch DISCTINCT dazu:
"SELECT DISTINCT products_id FROM " . TABLE_PRODUCTS_DESCRIPTION . " ORDER BY products_id LIMIT $offset, $step"

3.

while ($pID = xtc_db_fetch_array($pQuery))
    $pA[] = $pID['products_id'];

OK. Das ist klar. Aber

for ($k = 0; $k <= sizeof($pA); $k++)
    $pHash[] = array('pid' => $pA[$k]);

ist Murks. $pHash[] wird nur einmal verwendet und zwar da: $commerceSeo->createSeoDBTableProduct($i, $pHash[$i]['pid']);
Was ist das? Ein Array befüllen, ins andere die Werte drüberschaufeln und das letzte nutzen. Könnte man nicht von Anfang an das erste nutzen? Speicherverschwendung, CPU-Zeitverschwendung und das Script tut so, als es was sehr wichtiges zu tun hätte.

4. admin/includes/modules/cseo/commerce_seo_url.php, from line 156

$selbstaufruf = '<script language="javascript" type="text/javascript">setTimeout("document.img_continue.submit()", 3000);</script>';

Worauf warten wir drei Sekunden?

5. Auf dem Frontend wird nur Zähler geändert. Um die Änderung davon anzuzeigen wird aber die komplette Seite übertragen. Ajax doch mal!

inc/commerce_seo.inc_orig.php, from line 590
createSeoDBTableProduct

6.

while ($productList = xtc_db_fetch_array($product_query)) {
    ...
    if ($useLanguageUrl) {
        $lang_query = xtc_db_fetch_array(xtDBquery("SELECT code FROM " . TABLE_LANGUAGES . " WHERE languages_id = '" . $productList['language_id'] . "'"));
    ...
}

Unnötige Abfrage wird pro Produkt pro Sprache durchgeführt. Die Sprachen bleiben während der SEO-Linkgenerierung konstant.

7.

$doppel_query = xtc_db_query("SELECT COUNT(products_name) AS counter  FROM " . TABLE_PRODUCTS_DESCRIPTION . " WHERE products_name='" . $productListname . "'");
$doppel_id = xtc_db_fetch_array($doppel_query);
if ($doppel_id['counter'] > 1) {

Wenn es auf Anzahl größer eins verglichen wird, wozu MySql dazu zwingen, alle Treffer aufzuzählen? Es reicht doch nur die erste zwei finden:

$doppel_query = xtc_db_query("SELECT products_id FROM " . TABLE_PRODUCTS_DESCRIPTION . " WHERE products_name='" . $productListname . "' LIMIT 2");
if (xtc_db_num_rows($doppel_query, false) > 1) {

8. line 364, getCategoryPathForProduct
Der größte Zeitverlust geschieht in dieser Methode. Sie braucht das komplette Reengineering.

Zahlen:
Anzahl Produkten: 1921013
Anzahl Sprachen: 2
SEO-Modul Einstellungen:

Sprachabhängige URLs
../de/.. | ../en/..
True

Kurze URLs?
False

Lange URL auf Kurze URL umleiten?
True

URL in Kleinbuchstaben?
True

SEO Suchwörter?
False

404 oder 410?
True

Doppelten Content vermeiden
True

Doppelte Namen vermeiden
True

Anzahl Produkten pro Durchlauf: 5000

Nach dem beseitigen des "Fatal error: Allowed memory size of 268435456 bytes exhausted (tried to allocate 72bytes)"-Fehlers (Ich musste Limit auf 4 GB hochsetzen), habe ich die Durchlauf-Ausführungszeit aufgenommen.
Mittelwert: 254.72 Sek.
Nach dem Reengineering anhand der Punkte 1 bis 8:
Mittelwert: 11.07 Sek mit 256 MB Memory Limit.