Wrong type when get AdWindow instance
Description
Environment
Attachments
- 11 May 2016, 03:45 PM
- 02 Mar 2016, 07:41 PM
Activity
Hiep Lq May 11, 2016 at 5:20 PM
i just re-checked it.
when apply to core, just apply for class WindowValidatorManager
code in ADTabpanel is my customize code to implement https://idempiere.atlassian.net/browse/IDEMPIERE-2676#icft=IDEMPIERE-2676
now don't have any case to procedure issue but it's a implicit issue.
Hiep Lq May 11, 2016 at 4:04 PM
in my repository i have both. but maybe something out date when i merger with core. let me check on a clear idempiere code. i will give you correct patch. now please go with other issue.
Carlos Ruiz May 11, 2016 at 3:58 PM
No, I tried that and is working fine.
I see the first part of your patch can't be merged, the ADTabPanel doesn't have that code now.
For the WindowValidatorManager part, you cannot add events on non-dictionary windows, but still would be worthy as safety against NPE.
Is it ok to integrate just second part? Or first is needed somewhere else?
Hiep Lq May 11, 2016 at 3:55 PM
if i correct, you just open any dialog to set up accounting combine.
example:Business Partner/Customer Accounting field "Customer Receivables"
Carlos Ruiz May 11, 2016 at 3:47 PM
Sorry @Hiep Lq, how to reproduce "NPE when open accout dialog from any account tab"
WQuickEntry registry like ADWindow (ref:constructor of WQuickEntryublic WQuickEntry(int WindowNo, int AD_Window_ID))
so function ADWindow.get (window_no) will raise cast exception sometime