XML sitemap.org

[lithoverlag]

Kurze nachfrage ob mir jemand sagen kann ob ich spinne oder das System oder Tante G.

eine mit dem oben genannte Modul erstellte Sitemap habe ich bei google webmastertools einlesen lassen.
G. sagt. Fehlerdetails 16 Fehler.

zeigt aber nur einen Fehler Ungültige URL
Dies ist keine gültige URL. Korrigieren Sie die URL und reichen Sie die Daten erneut ein.

URL: #
Übergeordnetes Tag: url
Tag: loc

in Zeile 143,157,171

den hat g. bei der Problemanzahl 16x.

im XML- File kann ich nichts fehlerhaftes dazu entdecken. www.billiardbook.com/shop/sitemap.xml

Liegt dies an meinen 5 Sprachversionen im shop? Dann müsste aber die Fehler zahl höher sein oder?

Mit gerade etwas ratlos.

Danke vorab Gruß
litho

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

[karsta.de]

In denen von dir genannten Zeilen fehlen die Links zu den Produkten. Da steht jetzt:
KURZ:

Code: XML  [Auswählen]

<loc>#</loc>

LANG:

Code: XML  [Auswählen]

        <url>
                <loc>#</loc>
                <lastmod>2017-05-31T14:56:47+02:00</lastmod>
                <image:image>
                        <image:loc>https://www.billiardbook.com/shop/images/product_images/popup_images/billiardbook-com-Basics-of-Pool-Billiards.jpg</image:loc>
                        <image:title><![CDATA[]]></image:title>
                        <image:caption><![CDATA[]]></image:caption>
                </image:image>
                <image:image>
                        <image:loc>https://www.billiardbook.com/shop/images/product_images/popup_images/billiardbook-com-Basics-of-Pool-Billiards-backsite-1.jpg</image:loc>
                        <image:title><![CDATA[]]></image:title>
                        <image:caption><![CDATA[]]></image:caption>
                </image:image>
        </url>
        <url>
                <loc>#</loc>
                <lastmod>2017-05-31T12:03:30+02:00</lastmod>
                <image:image>
                        <image:loc>https://www.billiardbook.com/shop/images/product_images/popup_images/billiardbook-com-Billiards-Manual-Three-Cushion.jpg</image:loc>
                        <image:title><![CDATA[]]></image:title>
                        <image:caption><![CDATA[]]></image:caption>
                </image:image>
                <image:image>
                        <image:loc>https://www.billiardbook.com/shop/images/product_images/popup_images/billiardbook-com-Billiards-Manual-Three-Cushion-backsite-1.jpg</image:loc>
                        <image:title><![CDATA[]]></image:title>
                        <image:caption><![CDATA[]]></image:caption>
                </image:image>
        </url>
        <url>
                <loc>#</loc>
                <lastmod>2017-05-30T21:05:09+02:00</lastmod>
                <image:image>
                        <image:loc>https://www.billiardbook.com/shop/images/product_images/popup_images/billiardbook-com-Big-Points-in-Three-Cushion.jpg</image:loc>
                        <image:title><![CDATA[]]></image:title>
                        <image:caption><![CDATA[]]></image:caption>
                </image:image>
                <image:image>
                        <image:loc>https://www.billiardbook.com/shop/images/product_images/popup_images/billiardbook-com-Big-Points-in-Three-Cushion-backsite-1.jpg</image:loc>
                        <image:title><![CDATA[]]></image:title>
                        <image:caption><![CDATA[]]></image:caption>
                </image:image>
        </url>

Außerdem fehlen <image:title> und <image:caption>. Ich gehe mal davon aus, dass die Artikel noch gar nicht online zu sehen sind und noch namenlos sind.

BG kgd

[lithoverlag]

Den Artikel gibt es allerdings nur für die englische und niederländische Sprachversion der Seite.
https://www.billiardbook.com/shop/Books/Poolbilliard/The-Basics-of-Pool-Billiards::3.html?language=nl

Also nicht auf deutsch.

