Incorrect (IMHO) default value for BasisQuantity aka UnitQuantity

[PeterSawatzki-MGGM]

In InvoiceDescriptor23CIIReader.cs, method _parseTradeLineItem: the UnitQuantity gets a default value of 1:
UnitQuantity = XmlUtils.NodeAsDecimal(tradeLineItem, ".//ram:BasisQuantity", nsmgr, 1),

Wouldn't it be better to not assign a defualt value at all:

UnitQuantity = XmlUtils.NodeAsDecimal(tradeLineItem, ".//ram:BasisQuantity", nsmgr),

for example reading and writing zugferd232de\Beispiele\4. EXTENDED\EXTENDED_Warenrechnung\factur-x.xml would then suppress unnecessary (and wrong?) output:

  <ram:SpecifiedLineTradeAgreement>
    <ram:GrossPriceProductTradePrice>
      <ram:ChargeAmount>1.50</ram:ChargeAmount>
      <ram:BasisQuantity unitCode="H87">1.0000</ram:BasisQuantity>

original was:

  <ram:SpecifiedLineTradeAgreement>
    <ram:GrossPriceProductTradePrice>
      <ram:ChargeAmount>1.5000</ram:ChargeAmount>

I can make a pull request on this if you agree.

[stephanstapel]

Yes, thanks for the suggestion. Please go ahead. Can you do this for all profiles and add a test case?
There are other test cases already which are consuming the official examples like

ZUGFeRD-csharp/ZUGFeRD.Test/ZUGFeRD22Tests.cs

Line 3113 in 9b5c40f

string path = @"..\..\..\..\documentation\zugferd23de\Beispiele\4. EXTENDED\EXTENDED_InnergemeinschLieferungMehrereBestellungen\factur-x.xml";

[PeterSawatzki-MGGM]

will do.

[PeterSawatzki-MGGM]

I changed this and added a test "TestTradeLineItemUnitChargeFreePackageQuantity"