Fehler in google_sitemap.php

[noRiddle (revilonetz)]

Hallo modified-Team, hallo Community.
Ein paar Bemerkungen zu /google_sitemap.php:
Vorab sei bemerkt, daß ich davon ausgehe, daß die Datei für die kommende Version 2.0 ohnehin überarbeitet wurde.

• für das Thema ging es mir erstmal nur um das folgende:
  Oben in der Datei gibt es diese Zeile um den Zugriff nur Admins zu gewähren:

   

Code: PHP  [Auswählen]

if (!isset($_SESSION['customer_id']) || ($_SESSION['customers_status']['customers_status_id'] != '0' && $_SESSION['customers_status']['customers_status'] != '0')) {
    xtc_redirect(xtc_href_link(FILENAME_LOGIN, '', 'NONSSL'));
}

    Eine $_SESSION['customers_status']['customers_status'] gibt es nicht.
    Die Zeile sollte nach meiner Überzeugung also so aussehen:
   

Code: PHP  [Auswählen]

if (!isset($_SESSION['customer_id']) || (isset($_SESSION['customer_id']) && $_SESSION['customers_status']['customers_status_id'] != '0')) {
    xtc_redirect(xtc_href_link(FILENAME_LOGIN, '', 'NONSSL'));
}

• Die Datei hat bereits eine Implementation um die Sitemap in mehrere zu splitten wenn zuviele Daten da sind (Konstanten: MAX_ENTRYS, MAX_SIZE).
  Das funktioniert aber nicht. U.A. deshalb weil in der Funktion output_entry() vergessen wurde die Variable $i global zu setzen.
• Wie in anderen Threads bereits schonmal besprochen wurde, kann/muß man die Queries optimieren weil sie viele Daten holen die zur Bildung der Sitemap gar nicht benötigt werden.
• Da es bei hohem Artikelstamm auch schnell mal zu Server-Timeouts oder Memory-Limit-Überschreitungen kommen kann wäre ein automatiche Browser-Reload nach erstellung einer Sitemap (bei gesplitteten Sitemaps sinnvoll. Auch möglich wäre eine Generierung via AJAX.

Ein Feedback würde mich freuen.

Gruß,
noRiddle

Linkback: https://www.modified-shop.org/forum/index.php?topic=33379.0

[Alfred]

Hallo,

wenn das auf den Radar kommt kann man die Ausgabe auch ändern.
Es wird nur die URL und das Änderungsdatum inkl. Uhrzeit benötigt.

Gruß

[noRiddle (revilonetz)]

Du meinst changefreq ist unnötig ?

Gruß,
noRiddle

[Alfred]

ja

[Modulfux]

changefreg ist aber genauso optional wie lastmod.

Gruß
Ronny

[Alfred]

Hallo,

es geht nicht um optional sondern um nützlich im Sinne der Suchmaschine.
Auch das ändert sich ja. Die Bedeutung einer xml-Sitemap sinkt.
Man könnte ja mal nur noch geänderte Seiten in die Sitemap packen.

Gruß

[noRiddle (revilonetz)]

Off topic:
Alfred, wärst du so nett dich mal bei mir zu melden ? (z.B. über das kleine Häuschen links)

Merci,
Gruß,
noRiddle