exlo89 / laravel-sevdesk-api

Helpfull Laravel package to work with the Sevdesk API.
MIT License
7 stars 11 forks source link

InvoiceNumber wird nicht richtig generiert, headText #16

Closed Fernien closed 4 months ago

Fernien commented 7 months ago

Hi, habe aktuell das Problem, dass die Rechnungsnummern nicht mehr richtig generiert werden. Wenn ich in den parametern invoiceNumber => null übergebe erhalte ich diese Rechnungsnummer: RE-%YYYY%MM1038

Jahr und Monat werden nicht richtig gesetzt. headText und footText werden auch nicht übernommen, könnte aber vielleicht folgefehler sein.

Hat jemand ne Idee?

Fernien commented 7 months ago

Wird hier nicht sogar der falsche Key geprüft und überschrieben?

image

Müsste das nicht wie von mir markiert $parameters['invoice']['invoiceNumber'] statts $parameters['invoiceNumber'] sein? Laut SevDesk Doku gehört invoiceNumber unterhalb des invoice keys siehe: image

exlo89 commented 7 months ago

Welche version hast du denn aktuell installiert?

Fernien commented 7 months ago

Ups, ganz vergessen. exlo89/laravel-sevdesk-api dev-main 90239bb

Bin etwas irritiert, weil es vorher so funktioniert hatte. Mir ist kein commit aufgefallen der etwas geändert hätte

Fernien commented 7 months ago

@exlo89 Können wir nicht einfach invoiceNumber = null schicken? Dann dürfte SevDesk das ja komplett ausfüllen

exlo89 commented 7 months ago

Wenn man null zu Sevdesk schickt, dann generiert dir Sevdesk nicht die invoice number. Deswegen habe ich mich ja damals dafür entschieden, dass es gleich mit generiert wird.

exlo89 commented 7 months ago

seit wann tritt denn der Fehler auf?

Fernien commented 7 months ago

Wenn man null zu Sevdesk schickt, dann generiert dir Sevdesk nicht die invoice number. Deswegen habe ich mich ja damals dafür entschieden, dass es gleich mit generiert wird.

Ich glaub das haben die geändert. Hab noch ein Python Script wo ich deren API nutze und wenn ich null übergebe generiert er das richtig

Fernien commented 7 months ago

seit wann tritt denn der Fehler auf?

Kann's nicht genau sagen. Ich generiere darüber aktuell nur einmal im Monat Rechnungen. Letzten Monat ging es noch. Am 1.3 ist mir es aufgefallen

Fernien commented 6 months ago

@exlo89 problem besteht weiterhin. Das Rechnungsdatum wird nicht korrekt generiert. Die generierte rechnung sieht dann so "RE-%YYYY%MM0001" Statts "RE-2024040001"

Fernien commented 6 months ago

Das auszukommentieren hilft prinzipiell in meinem Fall, allerdings weiß ich nicht wo und wie es noch woanders genutzt wird. Wenn die Felder null sind füllt SevDesk den Rest aus image

exlo89 commented 6 months ago

Ich glaub ich weiß was du jetzt meinst. Also wenn die Rechnung erstellt werden mit den "OPEN" Status, dann werden die InvoiceNumber erstellt. Laut Dokumentation wird aber die nächste Rechnungsnummer nicht über die Rechnung erstellt, sondern über einen eigenen API Endpoint. Ich werd diesen Entpoint mit hinzufügen um die nächsten Rechnungsnummern zu generieren.

exlo89 commented 6 months ago

Hab ein neues Update erstellt. Es gibt jetzt getSequence() enpoints um die aktuelle Nummer zu bekommen von einer Rechnung oder einer Gutschrift. Die Formate mit Datum werden aber nicht mit ausgegeben.

Fernien commented 4 months ago

Hab ganz vergessen nochmal Feedback zu geben. Hat alles funktioniert