Zoom from QuickInfo to a specific window

Description

Since we create a zoom link from Quick Info widged.

This works weel for most default tables and windows, but in some cases that we have multiple windows for the same table - as an example we can mention Request and Request (All) - we get stuck as we are not able to define to wich window that the zoom action will open, using always the default.

Attached a patch that changes the code to allow a specific window ID to be defined when creating the zoom link. Note that this is optional, just if needed the window ID can be defined, if not, just use as the original ticket.

Example:

Environment

None

Activity

Show:
Carlos Ruiz
June 3, 2019, 2:47 PM

Thanks , this looks good, however I think to make it worthy for plugins we would need to work with the zoom window UUID instead of the ID.

Murilo Habermann Torquato
June 3, 2019, 7:38 PM

I agree with you , just adjusted a new patch

, the use is the same, but instedad of ID just inform 'UUID' on javascript.

I think that a lot of job related to UUID is needed into core (maybe a discussion first can guide us to better solutions and architecture)

for example on this i preferred to keep internal using ID and just implemented a method to get window ID by UU and keep the core without changes, in due of caching system and other lot of related implementations that trust on ID and is a hard work to change to UUID

Carlos Ruiz
June 4, 2019, 8:29 AM

For testing purposes:

Query with window parameter looks like:

The code from keeps working using zoom instead of zoomWindow.

Fixed

Assignee

Murilo Habermann Torquato

Reporter

Murilo Habermann Torquato

Labels

Tested By

None

Fix versions

Priority

Major