Modify

Opened 10 years ago

Closed 6 years ago

Last modified 6 years ago

#704 closed Bug/Fehler (fixed)

hreflang in 2.0 nicht korrekt implementiert

Reported by: noRiddle Owned by: Ronald Parcinski
Priority: hoch Milestone: modified-shop-2.0.5.0
Component: Shop Version: 2.0.0.0
Keywords: Cc:
Blocked By: Blocking:

Description

Die Implementation von hreflang ist im Demoshop 2.0 nicht korrekt.
Ist man auf Englisch steht dort dies:

<link rel="alternate" href="http://demo.modified-shop.org/tags/modified-2.00/index.php?language=de" hreflang="x-default" />
<link rel="alternate" href="http://demo.modified-shop.org/tags/modified-2.00/index.php?language=de" hreflang="de" />
<link rel="alternate" href="http://demo.modified-shop.org/tags/modified-2.00/" hreflang="en" />

Es fehlt also der GET-Parameter für die englische Sprache.
Der GET-Parameter muß da sein weil die URL ansonsten auf Deutsch führt da ja die

$_Session['language']

oder

$_SESSION['language_code']

bei Google-Ergenbnissen nicht gesetzt ist und somit ohne GET-Parameter auf die Standard-Sprache und somit Deutsch gelinkt wird.
Die Standardsprache muß außerdem beachtet werden für den Fall, daß jemand nicht Deutsch als Standard in seinem Shop wählt,
was bedeutet, daß eigtl. dann ja der hreflang für den Standard keinen GET-Parameter für die Sprache benötigt.
Da es jeodch außerdem noch eine Auslesung der bevorzugten Browsersprache gibt verkompliziert sich das Ganze.

Außerdem frage ich mich ob es gut ist, daß auf der Startseite bei Sprachumschaltung und somit auch im hreflang die URL mit */index.php steht. Da die URL auch ohne geht haben wir hier eine Art "double content". Ohne GET-Sprachparameter ist das nicht der Fall, klar, aber mit eben schon. Das ist auch in der 1.06 schon so.

Gruß,
noRiddle

Attachments (0)

Change History (21)

comment:1 by Torsten Riemer, 10 years ago

Component: AdminShop

comment:2 by anonymous, 10 years ago

*NACHTRAG*
Vielleicht sollte man in dem Zuge auch gleich darüber nachdenken ob man nicht virtuelle Unterordner für die Nicht-Standard-Sprachen nimmt und diese mittels htaccess intern umleitet auf den Query-String.
DOMAIN/en/REQUEST_URI
aber für die Standardsprache
DOMAIN/REQUEST_URI

comment:3 by Ines Willenbrock <ines@…>, 10 years ago

Also wenn, dann würde ich sagen, alle Sprachversion mit 2stelligem ISO-Code für die Sprache und keine Ausnahme für Default.

Dann gäbe es zwar 3 Versionen für die Startseite, wie google sie sieht, aber ich denke, das man das auch in den Griff bekommen kann:
domain.tld
domain.tld/index.php
domain.tld/DEFAULT/index.php

Alle Links von der Startseite sollten dann domain.tld/LANG/REQUEST_URI folgen.

Grüße
Ines

comment:4 by anonymous, 10 years ago

Ich finde da SEO beainträchtigt ist sollte das Hauptthema die Priorität 'hoch' bekommen.

Gruß,
noRiddle

comment:5 by Torsten Riemer, 10 years ago

Priority: normalhoch

comment:6 by Ronald Parcinski, 10 years ago

Owner: changed from somebody to Ronald Parcinski
Status: newaccepted

comment:7 by Ronald Parcinski, 10 years ago

hreflang="x-default" sollte immer auf en = Englisch stehen

Sprachparameter müssen bei allen hreflang vorhanden sein.

comment:8 by Ronald Parcinski, 10 years ago

Resolution: fixed
Status: acceptedclosed

In 9120:

fix #704

comment:9 by Torsten Riemer, 8 years ago

Reporter: changed from anonymous to noRiddle

comment:10 by anonymous, 7 years ago

