*** hieplq has joined #idempiere | 03:16 | |
*** norbertbede has joined #idempiere | 04:23 | |
*** hieplq has quit IRC | 04:51 | |
*** norbertbede has quit IRC | 05:35 | |
*** a42niem has joined #idempiere | 05:37 | |
*** norbertbede has joined #idempiere | 05:55 | |
*** tbayen_ has joined #idempiere | 06:08 | |
*** is-mw has joined #idempiere | 06:28 | |
*** nmicoud has joined #idempiere | 06:45 | |
*** norbertbede has quit IRC | 07:26 | |
*** norbertbede has joined #idempiere | 07:32 | |
*** norbertbede has joined #idempiere | 07:32 | |
*** norbertbede has quit IRC | 07:37 | |
*** norbertbede has joined #idempiere | 07:37 | |
*** norbertbede has quit IRC | 08:16 | |
*** norbertbede has joined #idempiere | 08:21 | |
*** norbertbede has quit IRC | 08:28 | |
*** norbertbede has joined #idempiere | 08:32 | |
*** norbertbede has quit IRC | 08:36 | |
*** norbertbede has joined #idempiere | 09:04 | |
*** norbertbede has quit IRC | 09:23 | |
*** norbertbede has joined #idempiere | 09:34 | |
*** norbertbede has quit IRC | 09:41 | |
*** norbertbede has joined #idempiere | 09:53 | |
*** norbertbede has quit IRC | 09:55 | |
*** KermitTheFragger has joined #idempiere | 10:11 | |
*** norbertbede has joined #idempiere | 10:12 | |
*** druiz has joined #idempiere | 12:23 | |
*** mhernandezve has joined #idempiere | 13:55 | |
*** norbertbede has quit IRC | 14:12 | |
*** CarlosRuiz has joined #idempiere | 14:54 | |
*** ChanServ sets mode: +o CarlosRuiz | 14:54 | |
*** druiz has quit IRC | 15:08 | |
*** KermitTheFragger has quit IRC | 15:14 | |
*** is-mw has quit IRC | 15:25 | |
*** tbayen has joined #idempiere | 15:25 | |
*** Not-604a has joined #idempiere | 15:35 | |
Not-604a | [IDEMPIERE] arthurmelo updated IDEMPIERE-2674 status set to "Peer Review Queue" | 15:35 |
---|---|---|
Not-604a | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2674 | 15:35 |
*** nmicoud has quit IRC | 15:40 | |
tbayen | CarlosRuiz, did diego talk with you about our jasperreports plugin issue? | 16:24 |
CarlosRuiz | nope | 16:24 |
tbayen | I want to replace the Jasperreports plugin with a better one. But that is not as easy as it should be. | 16:25 |
tbayen | We found different ways to do it and I want to ask your advice what is the best. | 16:25 |
tbayen | The problem is that the class ZkJRViewerProvider uses the JRViewerProvider interface (from org.adempiere.report.jasper plugin) and the JasperPrint class (from the jasperreports library that is included as a jar file in the org.adempiere.report.jasper.library plugin) | 16:27 |
tbayen | The result of that is that you can not use the Viewer in zk without using these both original plugins. | 16:28 |
tbayen | Solution 1) is that my new plugin uses the classes in the old (trunk) plugins. That means that we always have to have installed both. You can call that a waste of memory. | 16:29 |
CarlosRuiz | ZkJRViewerProvider is located as a service | 16:30 |
tbayen | Solution 2) is that we create a new trunk plugin that consists only of one class (the interface JRViewerProvider) and a dependency to a jasperreports osgi plugin from the MavenCentral repository.. That allows to share that "interface plugin" with all the different plugins. | 16:31 |
CarlosRuiz | so, if your plugin implement the service and give more priority must be found first? | 16:31 |
tbayen | Solution 3) is to implement the JRViewerProvider in org.adempiere.base. That means that the base plugin is dependent on the jasperreports osgi plugin (the one from MavenCentral). | 16:32 |
tbayen | The problem is that the service has to implement the JRViewerProvider. And this is implemented in org.adempiere.report.jasper. My new plugin has to be dependent on the old one to make that work. | 16:33 |
tbayen | Our goal is that our new plugin is better or equal than the old one in everything. When we do it right there is (near to) no reason to use the old one any more. | 16:34 |
tbayen | But as I expect us not to be perfect from day one I want to give the user the choice. (That is what OSGI is about) | 16:35 |
CarlosRuiz | As I understand your plugin is a replacement for org.adempiere.report.jasper | 16:39 |
tbayen | yes | 16:39 |
CarlosRuiz | seems easy if you just use a different starter class | 16:40 |
CarlosRuiz | but you would need to register that classname in the process definition | 16:41 |
tbayen | In principle it can also be used together. You can choose a different JasperStarter class in the AD_Process record to choose it. But I hope there will be no reason to use the old plugin any more. | 16:41 |
CarlosRuiz | JASPER_STARTER_CLASS is hardcoded | 16:42 |
CarlosRuiz | maybe we can make it a sysconfig | 16:42 |
tbayen | At the end of my plugin it has a JasperPrint object. And it needs to call a viewer to view that. The viewer is a service of the zk (or swing) plugin. And it implements JRViewerProvider. My question is: In which plugin is JRViewerProvider defined? | 16:43 |
tbayen | 1) in org.adempiere.report.jasper 2) in a very new plugin like org.adempiere.report.viewerinterface or 3) in org.adempiere.base or 4) ??? | 16:44 |
CarlosRuiz | making JASPER_STARTER_CLASS sysconfigurable could solve the issue to use a different class without touching AD_Process during the transition? | 16:44 |
tbayen | Yes, that could solve the transition. | 16:45 |
tbayen | But not my problem of the Viewer Interface. | 16:49 |
CarlosRuiz | I think that solves it too | 16:50 |
CarlosRuiz | cos the JASPER_STARTER_CLASS is calling the viewer | 16:50 |
CarlosRuiz | is self-contained there - no? | 16:50 |
tbayen | It calls the viewer through the interface. That means that the interface is needed. As it is defined in org.adempiere.report.jasper you need that plugin intalled to use it. | 16:57 |
tbayen | In the last year in my old version I solved it by creating the very same interface in my own plugin. That worked. But a) someone introduced new dependencies in the trunk that forbid to deinstall it and b) Diego wanted to try to use both plugins at the same time and two plugins with the same (but not the same) interface does not work and gives funny classloader exceptions. | 17:00 |
Not-604a | [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2674 status set to "Open" | 17:00 |
Not-604a | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2674 | 17:00 |
tbayen | I think it makes sense to be able to install both at the same time. That makes the transition easier. | 17:00 |
CarlosRuiz | tbayen, I mean - if you change the starter - you can call whatever viewer you prefer too | 17:13 |
*** aguerra has joined #idempiere | 17:13 | |
CarlosRuiz | I don't see the need to uninstall the core plugin - you just add a new one | 17:13 |
tbayen | That what we have working now. It is impossible to deinstall the old one. If that is waht you think we will keep that state and reopen the discussion in two years - when noone uses the old pligun any more. Hahaha! | 17:16 |
CarlosRuiz | yes - suppose is the same with 2pack - 2pack is pluggable | 17:29 |
CarlosRuiz | so you write a 6pack plugin | 17:30 |
CarlosRuiz | you can install it - no need to deinstall the 2pack - you just install one better and use it if needed | 17:30 |
CarlosRuiz | what I would expect if your plugin is backward compatible - can replace the core plugin | 17:30 |
CarlosRuiz | if is not - then we need to check what breaks and ask community | 17:31 |
*** CarlosRuiz has quit IRC | 18:20 | |
*** aguerra has quit IRC | 19:21 | |
*** a42niem has quit IRC | 22:08 | |
*** mhernandezve has quit IRC | 23:14 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!