Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.5
    • Component/s: Plugin - JasperReports
    • Labels:
      None

      Description

      JasperReportsResult.java is not compatible with jasperreports 6.0

        Activity

        Hide
        julianhyde Julian Hyde added a comment -

        I saw you excluded olap4j. As of version 1.2 it is on maven central. With luck the jasperreports folks will upgrade to 1.2 soon.

        Show
        julianhyde Julian Hyde added a comment - I saw you excluded olap4j. As of version 1.2 it is on maven central . With luck the jasperreports folks will upgrade to 1.2 soon.
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Struts-JDK7-master #495 (See https://builds.apache.org/job/Struts-JDK7-master/495/)
        Patch WW-4381 (lukaszlenart: rev a446cdacc0f6d9168d03d7602a751b48ce066f1a)

        • plugins/jasperreports/pom.xml
        • plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/JasperReportsResult.java
          WW-4381 - exclude org.olap4j:olap4j for compile, to fix broken builds (lukaszlenart: rev dd849e82f37d938a788554b80393f752aaf0b3bf)
        • plugins/jasperreports/pom.xml
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Struts-JDK7-master #495 (See https://builds.apache.org/job/Struts-JDK7-master/495/ ) Patch WW-4381 (lukaszlenart: rev a446cdacc0f6d9168d03d7602a751b48ce066f1a) plugins/jasperreports/pom.xml plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/JasperReportsResult.java WW-4381 - exclude org.olap4j:olap4j for compile, to fix broken builds (lukaszlenart: rev dd849e82f37d938a788554b80393f752aaf0b3bf) plugins/jasperreports/pom.xml
        Hide
        rgielen Rene Gielen added a comment -

        excluded dependencies that are not served from Maven Central and that are not needed during build

        Show
        rgielen Rene Gielen added a comment - excluded dependencies that are not served from Maven Central and that are not needed during build
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Struts-JDK7-master #406 (See https://builds.apache.org/job/Struts-JDK7-master/406/)
        WW-4381 - exclude org.olap4j:olap4j for compile, to fix broken builds (rgielen: rev ee83d4a6dbd558ac4022194ffb419c8f46516f79)

        • plugins/jasperreports/pom.xml
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Struts-JDK7-master #406 (See https://builds.apache.org/job/Struts-JDK7-master/406/ ) WW-4381 - exclude org.olap4j:olap4j for compile, to fix broken builds (rgielen: rev ee83d4a6dbd558ac4022194ffb419c8f46516f79) plugins/jasperreports/pom.xml
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit ee83d4a6dbd558ac4022194ffb419c8f46516f79 in struts's branch refs/heads/master from Rene Gielen
        [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=ee83d4a ]

        WW-4381 - exclude org.olap4j:olap4j for compile, to fix broken builds behind Maven mirror

        Show
        jira-bot ASF subversion and git services added a comment - Commit ee83d4a6dbd558ac4022194ffb419c8f46516f79 in struts's branch refs/heads/master from Rene Gielen [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=ee83d4a ] WW-4381 - exclude org.olap4j:olap4j for compile, to fix broken builds behind Maven mirror
        Hide
        rgielen Rene Gielen added a comment -

        Jasper Reports 6 unfortunately introduces custom repositories in its pom to provide a source for org.olap4j:olap4j which cannot be found in an official repository. This break builds on systems behind a Maven mirror.

        The solution is to exclude this dependency, since it is not required at compile time. Jasper Reports is marked as provided within the plugin, such that the end user is responsible for providing it in his project build.

        Show
        rgielen Rene Gielen added a comment - Jasper Reports 6 unfortunately introduces custom repositories in its pom to provide a source for org.olap4j:olap4j which cannot be found in an official repository. This break builds on systems behind a Maven mirror. The solution is to exclude this dependency, since it is not required at compile time. Jasper Reports is marked as provided within the plugin, such that the end user is responsible for providing it in his project build.
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Struts-JDK7-master #397 (See https://builds.apache.org/job/Struts-JDK7-master/397/)
        Patch WW-4381 (victornsosa: rev e42954219a5a57d8ad7685f7ae9ffbf44367c81d)

        • plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/JasperReportsResult.java
        • plugins/jasperreports/pom.xml
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Struts-JDK7-master #397 (See https://builds.apache.org/job/Struts-JDK7-master/397/ ) Patch WW-4381 (victornsosa: rev e42954219a5a57d8ad7685f7ae9ffbf44367c81d) plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/JasperReportsResult.java plugins/jasperreports/pom.xml
        Hide
        lukaszlenart Lukasz Lenart added a comment -

        PR merged, thanks!

        Show
        lukaszlenart Lukasz Lenart added a comment - PR merged, thanks!
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/struts/pull/67

        Show
        githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/67
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 110d63fed07f5383504fa74021c7e8958baafad5 in struts's branch refs/heads/master from Lukasz Lenart
        [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=110d63f ]

        WW-4381 Upgrade to JasperReports 6.0

        Show
        jira-bot ASF subversion and git services added a comment - Commit 110d63fed07f5383504fa74021c7e8958baafad5 in struts's branch refs/heads/master from Lukasz Lenart [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=110d63f ] WW-4381 Upgrade to JasperReports 6.0
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit e42954219a5a57d8ad7685f7ae9ffbf44367c81d in struts's branch refs/heads/master from Victor Sosa
        [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=e429542 ]

        Patch WW-4381

        upgrade to jasperreports 6.0

        to 6.0.3

        Show
        jira-bot ASF subversion and git services added a comment - Commit e42954219a5a57d8ad7685f7ae9ffbf44367c81d in struts's branch refs/heads/master from Victor Sosa [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=e429542 ] Patch WW-4381 upgrade to jasperreports 6.0 to 6.0.3
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user victorsosa opened a pull request:

        https://github.com/apache/struts/pull/67

        Patch WW-4381 upgrade to jasperreports 6.0

        Also I accidentally revert the Patch WW-3462 so here is the fix again

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/victorsosa/struts jasperreports6.0

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/struts/pull/67.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #67


        commit 316b8a926e7bec85c83f69b2460a8b7d938d1686
        Author: Victor Sosa <victornsosa@gmail.com>
        Date: 2016-01-03T11:34:00Z

        Patch WW-3462

        connection open after report generation

        Now the connection is close after been used.

        commit 38da3f67051bc557312adb234424becf533c5f95
        Author: Victor Sosa <victornsosa@gmail.com>
        Date: 2016-01-03T11:36:34Z

        Merge branch 'master' of git@github.com:victorsosa/struts.git

        commit 0e111125d596e3da2ac52f5317e7417d353844e8
        Author: Victor Sosa <victornsosa@gmail.com>
        Date: 2016-01-03T12:05:03Z

        A proper way to close the db connection

        commit e42954219a5a57d8ad7685f7ae9ffbf44367c81d
        Author: Victor Sosa <victornsosa@gmail.com>
        Date: 2016-01-04T17:50:41Z

        Patch WW-4381

        upgrade to jasperreports 6.0

        to 6.0.3


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user victorsosa opened a pull request: https://github.com/apache/struts/pull/67 Patch WW-4381 upgrade to jasperreports 6.0 Also I accidentally revert the Patch WW-3462 so here is the fix again You can merge this pull request into a Git repository by running: $ git pull https://github.com/victorsosa/struts jasperreports6.0 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/struts/pull/67.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #67 commit 316b8a926e7bec85c83f69b2460a8b7d938d1686 Author: Victor Sosa <victornsosa@gmail.com> Date: 2016-01-03T11:34:00Z Patch WW-3462 connection open after report generation Now the connection is close after been used. commit 38da3f67051bc557312adb234424becf533c5f95 Author: Victor Sosa <victornsosa@gmail.com> Date: 2016-01-03T11:36:34Z Merge branch 'master' of git@github.com:victorsosa/struts.git commit 0e111125d596e3da2ac52f5317e7417d353844e8 Author: Victor Sosa <victornsosa@gmail.com> Date: 2016-01-03T12:05:03Z A proper way to close the db connection commit e42954219a5a57d8ad7685f7ae9ffbf44367c81d Author: Victor Sosa <victornsosa@gmail.com> Date: 2016-01-04T17:50:41Z Patch WW-4381 upgrade to jasperreports 6.0 to 6.0.3
        Hide
        quaff Yanming Zhou added a comment -

        proposal steps:
        1. update pom to net.sf.jasperreports:jasperreports:6.0.3
        2. replace "JRLoader.loadObject(systemId)" with "JRLoader.loadObject(new File(systemId))" at line 325 in JasperReportsResult.java

        Show
        quaff Yanming Zhou added a comment - proposal steps: 1. update pom to net.sf.jasperreports:jasperreports:6.0.3 2. replace "JRLoader.loadObject(systemId)" with "JRLoader.loadObject(new File(systemId))" at line 325 in JasperReportsResult.java

          People

          • Assignee:
            rgielen Rene Gielen
            Reporter:
            quaff Yanming Zhou
          • Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development