Doppeltes itemprop="price" bei Staffelpreisen

[fiveBytes]

Hallo zusammen,

ich bin mir nicht sicher, ob das ein Bug ist oder ob das grundsätzlich kein Problem darstellt, aber sofern bei einem Produkt Staffelpreise angelegt werden, erzeugt das Template "module/includes/price_info.html" ein doppeltes itemprob="price", denn es wird sowohl bei der eigentlichen Preisausgabe (das enthält dann auch den  falschen Wert für die Bestelmenge 1) und bei den Staffelpreisen selber ausgegeben. Google und Co dürfte das wahrscheinlich auch ein wenig irritieren, von daher würde ich folgenden Anpassung vorschlagen:

Suche in Zeile 46:

Code: XML  [Auswählen]

<meta itemprop="price" content="{$price_data.PRODUCTS_PRICE_PLAIN}" />

Ersetze mit:

Code: PHP  [Auswählen]

{* Doppeltes itemprop="price" bei Staffelpreisen vermeiden!!! *}
{if !isset($MODULE_graduated_price) || $MODULE_graduated_price == ''}<meta itemprop="price" content="{$price_data.PRODUCTS_PRICE_PLAIN}" />{/if}

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

[noRiddle (revilonetz)]

Ich hab's nicht geprüft. Wenn du sicher bist mit dem was du sagst erstelle bitte ein Ticket im Bug-Tracker, sonst bringt das nichts und geht unter.

Gruß,
noRiddle

[Markus]

HI fiveBytes,

kann es sein dass du dich da in den Zeilen vertan hast?

In der price_info.html steht ja dieser Block hier... (ab Zeile 30)

Code: XML  [Auswählen]

                {* START Produkt Staffelpreis mit Ausgabe ab Preis, Stückpreis *}
                <span class="graduated_price">
                  {if isset($price_data.PRODUCTS_PRICE_FROM)}<span class="small_price">{$smarty.const.FROM}</span> {/if}{$price_data.PRODUCTS_PRICE_OLD_PRICE}
                </span>
                <span class="item_price"><span class="small_price">{$smarty.const.UNIT_PRICE}</span> {$price_data.PRODUCTS_PRICE_SPECIAL_PRICE}</span>
                <meta itemprop="price" content="{$price_data.PRODUCTS_PRICE_PLAIN}" />
                {* ENDE Produkt Staffelpreis mit Ausgabe ab Preis, Stückpreis *}

Hier sind wir ja schon in dem Block wenn Staffelpreise vorhanden sind ... es müsste demnach nur das <meta itemprop ... raus.

Markus

[fiveBytes]

@Markus
Hätte ich auch so gedacht, funzt aber nicht könnte allerdings sein, dass die vorherigen itemprops auch alle weg müssen, sofern Staffelpreise angezeigt werden. Denn das Template bringt ein eigenes itemprop mit. Für meine(n) Kunden passt es erst mal so und ich hab jetzt auch ein Ticket dafür aufgemacht, mal sehen, was die Pros dazu sagen ...

[Markus]

Hi fiveBytes,

ich hatte im Ticket auch schon geschrieben ... und bei mir im Testshop funktioniert das ohne Probleme wenn ich in der price_info.html die Zeile in dem Abschnitt hier raus nehme.

Code: XML  [Auswählen]

 {* START Produkt Staffelpreis mit Ausgabe ab Preis, Stückpreis *}

Hast du denn um den eigentlichen Preis auch einen Container mit "<span class="graduated_price">" oder wie heisst der bei dir?

Markus

[fiveBytes]

Nein, natürlich nicht, sonst wäre es ja die von Dir beschriebene Stelle gewesen ist eine komplett frische 2.0.5.0er Installation mit tpl_modified Template, der Quellcode schaut so aus:

Code: PHP  [Auswählen]

<div class="pd_summarybox" itemprop="offers" itemscope itemtype="http://schema.org/Offer">
  <meta itemprop="url" content="https://imagro.fivebytes.de/anzucht-stecklingsbedarf/substrate-anzucht/stecklingspad.html" />
  <meta itemprop="priceCurrency" content="EUR" />
  <meta itemprop="availability" content="http://schema.org/InStock" />
  <meta itemprop="itemCondition" content="http://schema.org/NewCondition" />
 
  <div class="pd_price">
    <div id="pd_puprice">
      <span class="standard_price">
       <span class="small_price"> ab  </span>                
       1,35 EUR
      </span>
      <meta itemprop="price" content="1.35" />
    </div>
  </div>
 
  <div class="pd_vpe">0,09 EUR pro Stück</div>          
  <div class="pd_tax">inkl. 19 % MwSt. zzgl. <a rel="nofollow" target="_blank" href="https://imagro.fivebytes.de/popup_content.php?coID=1" title="Information" class="iframe">Versandkosten</a></div>

