Details

    • Type: Task Task
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: None
    • Component/s: framework, order
    • Labels:
      None

      Description

      In OFBIZ there is only two Jasper Reports in the order component and both of them are for demonstration purposes only contributed by Britton LaRoche.
      For run those two reports you have to manually execute the request orderreportlist from the order report application.
      They are the only ones reports remaining that still use Jasper Report and so I propose this patch to remove those from the SVN.
      If this patch will be applied to SVN the only reference to Jasper Report that still remains in OFBIZ are:

      • JasperReportsPdfViewHandler.java
      • JasperReportsXmlViewHandler.java

      Does it make sense for you to remove from OPTIONAL_LIBRARIES the reference to JasperReports ?

      Thanks in advance
      Marco

        Activity

        Hide
        Jacques Le Roux added a comment -

        A comment from Erwan taken from dev ML (added here for convenience link from FAQ)

        You have to add the jasper reports libraries
        (jasperreports-3.6.0-applet.jar and jasperreports-3.6.0.jar) in
        framework/webapp/lib, and update your .classpath file with

        <classpathentry kind="lib"  path="framework/webapp/lib/jasperreports-3.6.0.jar"/>
        <classpathentry kind="lib"  path="framework/webapp/lib/jasperreports-3.6.0-applet.jar"/>
        

        then you should be able to access the librairies directly in your java code.

        The reason why Jasper Report was removed from OFBiz is an incompatibility of licences (Apache2 and GPL)

        Show
        Jacques Le Roux added a comment - A comment from Erwan taken from dev ML (added here for convenience link from FAQ) You have to add the jasper reports libraries (jasperreports-3.6.0-applet.jar and jasperreports-3.6.0.jar) in framework/webapp/lib, and update your .classpath file with <classpathentry kind= "lib" path= "framework/webapp/lib/jasperreports-3.6.0.jar" /> <classpathentry kind= "lib" path= "framework/webapp/lib/jasperreports-3.6.0-applet.jar" /> then you should be able to access the librairies directly in your java code. The reason why Jasper Report was removed from OFBiz is an incompatibility of licences (Apache2 and GPL)
        Hide
        Chris Howe added a comment -

        How 'bout moving these examples to the examples component?

        Show
        Chris Howe added a comment - How 'bout moving these examples to the examples component?
        Hide
        David E. Jones added a comment -

        Somewhere or other we need some example of how to use JasperReports "the OFBiz way", ie using the OFBiz integration. If these are all that's left, let's leave them.

        Show
        David E. Jones added a comment - Somewhere or other we need some example of how to use JasperReports "the OFBiz way", ie using the OFBiz integration. If these are all that's left, let's leave them.
        Hide
        Marco Risaliti added a comment -

        Hi Jacopo - David,

        first of all thanks a lot for your work, I have only one question about this issue.
        Why don't we remove also the following files from SVN.

        applications\order\webapp\ordermgr\reports\orderitemreport.xml
        applications\order\webapp\ordermgr\reports\orderreport.xml
        applications\order\webapp\ordermgr\reports\reportlist.ftl
        applications\order\webapp\ordermgr\WEB-INF\actions\reports.bsh
        applications\order\widget\ordermgr\OrderReportScreens.xml

        They are demonstration purpose only and they are also not working because they have problems into the definition (not correct band-height) and one missing field (shipCarrier).
        They are also has been replaced by the new FOP reports into the order application.

        Are you agree to remove those reports and forms that now also can be called because the view/request on the controller.xml has been removed.

        Thanks a lot
        Marco

        Show
        Marco Risaliti added a comment - Hi Jacopo - David, first of all thanks a lot for your work, I have only one question about this issue. Why don't we remove also the following files from SVN. applications\order\webapp\ordermgr\reports\orderitemreport.xml applications\order\webapp\ordermgr\reports\orderreport.xml applications\order\webapp\ordermgr\reports\reportlist.ftl applications\order\webapp\ordermgr\WEB-INF\actions\reports.bsh applications\order\widget\ordermgr\OrderReportScreens.xml They are demonstration purpose only and they are also not working because they have problems into the definition (not correct band-height) and one missing field (shipCarrier). They are also has been replaced by the new FOP reports into the order application. Are you agree to remove those reports and forms that now also can be called because the view/request on the controller.xml has been removed. Thanks a lot Marco
        Hide
        Jacopo Cappellato added a comment -

        I've commented out the controller's entries in rev. 468304

        Show
        Jacopo Cappellato added a comment - I've commented out the controller's entries in rev. 468304
        Hide
        David E. Jones added a comment -

        I'd say there is a very good chance that people will want to use JasperReports with OFBiz. Because of that I'd really rather see these things stay in SVN, but commenting out the controller entries to avoid the startup warnings would be fine, and maybe even a good idea.

        Show
        David E. Jones added a comment - I'd say there is a very good chance that people will want to use JasperReports with OFBiz. Because of that I'd really rather see these things stay in SVN, but commenting out the controller entries to avoid the startup warnings would be fine, and maybe even a good idea.
        Hide
        Jacopo Cappellato added a comment -

        Marco,

        I think that, even if not currently used by any of the standard OFBiz's applications, it's nice to have the JasperReport plugin as an optional tool.
        So I'd leave as is the note in the OPTIONAL_LIBRARIES file and the two JasperReport classes (JasperReportsPdfViewHandler.java, JasperReportsXmlViewHandler.java).

        About the two reports in the order component: it would be nice to cleanup the JasperReport entries from the controller (in fact that reports are really just examples of how to create a report with such tools) but I don't know if we should actually remove the files from SVN or just leave them in and add a comment in them that explains what these files are...

        Opinions?

        Jacopo

        Show
        Jacopo Cappellato added a comment - Marco, I think that, even if not currently used by any of the standard OFBiz's applications, it's nice to have the JasperReport plugin as an optional tool. So I'd leave as is the note in the OPTIONAL_LIBRARIES file and the two JasperReport classes (JasperReportsPdfViewHandler.java, JasperReportsXmlViewHandler.java). About the two reports in the order component: it would be nice to cleanup the JasperReport entries from the controller (in fact that reports are really just examples of how to create a report with such tools) but I don't know if we should actually remove the files from SVN or just leave them in and add a comment in them that explains what these files are... Opinions? Jacopo

          People

          • Assignee:
            Jacopo Cappellato
            Reporter:
            Marco Risaliti
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development