[karsta.de]

Dann müßtest du in der admin\includes\modules\system\sitemaporg.php die function process_products() erweitern, dass die Produkte nur für die Sprachen angezeigt werden, für die sie aktiviert wurden.

BG kgd

[lithoverlag]

ok da war es wieder mein Problem mit dem programmieren....

[karsta.de]

Ich weiß ja nicht welche sitemaporg.php du verwendest, denn im Testshop wird immer nur eine Sitemap erstellt, in der Sprache in der der Admin sich gerade befindet. Also gehe ich davon aus, dass du ein verändertes Modul verwendest. Du könntest natürlich auch für jede Sprache eine eigene Sitemap erstellen und diese bei Google hochladen. (siehe Version 2.0 - Modul Google Sitemap)

[EDIT]
Habe gerade bei einem Test gesehen, wenn man den Artikel nicht mit Namen in z.B. Deutsch versieht, dann wird er allerdings trotzdem angezeigt und zwar ohne Namen mit der ID des Artikels <loc>http://www.xxxx.de/Kategorie/--2.html</loc>

BG kgd

[web28]

Welche Shopversion ist das?

Gruss Web28

[karsta.de]

Wenn du mich meinst, Version-2.0.2.2. Ich habe das export_query der function process_products() erweitert mit AND pd.products_name != '' .
So werden jetzt in der jeweiligen Sprache nur die Produkte gelistet, die in der jeweiligen Sprache ein Namen haben. Hier nochmal das gesamte export_query:

Code: PHP  [Auswählen]

    $export_query =xtc_db_query("SELECT p.products_id,
                                        p.products_last_modified,
                                        p.products_date_added,
                                        p.products_image,
                                        pd.products_name
                                   FROM " . TABLE_PRODUCTS . " p,
                                        " . TABLE_PRODUCTS_DESCRIPTION . " pd
                                  WHERE p.products_status = 1
                                    AND p.products_id = pd.products_id
                                    AND pd.language_id = ".(int)$_SESSION['languages_id']."
                                    AND pd.products_name != ''
                               ORDER BY p.products_id");

BG kgd

[lithoverlag]

OK im lesen zwischen den Zeilen  habe ich für mich wieder ein Geheimnis entnommen:
Sitemaps müssen für die Sprachversionen separat erstellt werden. Bisher ging ich davon aus, dass alle Sprachversionen in einer XML exportiert werden. (habe es aber auch nicht kontrolliert).

Da ich auch versucht hatte das https://www.modified-shop.org/forum/index.php?topic=34183.90 Optimierte Sitemap Bildern von Bonsai zu installieren muss ich diese Version wohl erstmal rückgängig machen und den Core Urzustand herstellen.

Meine Idee zu diesem export Modul wäre, dass wenn es aktiv ist es automatisch ausgelöst wird sobald ein Artikel oder Attribut oder Kategorie bearbeitet würden. Alternativ werde ich checken ob ich einen Cronjob auf die sitemaporp.php legen kann. Dann gibt es halt täglich eine oder zwei neue Sitemaps.

[p3e]

OK im lesen zwischen den Zeilen  habe ich für mich wieder ein Geheimnis entnommen:
Sitemaps müssen für die Sprachversionen separat erstellt werden. Bisher ging ich davon aus, dass alle Sprachversionen in einer XML exportiert werden. (habe es aber auch nicht kontrolliert).

Nein, das ist kein muss sondern ein kann. Es spricht nichts dagegen alle Sprachversionen in einer XML zu haben. Google kommt damit gut klar.
Gerade für mehrsprachige Shops ist die Version von Bonsai sehr gut, da hier jeweils auf die Alternativen in den anderen Sprache verwiesen wird:

Code: PHP  [Auswählen]

  <url>
    <loc>https://www.shop.com/en/categori/product1::7.html</loc>
    <lastmod>2017-05-03</lastmod>
    <xhtml:link rel="alternate" hreflang="de" href="https://www.shop.com/kategorie/produkt1::7.html"/>
    <xhtml:link rel="alternate" hreflang="en" href="https://www.shop.com/en/categori/product1::7.html"/>
    <image:image>
      <image:loc>https://www.shop.com/images/product_images/popup_images/produkt1.jpg</image:loc>
      <image:caption>produkt</image:caption>
      <image:title>produkt1</image:title>
    </image:image>
  </url>

Ich habe Bonsais Version noch um die Cache Funktion erweitert, so dass die sitemap maximal einmal pro Tag neu berechnet wird.

[lithoverlag]

Bei der Version habe ich das Problem sie meldet nur HTML 500 Fehler weil der Shop auf PHP7 läuft.

[lithoverlag]

@kgd

Wenn du mich meinst, Version-2.0.2.2. Ich habe das export_query der function process_products() erweitert mit AND pd.products_name != '' .
So werden jetzt in der jeweiligen Sprache nur die Produkte gelistet, die in der jeweiligen Sprache ein Namen haben. Hier nochmal das gesamte export_query:

in welche Datei muss das rein?

Nachtrag: ERLEDIGT  steht ja ganz oben hab's gefunden.

@p3e

    <xhtml:link rel="alternate" hreflang="de" href="https://www.shop.com/kategorie/produkt1::7.html"/>
    <xhtml:link rel="alternate" hreflang="en" href="https://www.shop.com/en/categori/product1::7.html"/>

und bei weiteren Sprachen entsprechend zu kopieren... oder kann man daraus auch eine Schleife machen die alle Sprachversionen abarbeitet?

p.s. kannst du dass an den dazugehörigen Thread noch dranhängen, weil gehört hier ja nicht dazu... oder? Danke!

[lithoverlag]

Ich habe das export_query der function process_products() erweitert mit AND pd.products_name != '' .
So werden jetzt in der jeweiligen Sprache nur die Produkte gelistet, die in der jeweiligen Sprache ein Namen haben. Hier nochmal das gesamte export_query:

So nach Einbau des Query hat sich die Fehlerzahl deutlich gesenkt auf 1 bis 13 Fehler je nach Sprachversion.

   
1 /sitemap_de.xml Sitemap 14. Juli 2017
1 Fehler
Web 72 Ausstehend
Bilder 47 Ausstehend

2 /sitemap_ee.xml Sitemap 14. Juli 2017
7 Fehler
Web 27 Ausstehend
Bilder 6 Ausstehend

3 /sitemap_en.xml Sitemap 14. Juli 2017
1 Fehler
Web 53 Ausstehend
Bilder 23 Ausstehend

4 /sitemap_fr.xml Sitemap 14. Juli 2017
7 Fehler
Web 24 Ausstehend
Bilder 4 Ausstehend

5 /sitemap_nl.xml Sitemap 14. Juli 2017
1 Fehler
Web 40 Ausstehend
Bilder 19 Ausstehend

6  /sitemap_ru.xml Sitemap 14. Juli 2017
13 Fehler
Web 16 Ausstehend
Bilder 2 Ausstehend

Fehler ist immer in entsprechender Anzahl: Url:#
in der deutschen xml Zeile 505
in der spanischen Zeile 43, 47, 55

Kann mir jemand weiterhelfen?

[web28]

Lösung ist einfach.

Datei:
admin\includes\modules\system\sitemaporg.php

Org. Code:

Code: PHP  [Auswählen]

function xml_sitemap_entry($url, $lastmod = '', $products = '') {    
    $this->schema .= "\t<url>\n";

Fix:

Code: PHP  [Auswählen]

function xml_sitemap_entry($url, $lastmod = '', $products = '') {  
    if (trim($url) == '#') return;
    $this->schema .= "\t<url>\n";

Gruss Web28

[lithoverlag]

Ok merci das tut soweit. Alle Fehler bei google webmastertools sind weg. Super!

Bleibt nur noch das Problem, dass die Sitemaps im Moment händisch generiert werden müssen.

Gruß
litho