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

Fixed

Details

Assignee

Reporter

Labels

Tested By

Anna Fadeeva

Components

Fix versions

Priority

Created December 27, 2018 at 1:55 PM
Updated August 1, 2021 at 11:45 AM
Resolved June 28, 2021 at 11:58 AM