Zoom condition fixes

Description

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

Environment

None

Attachments

2

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

Fixed

Details

Assignee

Reporter

Labels

Fix versions

Priority

Created August 1, 2013 at 9:53 AM
Updated March 13, 2014 at 2:21 PM
Resolved August 12, 2013 at 6:42 AM