AddTradeLineItem wurde als Obsolete gekennzeichnet

[IngoManthey]

In ReadMe wurde
desc.AddTradeLineItem("Item name", "Detail description", QuantityCodes.C62, ....); erklärt.
Ich finde leider keine Methode mit der ich meine Rechnungspositionen eintragen kann.

Danke für jeden Hinweis und Tipp

[stephanstapel]

@IngoManthey : schau Dir am besten das Beispiel unter
https://github.com/stephanstapel/ZUGFeRD-csharp/blob/master/ZUGFeRD.Test/InvoiceProvider.cs
an. Dort wird eine komplette Rechnung aufgebaut

[IngoManthey]

Danke für den link.
Wenn ich es implementiere wie folgt:
desc.AddTradeLineItem(name: item.Article,
unitCode: QuantityCodes.C62,
sellerAssignedID: null,
id: null,
grossUnitPrice: grossPrice,
netUnitPrice: item.UnitPrice,
billedQuantity: item.Amount,
taxType: TaxTypes.VAT,
categoryCode: TaxCategoryCodes.S,
taxPercent: item.RateOfTax * 100
);

Wird mir immer noch anzeigt das AddTradeLineItem obsolete ist. Das verwirrt mich!

[stephanstapel]

Ist mir bewusst, dass das Obsolete fragwürdig ist. Der Text, der angezeigt wird, sollte aber hoffentlich Klarheit verschaffen.

[IngoManthey]

Danke für die Hilfe

[Matze177]

Moin. Die gegebenen Antworten sowie der Hinweis aus dem Obsolete-Attribute machen mich leider noch nicht wirklich schlauer:

1. Warum sind alle Überladungen der Methode aktuell als obsolete markiert?
2. Was ist die (nicht obsolete)-Alternative die verwendet werden soll?

[stephanstapel]

Hallo @Matze177 ,
bitte schaue auf den Text, der Dir angezeigt wird. Dort wird erklärt, dass manche Parameter Obsolete sind.

[Matze177]

Ok, ich verstehe. Danke für den Hinweis. Allerdings sind es scheinbar Parameter die ich und der OP nur über den Default-Value "verwenden", also nicht einmal im Code angeben.

Wird es dann demnächst eine Überladung ohne diese Parameter geben?

[stephanstapel]

Hallo @Matze177 ,
ich versuche, immer möglichst klar :) zu markieren, was sich in der nächsten Major-Version ändern wird. Dass es bis zur nächsten Major Version noch etwas hin ist, steht dann auf einem anderen Blatt. Es gibt noch einige Themen, die bearbeitet werden müssen, allerdings habe ich aktuell andere Prioritäten.

[Tragen]

Schade dass es keine Methode ohne Obsolete gibt weil wir mit der Einstellung "Treat Warnings as Errors" arbeiten.

[stephanstapel]

Ich hatte das tatsächlich so umgesetzt, dann ist allerdings die Signatur nicht mehr eindeutig, da die hinteren Parameter alle ohnehin optional sind.

[Tragen]

Ja, dotnet ist manchmal ein bisschen "dumm".
Ich hab es jetzt mit einem
#pragma warning disable 0618
#pragma warning restore 0618
gelöst um die Zeilen.