0022902: Create a "Print" link button to be used within QueryList widget

The test case 2 is an advanced one just for demonstration purposes (I don't think nobody is going to use it) if you set the things as are in the screenshot, it should work. It doesn't work for you, or you mean another different thing?

About the other comments:

1) There could be the case of isHasLink, but no "printUrl" for a particular tab, and I don't think have log writting each time it doesn't have "printUrl" be a good idea... It could sound like a warning, when it isn't.
In fact, my idea of getPrintUrl was: give me the url if exists or "" (or null) if it doesn't exist, but without any special emphasis if there is no printUrl for this particular tab. Anyway, no strong opinion on this point

2) I tried, but if I avoid directly cPopup.show(), some logic (the iframe code addition hack) doesn't work (several JS erros appear), so the show() needs to be called.

3) isHasLink is the field "Has Link". It is used before for example to determine that a field is going to be a link to its window (determined by the "Tab" combo box). For the particular case of the print button, it needs all the parameters sent by the isHasLink check
https://code.openbravo.com/erp/devel/pi/file/603290cb6a8a/modules/org.openbravo.client.querylist/src/org/openbravo/client/querylist/QueryListUtils.java#l67 [^]
but also I need the "printURL". Since a user can create any other kind of print link clientClass (for example, one with a icon button with a printer instead of the link), I have forced to retreive always this parameter too (when clientClass + hasLink are both present).
I understand your point of add another check "is Print" to have this "printUrl" only in these cases, but I think is make the model more complex for a feature (the new Clientcanvas field) that will be used in the 90% of the cases by adding a Print button, so I would remain it as it is

4) Is not a typo. If you look at this line, it was "Command" also before my changes:
https://code.openbravo.com/erp/devel/pi/rev/603290cb6a8ae0459ac4a2c34d7bea0a161eed51#l6.33 [^]
In fact, I tried to change it to "Command", but then a security error raises. (You can try yourself) Since it is an old implementation and it will be refactored, I prefer leave it as it is (as it was) and don't complicate searching the root problem (it should be another different issue)