Atlassian uses cookies to improve your browsing experience, perform analytics and research, and conduct advertising. Accept all cookies to indicate that you agree to our use of cookies on your device. Atlassian cookies and tracking notice, (opens new window)
iDempiere JIRA
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