<div class="graduated_prices cf">
  <div class="gp_headline">Staffelpreise</div>
      <div class="gp_box vpe">
      <span class="gp_title">ab 1 Stk.</span>
      <span class="gp_price">je 2,45 EUR</span>
      <span class="gp_vpe">0,16 EUR pro Stück</span>    </div>
              <div class="gp_box vpe last">
      <span class="gp_title">ab 180 Stk.</span>
      <span class="gp_price">je 1,35 EUR</span>
      <span class="gp_vpe">0,09 EUR pro Stück</span>    </div>
            <div itemprop="PriceSpecification" itemscope itemtype="http://schema.org/PriceSpecification">
    <meta itemprop="priceCurrency" content="EUR">
    <meta itemprop="price" content="2.45">
    <meta itemprop="minPrice" content="1.35">
    <meta itemprop="maxPrice" content="2.45">
  </div>
</div>
 

[Markus]

Hi,

das kann eigentlich nicht sein.
Hast du mal den Template-Cache geleert oder ist sonst irgendein Cache aktiv?

Wenn Staffelpreise angezeigt werden muss kommen:

Code: XML  [Auswählen]

<span class="graduated_price">

und nicht:

Code: XML  [Auswählen]

<span class="standard_price">

Markus

[fiveBytes]

Nein und wie gesagt, dass das wohl so sein sollte ist mir durchaus klar. Ist aber leider nicht der Fall. Wird wohl ein Bug in der Generierung des Preis-Arrays sein, evtl. bei der Generierung vom PRODUCTS_PRICE_FLAG.

Ist wie gesagt eine komplett frische 2.0.5.0er Version mit dem Standardtemplate "tpl_modified", alles ohne Cache o.ä. einfach nur installiert, Kategorien und Artikel rein und Staffelpreise angelegt.

[Tomcraft]

Kannst du bitte mal einen Screenshot der Preise des Artikels aus dem Adminbereich hier anhängen?
Wir können es nämlich nicht nachvollziehen. Wir erhalten immer das folgende DIV:

Code: XML  [Auswählen]

<span class="graduated_price">

und nicht:

Code: XML  [Auswählen]

<span class="standard_price">

Grüße

Torsten

[fiveBytes]

Hier kommen die Screens, evtl. liegt es an der Kombi Staffelpreis und Grundpreis (VPE)?!

[manne35]

Bei Staffelpreise immer den Einzelpreis mit eintragen.
Vielleicht geht's dann ja auch schon.
Lgmanne

[Markus]

Hi fiveBytes,

ich hab das Problem ... es liegt am fehlenden Kundengruppenpreis.
Wenn du beim Kundengruppenpreis auch die 2.45 einträgst läuft es richtig. Ansonsten eben nicht ... dann kommt "standard_price".

Ich schreib das mal ins Ticket ... mal schauen wie wir das lösen
In dem Zuge kann dann auch die meta-price Angabe an der richtigen Stelle rausgelöscht werden.

Markus

[fiveBytes]

In der Tat, das kann ich so bestätigen. Ich halte ein solches Eingabeverhalten allerdings bei den meisten Nutzern/Shopbetreibern für eher unwahrscheinlich, zumindest eben bei gleichem Grundpreis für alle Gruppen. Auch die Darstellung mit dem zusätzlichen Stückpreis finde ich doch eher verwirrend als hilfreich. Die Staffeln werden ja einzeln mit allen nötigen Angaben aufgeführt. Aber das ist ja im Endeffekt auch wenig "Geschmackssache" und kann jeder im Template anpassen, wie es für den Shop am besten passt.

Insofern müssten entweder die Standardtemplates entsprechend an besprochener Stelle angepasst werden (also doppeltes itemprop raus) oder die itemprops werden gesammelt an einer Stelle im Grundtemplate ausgegeben, dann könnten die Angaben direkt in der product Klasse erzeugt werden, wenn ich das so richtig überblicke!?

Ich lass mich von Eurer Lösung überraschen, vielen Dank bis hierhin für Eure Hilfe
Gruss Stefen