#idempiere IRC log for Wednesday, 2019-09-11

Not[iDempiereDaily] jenkins build #90 started http://ci.idempiere.org/job/iDempiereDaily/90/02:08
Not[iDempiereDaily] jenkins build #90 completed: success http://ci.idempiere.org/job/iDempiereDaily/90/02:34
*** a42niem <a42niem!~niem@p578b7daf.dip0.t-ipconnect.de> has joined #idempiere05:21
*** nmicoud <nmicoud!~Nicolas@2a01:e34:eeb6:cb10:58b6:133d:6ef8:10e2> has joined #idempiere05:50
Not[IDEMPIERE] hengsin created IDEMPIERE-4047 Minor trx fix for GridTabCSVImporter07:06
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-404707:06
Not[iDempiere] 1 commit to default [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/07:08
Not[iDempiere] hengsin 6a6caab - IDEMPIERE-4047 Minor trx fix for GridTabCSVImporter07:08
Not[IDEMPIERE] hengsin updated IDEMPIERE-4047 status set to "Resolved" -Fix Version set to "6.2z" -resolution set to "Fixed"07:08
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-404707:08
Not[iDempiere] jenkins build #178 started http://ci.idempiere.org/job/iDempiere/178/07:21
Not[iDempiere] jenkins build #178 completed: failure http://ci.idempiere.org/job/iDempiere/178/07:25
*** CarlosRuiz <CarlosRuiz!~CarlosRui@ip-95-223-106-12.hsi16.unitymediagroup.de> has joined #idempiere09:13
nmicoudHello CarlosRuiz, wdyt of integrating SmartFinReport plugin6.2 in the trunk (see https://groups.google.com/forum/#!topic/idempiere/BhGMIaI8ZQo) ?11:39
CarlosRuizwe did a review recently12:05
CarlosRuizthe way it was developed make it not core friendly12:05
nmicoudI agree, it need to be rethinked from the start12:06
nmicoudidea would be to take all enhancements and add them in the "good" way12:06
CarlosRuizwe took a look to this plugin plus some add-ons for finreport that were contributed from adaxa to adempiere12:06
CarlosRuizthat was poorly integrated to adempiere12:07
CarlosRuizthe smartfinreport is integrated but is doesn't work unless you change some sysconfig I think - and in that case some other things may be broken probably12:07
CarlosRuizand some of the contributions from adaxa did the same things that some of the things from smartfinreport12:07
CarlosRuizand both added the same column with different name12:08
CarlosRuizthe changes from adaxa were very clear and integratable - Steven told me Deepak will work on integrating that to iDempiere - but I think maybe he hasn't found the time12:09
CarlosRuizfor the smartfinreport - after the review we decided to keep it as plugin12:10
nmicoudOk, so if I need some enhancements from smf, i integrate them in my own version, right ? no need to find a clean way to make it part of the trunk ?12:11
CarlosRuizif you find a way trunk-friendly - all those add-ons are great12:12
nmicoudSo maybe, add them one by one ?12:12
CarlosRuizI would try to start with the adaxa pieces first - have you seen that?12:12
nmicoudthat's not urgent for me as the customer do all those calculations outside of iDempiere, but that would be better if he can do it directly in12:13
nmicoudno, never head of12:13
CarlosRuizlast time I tried a finreport on adempiere - it showed nothing - so don't know really if is working or was wrongly integrated12:13
nmicoudor both :D12:13
nmicoudlast time I played with FinReport, it was on Compiere times (10 years ago). And I rediscover it some months ago, and that's a very powerful tool. But some tools are missing12:14
CarlosRuizfrom what I see is a cube viewwer12:15
CarlosRuizcube viewer12:15
nmicoudIs there a repository where I can find adaxa enhancements ?12:15
CarlosRuizlet me try to find my notes on the review12:16
nmicoudno need to hurry12:16
nmicoudwhen you have time12:17
CarlosRuizah yes - first complication we found the manual is gone12:17
CarlosRuizhttp://smartjsp.com/smartreport/AdvancedFinancialReportingForAdempiereV1.pdf12:17
nmicoudPeter write a new one (which seems to match with 'his' plugin)12:20
CarlosRuizthese notes are from Diego:12:20
CarlosRuiz"the plugin does not reuse the existing code in core for Reports and PDF/Excel creation, and instead, creates code that rewrites existing logic but make it in a way that is won't work for any other piece of the core, as it has a dependency on T_Report, so basically it duplicates code with some differences, one class for financial report another one for every other report in the system. In my opinion, this is a very bad12:20
CarlosRuizapproach and would be expensive to maintain and scalate in the future."12:20
CarlosRuizwhich Peter?12:20
nmicoudPeter Sheptenko (or something like that)12:20
CarlosRuizah ok12:20
CarlosRuizand my note after the review:12:21
CarlosRuiz"I wonder if would be easier to create some sort of styling and pass that information to jasper and let jasper do the work.  I think that option maybe is easier and more extensible to implement"12:21
nmicoudWhen I got the need to have percentage calcultation, he offers to publish the plugin he made for a previous version. And I offer to integrate it in the trunk (if is ok for you)12:22
CarlosRuizyes - percentage calculation is simple and doesn't require any PDF change12:22
nmicoudThat's why maybe the easier way to integrate those enhancements is : 1. list them ; 2. add them one by one12:23
nmicoudI got to go outside for a while, I should be back around 1512:29
CarlosRuizI can't find the adaxa stuff12:29
CarlosRuizwill note it here if I find it12:29
nmicoudok, thanks12:29
CarlosRuizwe reviewed the percentage stuff - it as modifying FinReport - not the PDF side - so is integrable12:31
CarlosRuizbut it uses Float for the calculations - that's a bad practice - it must be changed to BigDecimal12:31
CarlosRuizcouldn't find the adaxa changes12:40
Not[IDEMPIERE] hengsin created IDEMPIERE-4048 Fix maven build error12:56
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-404812:56
Not[IDEMPIERE] hengsin updated IDEMPIERE-4048 description set to "mvn verify -U error out on default branch [https://groups.google.com/d/topic/idempiere/6VHtkPh9Lo8/discussion|https://groups.google.com/d/topic/idempiere/6VHtkPh9Lo8/discussion]"12:59
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-404812:59
Not[IDEMPIERE] Heng Sin Low updated IDEMPIERE-4048 assignee set to "Heng Sin Low"13:00
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-404813:00
Not[iDempiere] 1 commit to default [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/13:04
Not[iDempiere] hengsin bd8f37d - IDEMPIERE-4048 Fix maven build error. Integrate solution from Ken Longnan13:04
Not[IDEMPIERE] Heng Sin Low updated IDEMPIERE-4048 status set to "Resolved" -Fix Version set to "6.2z" -resolution set to "Fixed"13:05
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-404813:05
nmicoudok, i'll make a patch for the percentage part (using BigDecimal).13:10
Not[iDempiere] jenkins build #179 started http://ci.idempiere.org/job/iDempiere/179/13:21
Not[iDempiere] jenkins build #179 completed: success http://ci.idempiere.org/job/iDempiere/179/13:29
*** nmicoud <nmicoud!~Nicolas@2a01:e34:eeb6:cb10:58b6:133d:6ef8:10e2> has quit IRC15:16
*** CarlosRuiz <CarlosRuiz!~CarlosRui@ip-95-223-106-12.hsi16.unitymediagroup.de> has quit IRC15:42
*** CarlosRuiz <CarlosRuiz!~CarlosRui@ip-95-223-106-12.hsi16.unitymediagroup.de> has joined #idempiere18:10
*** CarlosRuiz <CarlosRuiz!~CarlosRui@ip-95-223-106-12.hsi16.unitymediagroup.de> has quit IRC20:38
*** a42niem <a42niem!~niem@p578b7daf.dip0.t-ipconnect.de> has quit IRC21:26
Not[IDEMPIERE] Heng Sin Low created IDEMPIERE-4049 Add jersey jax-rs bundles to org.adempiere.target.platform.feature22:41
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-404922:41
Not[iDempiere] 1 commit to default [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/23:27
Not[iDempiere] hengsin 8e31c79 - IDEMPIERE-4049 Add jersey jax-rs bundles to org.adempiere.target.platform.feature23:27
Not[IDEMPIERE] Heng Sin Low updated IDEMPIERE-4049 status set to "Resolved" -Fix Version set to "6.2z" -resolution set to "Fixed"23:28
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-404923:28
Not[IDEMPIERE] Heng Sin Low updated IDEMPIERE-4006 assignee set to "Elaine Tan"23:34
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-400623:34
Not[iDempiere] jenkins build #180 started http://ci.idempiere.org/job/iDempiere/180/23:51

Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!