Zoom condition fixes
Description
Environment
Attachments
discovered while testing
Activity
Carlos Ruiz August 21, 2013 at 1:53 PM
Thanks Nicolas, I see Heng Sin fixed it at commit https://bitbucket.org/idempiere/idempiere/commits/319a8da
Carlos Ruiz August 10, 2013 at 12:17 AM
Sorry again - do you mean the little zoom button on the WFEditor that zoom to the workflow window?
Nicolas Micoud August 9, 2013 at 9:32 PM
Hi,
The method zoom of WFEditor zoomes on the default window (definedon AD_Table.Ad_Window) whereas it should check in the Zoom Condition table.
I have not Eclipse there but i think that the AEnv.zoom method should be modified.
Regards,
Nicolas
Carlos Ruiz August 9, 2013 at 4:02 PM
Please peer review
https://bitbucket.org/trekglobal/idempiere/commits/cf39811
Fixes:
created UUID and index and filled official values
created zoom condition for RMA documents on M_InOut
allowed to use context variables on zoom condition (so you can define for example a zoom condition for "@#AD_Role_ID@=102"
fixed the zoomable link on zk when no access to direct zoom window
@Nicolas, how is the test case for:
"the new table is not used when zooming from WFEditor" ?
Regards,
Carlos Ruiz
Nicolas Micoud August 7, 2013 at 12:02 PM
Another bug related to role.
Steps :
Duplicate a window (eg: Organization).
For a given role :
give access only to the new window
remove access to the 'old' window
Then, you won't be able to zoom on Organization (Org field won't be underlined)
If you give access to the old window again, the zoom can be done.
Seems that access rights are only based on the main window (which is set on AD_Table) whereas it should be done on the new AD_ZoomCondition.
WDYT ?
Regards,
Nicolas
Hi,
I've tested ticket 528 (Zoom Condition) and found out 2 things :
AD_ZoomCondition_UU column is missing
the new table is not used when zooming from WFEditor
Regards,
Nicolas