-
-
Notifications
You must be signed in to change notification settings - Fork 97
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.
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
DefinedTradeContact missing in Zugferd 2.1, EN16931/Comfort #210
Comments
I see your point. Things are a bit more complicated unfortunately. Please search for DefinedTradeContact in the TA document. DefinedTradeContact is written in Comfort profile only for the parties:
but not:
i.e. trade party is not trade party anymore. I honestly have no idea how to implement this as _writeOptionalParty() is used for all sorts of parties. |
Same issue for me. When Profile basic is used, then DefinedTradeContact is omitted. When using XRECHNUNG, everything works. |
@KSafadi: yes, that should be alright. It is defined like this in the specification. |
@stephanstapel alright! I am writing to the SellerTradeParty. |
|
@stephanstapel ah okay that makes sense. Thank you. |
@stephanstapel I don't understand. Wouldn't that call of _writeOptionalContact need Profile.Comfort added to the Profile-parameter for it to be written in the profile for EN 16931? |
@marvinvalent : you are right, thanks |
Hi,
documentation for Zugferd 2.1 says that DefinedTradeContact is used in EN16931/Comfort.
If output profile XRechnung is used, then DefinedTradeContact is correctly written,
but if EN16931/Comfort is used, then DefinedTradeContact is omitted.
I guess that the reason is that in InvoiceDescriptor21Writer.cs Line 1185 Profile.Comfort is missing:
_writeOptionalContact(writer, "ram:DefinedTradeContact", contact, Profile.Extended | Profile.XRechnung1 | Profile.XRechnung);
Is this a bug, or is this DefinedTradeContact in EN16931 intentionally missing for some reason ?
Regards
Werner
The text was updated successfully, but these errors were encountered: