Abonnementpreis kann nicht auf $19,99 festgelegt werden

[RGJ (Richard - Communiteq)]

• Discourse-Subscriptions verwenden
• Versuchen, einen Preisplan zu einem Produkt hinzuzufügen
• Geben Sie 19,99 in das Preisfeld ein (dies ist bereits schwierig, da die Eingabe sich sehr seltsam verhält)
• Versuchen Sie, zu senden
• Fehlermeldung erhalten: „Ungültige Ganzzahl 1998.9999999999998“

Ich habe dies bisher nur mit Stable 3.5.3 getestet, aber der Code ist in Beta derselbe.

Code

  @computed("unit_amount")
  get amountDollars() {
    return parseFloat(this.get("unit_amount") / 100).toFixed(2);
  }

  set amountDollars(value) {
    const decimal = parseFloat(value) * 100;
    this.set("unit_amount", decimal);
  }

Javascript / IEEE-754 Float-Tücken:

-> parseFloat("19.99")*100
<- 1998.9999999999998

Vorgeschlagene Korrektur: const decimal = Math.round(Number(value) * 100)

Meine größte Frage: Warum ist das niemandem aufgefallen? Hat niemand zuvor den Preis auf 19,99 $ festgelegt?

[pfaffman (Jay Pfaffman)]

@RGJ

Javascript / IEEE-754 Gleanichkeit:

Ich wusste es sofort. Eine weitere Sache, die ich vor über 40 Jahren über Computer gelernt habe und die sich nicht geändert hat.

@RGJ

Vorgeschlagene Lösung: const decimal = Math.round(Number(value) * 100)

Oder 1-Cent-Ganzzahlen verwenden? Ich glaube, das macht Stripe tatsächlich. (Aber Ihre Lösung erfordert viel weniger Codeänderungen, also gewinnt sie wohl.)

[RGJ (Richard - Communiteq)]

@pfaffman

Oder 1-Cent-Ganzzahlen verwenden? Ich glaube, das macht Stripe tatsächlich.

Ja, das tut es, unit_amount sind Cents und amountDollars ist ein ‘XX.XX’-Eingabewert.
Der Fehler „ungültige Ganzzahl“ kommt sogar von der Stripe API.

@pfaffman

Das wusste ich sofort. Noch eine Sache, die ich vor über 40 Jahren über Computer gelernt habe und die sich nicht geändert hat.

Ich schätze, wir sind beide alt genug, um vom FDIV-Bug zu wissen

[sam (Sam Saffron)]

Ehrlich gesagt halte ich es für falsch, dies in einem Float zu speichern. Diese Spalte sollte ein Integer sein und in Cents gespeichert werden.

[RGJ (Richard - Communiteq)]

Es ist, das parseFloat wird verwendet, um von/zu dem Eingabefeld zu konvertieren, das „Währung mit zwei Dezimalstellen“ ist. Die Eigenschaft des unit_amount-Objekts ist offensichtlich typenlos, daher mein Vorschlag, Math.round() zu verwenden.

Es ist keine Spalte auf der Serverseite, es wird an die Stripe API übergeben.

[Bruce5051 (Bruce)]

Für die Währung BCD ist Gleitkommazahl im Allgemeinen besser als binäre Gleitkommazahl.

[zogstrip]

Danke @RGJ für den Bericht – das sollte es beheben

FIX: Round dollar-to-cents conversion in subscription plans (#37877)