Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bei Gastbestellung wird per Default "Abweichende Versand-Adresse" ausgewählt #2316

Closed
contaoacademy opened this issue Jul 4, 2022 · 8 comments
Closed

Bei Gastbestellung wird per Default "Abweichende Versand-Adresse" ausgewählt #2316

contaoacademy opened this issue Jul 4, 2022 · 8 comments
Labels
bug
Milestone
2.8.9

Comments

@contaoacademy
Copy link
Contributor

contaoacademy commented Jul 4, 2022 •
edited

  • Contao: 4.13.5
  • Isotope: 2.8.4

Bisher war beim Checkout (Kasse) bei Gastbestellung als Standardwert «Rechnungs-Adresse verwenden» ausgewählt.
Was auch so gewünscht ist.

Aktuell ist allerdings «Abweichende Versand-Adresse» als Standard gesetzt und kann nicht geändert werden.
Beim Debuggen ist mir Folgendes aufgefallen.
(siehe Screenshot)

Warum hier einmal bei default = 1 und einmal default = true zurückgegeben wird, konnte ich nicht herausfinden.

screen-20220704_LgJS9AMt

screen-20220704_81ovRw5V
@netzarbeiter
Copy link
Member

Fehler tritt nur mit Contao 4.13 auf.

@drum-und-dran
Copy link

Same here. Contao 4.13 LTS

Firefox_Screenshot_2022-12-20T13-37-02 784Z

@drum-und-dran
Copy link

drum-und-dran commented Dec 28, 2022 •
edited

Wie wird denn dieser Bug offiziell als Bug deklariert?

Kann das bitte ein Entwickler mit der entsprechenden Berechtigung übernehmen?
Eventuell beschleunigt dies die Fehlerbehebung etwas.

Mittlerweile ist das Issue ja seit Anfang Juli offen, jetzt haben wir Ende Dezember.

Besten Dank!

@netzarbeiter
Copy link
Member

Wenn dir die Fehlerbehebung hier zu langsam ist, hast du folgende Möglichkeiten:

  • Du nimmst Kontakt mit einem Entwickler bzw. einer Entwicklerin auf und bezahlst sie für den Bugfix
  • Du korrigierst den Fehler selber und lässt uns per PR daran teilhaben
  • Du wirst Mitglied im Circle und erhältst Support zum Vorzugspreis

aschempp added a commit that referenced this issue Jan 17, 2023
@aschempp
Correctly check for default shipping address (#2316)
1c9d337 
The comparison previously worked because `null !== '0'`.
However, Doctrine/Contao now return real integer values from the database,
so, `null == 0` is `true`
@aschempp
Copy link
Member

Der Fehler hat tatsächlich nichts mit Contao 4.9/4.13 zu tun sondern mit der Datenbank bzw. Doctrine-Version. Behoben in 1c9d337

@aschempp aschempp added this to the 2.8.9 milestone Jan 17, 2023
@contaoacademy
Copy link
Contributor Author

Danke

@AlexanderWillner
Copy link
Contributor

Ich glaube, dass es hier u.U. einen Folgefehler gibt: aktuell wird in der DB keine Versandadresse gespeichert (shipping_address_id = 0), wenn die Standardoption Rechnungs-Adresse verwenden ausgewählt wird. Kann das sein?

@aschempp
Copy link
Member

alles kann sein 😆 gerne analysieren, einen Bugfix-PR machen oder wenigesten ein GitHub Issue, weil dass hier ist geschlossen und da geht es verloren.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
2.8.9
Development

No branches or pull requests
5 participants
@AlexanderWillner @aschempp @netzarbeiter @drum-und-dran @contaoacademy