reportstart: switch query process by process_id instead of AD_PInstance_ID
Description
ReportStart.getReportData() query report info by
"SELECT pr.JasperReport, pr.IsDirectPrint "+"FROM AD_Process pr, AD_PInstance pi "+"WHERE pr.AD_Process_ID = pi.AD_Process_ID "+" AND pi.AD_PInstance_ID=?";
i don't see any reason to use AD_PInstance_ID. at that time we have AD_Process_ID, so better use AD_Process_ID
Environment
None
Activity
Anna Fadeeva
June 25, 2020 at 9:10 PM
I think that AD_PInstance_ID is a general entry point in the implementation of all processes, so it’s not worth changing anything here.
Hiep Lq
June 25, 2020 at 8:34 PM
this query just get data from AD_Process, it ever don't need to join
Anna Fadeeva
June 25, 2020 at 4:01 PM
AD_PInstance allready have AD_Process_ID.
If the process is started two or more times, then these are different AD_Pinstance_ID, the system will not understand, for example, in the report, in which window what data to show
ReportStart.getReportData() query report info by
"SELECT pr.JasperReport, pr.IsDirectPrint " + "FROM AD_Process pr, AD_PInstance pi " + "WHERE pr.AD_Process_ID = pi.AD_Process_ID " + " AND pi.AD_PInstance_ID=?";i don't see any reason to use AD_PInstance_ID. at that time we have AD_Process_ID, so better use AD_Process_ID