Preis pro Einheit / Brooklyn Theme

[arndmesser]

Hallo Zusammen,

ich verwende das Theme Brooklyn, in der Version 15.2.0.

"Price per unit" bzw. "Preis pro Einheit", wie in dem Manual beschrieben, ist dieser Version schon integriert. Allerdings mit einer Abweichung, aber dazu später mehr.

Das Manual:

https://help.shopify.com/de/manual/intro-to-shopify/initial-setup/sell-in-germany/price-per-unit

Die Ausgabe in Forntend ist unvollständig und wird nicht angezeigt.

https://arndmesser.de/collections/bioland-honig/products/bioland-rapshonig

Im Backend ist folgendes konfiguriert:

Im Frontend fehlt die Unit-Bezeichnung "100g":

<div class="product-single__unit">
  <span class="product-unit-price">
    <span class="visually-hidden">Stückpreis</span>
    <span data-unit-price="">€1,74</span>
    <span aria-hidden="true">/</span>
    <span class="visually-hidden">&nbsp;pro&nbsp;</span>
    <span data-unit-price-base-unit=""></span>
  </span>
</div>

Ausserdem ist das entsprechende DOM-Element "ausgeblendet":

.product-single__unit {
    display: none;
}

Ich hab den Code aus dem Manual flüchtig verglichen, um die Integration zu checken. Dabei ist mir folgendes aufgefallen:

Der Code in der Datei theme.js.liquid

        if (variant.unit_price) {
          var $unitPrice = $(this.selectors.unitPrice, this.$container);
          var $unitPriceBaseUnit = $(
            this.selectors.unitPriceBaseUnit,
            this.$container
          );

          $unitPrice.html(
            theme.Currency.formatMoney(variant.unit_price, moneyFormat)
          );
          $unitPriceBaseUnit.html(this.getBaseUnit(variant));

          $(this.selectors.priceContainer, this.$container).addClass(
            this.classes.priceContainerUnitAvailable
          );
        }

Der Code wie er laut Manual sein sollte:

if (variant.unit_price_measurement) {
  var $unitPrice = $(this.selectors.unitPrice, this.$container);
  var $unitPriceBaseUnit = $(
    this.selectors.unitPriceBaseUnit,
    this.$container
  );

  $unitPrice.html(
    theme.Currency.formatMoney(variant.unit_price, moneyFormat)
  );
  $unitPriceBaseUnit.html(this.getBaseUnit(variant));

  $(this.selectors.priceContainer, this.$container).addClass(
    'price-container--unit-available'
  );
}

console.log() wurde in beiden Code-Blöcke nicht ausgeführt.

Ich bin etwas ratlos.

Eventuell habe ich auch etwas übersehen, deswegen suche ich hier Hilfe.

Was könnte die korrekte Ausgabe des Unit-Price im Frontend verhindern?

Grüße,

Arnd

[Gabe]

Hey Arnd @arndmesser 

Hast du das Problem jemals gelöst?

VG,

Gabe

[arndmesser]

Hallo Gabe,

nein, das Problem habe ich nicht gelöst.
Ich könnte das im Prinzip, aber leider fehlt mir die Zeit für eine intensivere Fehlersuche.

Das Problem müsste doch auch in anderen Shops aufgetaucht sein?

Grüße

Arnd

[Gabe]

Hi Arnd! @arndmesser 

Ich werde das jetzt mit unserem Grundpreis Entwicklerteam besprechen. Die haben harte Arbeit geleistet mit dem Grundpreis denn da gibt es viele 'moving parts' im System aber mal sehen was die sagen.

VG,

Gabe

[Gabe]

Hey Arnd! @arndmesser 

Dank dein Hinweis sind unsere Entwickler jetzt auf dieses Problem aufmerksam geworden und bauen gerade einen Fix.

Im großen und ganzen wird der Grundpreis ausschließlich im Brooklyn-Theme nicht angezeigt, wenn ein Produkt nicht mehr im Bestand vorrätig ist. Die Hilfe-Dokumente müssen wahrscheinlich auch aktualisiert werden.

VG,

Gabe

[arndmesser]

@Gabe 

Gibt es die Möglichkeit die Änderungen irgendwo einzusehen?
Ich habe den Theme, wie man unschwer im Frontend sehen kann, an meine Vorstellungen angepasst. Ein komplettes Update würde ich gerne bis zum nächsten Winter vermeiden. =;o)

Grüße

Arnd

[Gabe]

Hi Arnd,

Meinst du die neuen Fixe und dass die deine Code Anpassung nicht zerschiessen? Nein, dein Theme bleibt so wie es ist. Die Fixe kommen nur in neue Themes rein und nicht in bestehende Themes.

Darf ich fragen wie du das Problem im Code gelöst hast? 

VG,

Gabe

[arndmesser]

Moin Gabe,

ich würde gerne die Änderungen in meinen Code integrieren.

Mein Theme kann ich im Moment leider nicht updaten. Ich müsste dann alles noch mal anpassen, und mir fehlt gerade einfach die Zeit dazu.

Ich könnte natürlich nach dem Release jede Datei fleißig diff-en. Wenn ich von euch die Änderungen bekommen könnte, wäre das jedoch wesentlich schneller.

So meinte ich das.

Ich habe im Moment gar nichts gelöst.

Vor einem Jahr habe ich meinen alten Job als Programmierer an den Nagel gehängt und mich mit meiner Imkerei selbstständig gemacht. Zeit ist gerade extreme Mangelware. Ich habe überall Baustellen.

Wenn, dann hätte ich dreckig die Nummer hart verdrahtet und gehofft, dass der TÜV nicht kommt ^^

Grüße aus der Pfalz

Arnd

[Gabe]

Hey @arndmesser 

Tut mir leid, habe ich nicht gesehen da ich nicht getaggt war. Hast du das gelöst?