Process saved params doesn't update Display and ReadOnly Logics

Description

Hi,

Steps to reproduce :
Add a process with 2 params :
1 : checkbox (Name = Is)
2 : date with display logic @Is@ = Y

Then execute the process.
Tick the checkbox and fill datebo with a value.
Click on the execute button.

Now open the process again and select the last saved params item.
The checkbox will be ticked but the datebox will not be displayed (and it has a value, which could lead to issue if it is not tested in code)

Same happens with Read Only Logic

I created an example in test.idempiere.com

Regards,

Nicolas

Environment

None

Attachments

1

Activity

Nicolas Micoud 
June 3, 2015 at 1:50 PM

; ok tested again and it works !

But here another use case that failed :

  • remove the display logic form C_Currency_ID

  • add display logic @DateAcct@ = Y for IsSOTrx

Then launch the process, check Account Date and execute
Open the report again and select last param.

Account Date will be checked but IsSOTrx will remain hidden

Carlos Ruiz 
June 3, 2015 at 1:32 PM

, I just packed in RV_T_Aging_IDEMPIERE-2361.zip and the field currency disappear after I select the last saved param.

Nicolas Micoud 
May 29, 2015 at 7:35 PM

Hi ,

I think i found something interesting

When you call the valueChange method, it uses the 'New' value to call the processNewValue method
But this value is always null

I've modifier a little you patch by replacing :

by

I have not the time ATM to test it further, but it seems to work in my use case

wdyt ?

Thanks

Nicolas

Nicolas Micoud 
May 28, 2015 at 7:33 AM

Install the 2Pack
Launch the process
Untick 'Sales transation' (Currency field will disappear) and execute
Launch the process again (from the menu)
Select the last param : Sales transaction will be automatically unticked but Currency field will still be shown

Nicolas Micoud 
May 28, 2015 at 7:32 AM

Hi Carlos,
Issue is still there
I've created a 2Pack for testing purpose
Trying to fix :-/

Regards,

Nicolas

Fixed

Details

Assignee

Reporter

Priority

Created May 11, 2015 at 7:41 AM
Updated August 1, 2015 at 2:21 PM
Resolved June 3, 2015 at 3:31 PM