MInvoice.getDocumentInfo should use new getDocTypeID
Description
Cause it hurts when the invoice is not processed (and C_DocType_ID is null)
Environment
None
Attachments
1
Activity
Show:
Nicolas Micoud October 28, 2015 at 4:01 PM
ok, i see what you mean
Nicolas
Carlos Ruiz October 28, 2015 at 3:59 PM
I think that's for safety - avoiding to send to external parties information about draft invoices that could change before completing. Would be better not to open such capability to break a safety measure in core.
Nicolas Micoud October 28, 2015 at 3:57 PM
or maybe add a param (getTargetIfDocTypeIsNotSet) ?
Carlos Ruiz October 28, 2015 at 3:56 PM
, maybe that's intentional, is better in some cases (like when printing) to show "** New **" as the Document Type for unprocessed docs.
In this case I see is used for dunning, sending email. I think there is a correct idea to show it that way.
Cause it hurts when the invoice is not processed (and C_DocType_ID is null)