Anonymous | Login

Project:

News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ]

[ Issue History ] [ Print ]

ID

0036281

Type

Category

Severity

Reproducibility

Date Submitted

Last Update

feature request

[Openbravo ERP] A. Platform

minor

have not tried

2017-06-19 18:33

2018-01-18 12:34

Reporter

shuehner

View Status

public

Assigned To

jarmendariz

Priority

normal

Resolution

open

Fixed in Version

Status

acknowledged

Fix in branch

Fixed in SCM revision

Projection

none

ETA

none

Target Version

OS

Any

Database

Any

Java version

OS Version

Database version

Ant version

Product Version

SCM revision

Review Assigned To

Web browser

Modules

Core

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

No

Summary

0036281: "Grid Legacy" warning should be moved from runtime to compile time

Description

Warning about 'Grid Legacy' asking for updating the process definition+code is currently shown only when running the process via BaseProcessActionHandler.

That makes is very hard to spot all processes in need for conversion.

The simple part which can be checked with looking at actual request values should be moved to compile time to notice when compiling or in ci all the processes

Quick check of the code shows an sql similar to the following seems cover:
a.) the basic check about legacy
b.) the 'can be safely marked non-legacy part also

select m.javapackage, p.obuiapp_process_id, p.name, (select count(*) from obuiapp_parameter param where param.ad_reference_id = 'FF80818132D8F0F30132D9BC395D0038' and param.obuiapp_process_id = p.obuiapp_process_id) as grid_param from obuiapp_process p, ad_module m where p.ad_module_id = m.ad_module_id and p.isgridlegacy = 'Y' order by m.javapackage, p.name;

Note: the sub-select covers checking the defined grid-parameters similar to the check in BaseProcessActionHandler (without the 'request' part which is unclear if needed to be checked for this)

Steps To Reproduce

-

Tags

No tags attached.

Relationships [ Relation Graph ] [ Dependency Graph ]

related to	defect	0036282	3.0PR17Q4	closed	vmromanos	Openbravo ERP	Several processes are marked as 'Grid Legacy' and should be converted to avoid runtime warning on every use
related to	defect	0036283		closed	ranjith_qualiantech_com	Retail Modules	Several processes are marked as 'Grid Legacy' and should be converted to avoid runtime warning on every use
related to	defect	0036284		closed	ranjith_qualiantech_com	Retail Modules	Several processes are marked as 'Grid Legacy' and should be converted to avoid runtime warning on every use
related to	defect	0036285		closed	vmromanos	Openbravo ERP	Several processes are marked as 'Grid Legacy' and should be converted to avoid runtime warning on every use
related to	defect	0036286		closed	jarmendariz	Openbravo ERP	Several processes are marked as 'Grid Legacy' and should be converted to avoid runtime warning on every use
blocks	design defect	0036162		acknowledged	Triage Platform Base	Openbravo ERP	clean up openbravo.log

Notes
There are no notes attached to this issue.

Issue History
Date Modified	Username	Field	Change
2017-06-19 18:33	shuehner	New Issue
2017-06-19 18:33	shuehner	Assigned To	=> platform
2017-06-19 18:33	shuehner	Modules	=> Core
2017-06-19 18:33	shuehner	Triggers an Emergency Pack	=> No
2017-06-19 18:35	shuehner	Relationship added	related to 0036282
2017-06-19 18:36	shuehner	Relationship added	related to 0036283
2017-06-19 18:41	shuehner	Relationship added	related to 0036284
2017-06-19 18:43	shuehner	Relationship added	related to 0036285
2017-06-19 18:45	shuehner	Relationship added	related to 0036286
2017-06-20 09:26	alostale	Status	new => acknowledged
2017-06-20 09:27	alostale	Relationship added	blocks 0036162
2018-01-18 12:34	jarmendariz	Assigned To	platform => jarmendariz

Copyright © 2000 - 2009 MantisBT Group