There are a few cases where EDI passes a bare object to the logs.. we should update those cases to log identifiable information instead.
Created attachment 122648 [details] [review] Bug 28672: Pass itemnumber instead of item object The warn call here was passing the full item object which leads to HASH() in the logs.. this patch updates it to pass the itemnumber which is more helpful.
The patch applies but I am unsure of how to proceed in order to confirm that this works as intended. Can you provide a test plan?
Created attachment 145931 [details] [review] Bug 28672: Pass itemnumber instead of item object The warn call here was passing the full item object which leads to HASH() in the logs.. this patch updates it to pass the itemnumber which is more helpful. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 145932 [details] [review] Bug 28672: (QA follow-up) Method calls cannot be interpolated directly Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 145933 [details] [review] Bug 28672: (QA follow-up) Method calls cannot be interpolated directly Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(In reply to Olivier Hubert from comment #2) > The patch applies but I am unsure of how to proceed in order to confirm that > this works as intended. Can you provide a test plan? Testing this patch requires importing an EDI invoice with a partially received quantity. This is trivial and hard to test so I am passing qa directly.
Thanks for tackling this one Kyle, and thanks for the follow-up.. just reading it this morning I thought we probably needed that.
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work everyone! Pushed to 22.11.x for next release
Technically an enhancement. However, better logging is always good so I choose to backport. Backported to 22.05.x for upcoming 22.05.10
same as lucas, applied to 21.11.x for 21.11.18
Not backported to 21.05.x