Modify

Opened 6 years ago

Closed 6 years ago

Last modified 3 years ago

#1765 closed Bug/Fehler (fixed)

Doppeltes itemprop="price" bei Staffelpreisen

Reported by: fiveBytes Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.5.1
Component: Template Version: 2.0.5.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Torsten Riemer)

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 itemprop="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 zumidest folgenden Anpassung vorschlagen:

aus <meta itemprop="price" content="{'''$price_data'''.PRODUCTS_PRICE_PLAIN}" /> in Zeile 46 wird
{* Doppeltes itemprop="price" bei Staffelpreisen vermeiden!!! *}
{if ! __[http://www.php.net/isset isset]__($MODULE_graduated_price) || $MODULE_graduated_price == ''}<meta itemprop="price" content="{'''$price_data'''.PRODUCTS_PRICE_PLAIN}" />{/if}

Attachments (0)

Change History (9)

comment:1 by FräuleinGarn, 6 years ago

So sah der Code im Forum aus

Suche in Zeile 46:

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

Ersetze mit:

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

Siehe: Doppeltes itemprop="price" bei Staffelpreisen

Last edited 6 years ago by Torsten Riemer (previous) (diff)

comment:2 by Torsten Riemer, 6 years ago

Component: ShopTemplate
Milestone: modified-shop-2.0.5.1
Version: 2.0.5.0

comment:3 by Markus Rehm, 6 years ago

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)

{* 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

comment:4 by Torsten Riemer, 6 years ago

Description: modified (diff)
Summary: Doppeltes itemprob="price" bei StaffelpreisenDoppeltes itemprop="price" bei Staffelpreisen

comment:5 by Markus Rehm, 6 years ago

Bei einem fehlenden Kundengruppenpreis in Verbindung mit Staffelpreisen für verschiedene Kundengruppen wird in der price_info.html nicht der Abschnitt für "graduated_price" gezogen sondern es kommt der Abschnitt für "standard_price".

Wenn das korrigiert ist, kann in dem Zuge im Abschnitt "graduated_price" auch die Meta-Angabe für den Preis raus da die dann über die Staffelpreise kommt.

comment:6 by Gerhard Waldemair, 6 years ago

Resolution: fixed
Status: newclosed

In 12573:

fix #1765 - removed duplicate itemprop="price" on graduated prices

comment:7 by Markus, 3 years ago

In 15023:

fix #1362, #2435 - partly revoked r12573 (fix #1765 - removed duplicate itemprop="price" on graduated prices) - add missing itemprop="price" on graduated prices when no product price is set

comment:8 by Torsten Riemer, 3 years ago

In 15024:

revoked r15023 (fix #1362, #2435 - partly revoked r12573 (fix #1765 - removed duplicate itemprop="price" on graduated prices) - add missing itemprop="price" on graduated prices when no product price is set) - due to wrong fix

comment:9 by Markus, 3 years ago

In 15026:

fix #1362, #2435 - revoked r12573 (fix #1765 - removed duplicate itemprop="price" on graduated prices) - add missing itemprop="price" on graduated prices when no product price is set

Modify Ticket

Action
as closed The owner will remain somebody.
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.