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

Activity

Show:
Nicolas Micoud
May 28, 2015, 7:32 AM

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

Regards,

Nicolas

Nicolas Micoud
May 28, 2015, 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 29, 2015, 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

Carlos Ruiz
June 3, 2015, 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
June 3, 2015, 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

Fixed

Assignee

Carlos Ruiz

Reporter

Nicolas Micoud

Labels

None

Tested By

None

Priority

Major