Sicherheitshalber nachgefragt:
Es gibt in der aktuellen Shop-Version 2.0.4.2 nach wie vor double Content für die Startseite.

  1. Klickt man z.B. die deutsche Fahne an ergibt sich diese URL DOMAIN.TLD/index.php?language=de und die hreflang sehen so aus
    <link rel="alternate" href="DOMAIN/index.php?language=en" hreflang="x-default" />
    <link rel="alternate" href="DOMAIN/index.php?language=de" hreflang="de" />
    <link rel="alternate" href="DOMAIN/index.php?language=en" hreflang="en" />
    
    Das ist
    1. double Content weil der selbe Inhalt auch einfach mittels DOMAIN.TLD aufgerufen werden kann, also ohne index.php, und
    2. potentiell double Content weil in der Default-Sprache, also ohne einen Klick auf eine Sprache, der selbe Inhalt ebenfalls mittels DOMAIN.TLD aufgerufen werden kann, weil die Sprache in der Session dann automatisch die Default-Sprache ist.
  2. stellt sich allgemein die Frage ob man Google überhaupt Links mit ?language=XX mitteilen sollte wenn es sich um die Default-Sprache handelt, wo es dann ja immer zu double Content kommen dürfte, eben wegen des in 1.2. Beschriebenen.

Da der Link hinter den Sprach-Fähnchen den GET-Parameter für die Sprache benötigt, müsste es bei Aufruf der Startseite in der Default-Sprache mit ?language=XX, also nach Klick auf die Default-Sprache, einen Canonical-Link auf denselben Link ohne den GET-Parameter geben.

Ist das alles gefixt worden ? (was ich frage weil das Ticket ja geschlosssen wurde)

Gruß,
noRiddle

comment:11 by anonymous, 7 years ago

Sistrix moniert übrigens bei Aufruf von DOMAIN.TLD (und somit in der Default-Sprache) genau das

"Es fehlt die Rückbeziehung auf die Quellseite"

da ja im hreflang steht DOMAIN.TLD?language=de
oder was immer die Default-Sprache ist.

Gruß,
noRiddle

comment:12 by Torsten Riemer, 7 years ago

Hi noRiddle,

wenn du testest, dann bitte immer im Dev-Demoshop, da hier bereits weitere Änderungen aus Ticket #1438 & #1523 mit eingeflossen sind.

Das von dir beschriebene Problem ist in Ticket #1366 festgehalten, wenn ich das richtig sehe.

comment:13 by anonymous, 7 years ago

Im Dev-Demo-Shop sehe ich keinen Unterschied was die von mir im vorigen Post beschriebenen Dinge angeht.
In wie weit das von mir in Ticket #1438 monierte hard-codierte hreflang=x-default gefixt wurde, kann ich ja so nicht sehen und beurteilen.

Wie kann ich eigtl. hier im Bug-Tracker Benachrichtigungen bekommen wenn etwas gepostet wurde ? Ich schaue da mom. immer mal wieder vorbei.
Anscheinend ist das Konto für das Forum ja nicht im Bug-Tracker verwendbar ?

Gruß,
noRiddle

comment:14 by anonymous, 7 years ago

Bzgl. "Es fehlt die Rückbeziehung auf die Quellseite" hat auch Tomcraft mal ein Ticket #1366 erstellt.
In dem Ticket habe ich auf dieses hier verlinkt, deshalb hier der Querverweis.

Gruß,
noRiddle

Version 0, edited 7 years ago by anonymous (next)

comment:15 by demoncleaner, 7 years ago

Moin, Ich seh das genau so wie noRiddle und stehe vor dem selben Problem. Sollte man das Ticket daher nicht wieder öffnen?
@noRiddle: hast du dir das händisch gefixed?
Sorry, wenn ich das hier so rienschriebe, da das ja nicht das Forum ist, aber ganz eigennützig wollte ich das Ticket hier noch mal in irgendeiner Form pushen. ich habe nämlich Sorge, dass durch die jetzt Mehrsprachigkeit meines Shops und die angezeigten Fehler, ein SEO-Nachteil entstehen könnte.

comment:16 by Torsten Riemer, 6 years ago

Resolution: fixed
Status: closedreopened

Ich öffne das Ticket nochmal aufgrund der Rückmeldungen.

comment:17 by Gerhard Waldemair, 6 years ago

Resolution: fixed
Status: reopenedclosed

In 11958:

fix #704, #1366, #1438 & #1529

comment:18 by Gerhard Waldemair, 6 years ago

In 11959:

fix #704, #1366, #1438 & #1529

comment:19 by Torsten Riemer, 6 years ago

In 11962:

Addition on r11958 (fix #704, #1366, #1438 & #1529)

comment:20 by Torsten Riemer, 6 years ago

Milestone: modified-shop-2.0.0.0modified-shop-2.0.5.0

comment:21 by Torsten Riemer, 6 years ago

In 11963:

Addition on r11959 (fix #704, #1366, #1438 & #1529)

Modify Ticket

Action
as closed The owner will remain Ronald Parcinski.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.