Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-8615

Wrong commons-collections version when depending on rice-kim-api

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2.0-m4, 2.2
    • Fix Version/s: 2.1.3
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-8616Wrong slf4j-log4j12 version resolved when depending on rice-core-impl
      KULRICE-13066move our code to commons-collections4, make commons-collections a runtime dependency
      KULRICE-6353Add explicit dependency to asm version 3.3.1 in ksb-client-impl pom
      KULRICE-1794workflow depends on unauthorized jsp being in wrong location when running with workflow web content embedded
      KULRICE-6967Rice impl and krad depend on different versions of slf4j
      KULRICE-1417Add BCEL 5.1 to dependencies
      KULRICE-8499Rice version is wrong in installation guide documentation for maven archetype (section 5, maven cli)
      KULRICE-12842RoleDaoJdbc generating wrong SQL when null qualifier passed in.
      KULRICE-7111dependency issues on asm-1.5.3
    • Rice Module:
      KIM
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      If a project depends on rice-kim-api, an institutional kim impl for example, an older dependency of commons-collections (3.1) will be resolved by maven. This causes compilation errors. The kim api code uses commons-collections classes, but it doesn't declare a dependency (it gets the older version from db-ojb). While the proper version is declared in the dependencyManagement of the Rice parent, this doesn't seem to affect external projects. Adding commons-collections as an explicit dependency seems warranted and will resolve the problem for external projects.

        Activity

        Hide
        Peter Giles (Inactive) added a comment -

        Corey, can you check if this issue affects 2.1.3 as well? If so we'll fix it and merge it over to trunk.

        Show
        Peter Giles (Inactive) added a comment - Corey, can you check if this issue affects 2.1.3 as well? If so we'll fix it and merge it over to trunk.
        Hide
        Corey Pedersen (Inactive) added a comment -

        Currently if your project just has a dependency on the rice-kim-api module, maven will load commons-collections:3.1. If you include another module with commons-collections dependency defined (such as rice-ksb-api) maven will load common-collections:3.2.1. To utilize a consistant and common current version in rice-kim-api, a dependency should be added which will inherit it's version number from the rice parent.

        Show
        Corey Pedersen (Inactive) added a comment - Currently if your project just has a dependency on the rice-kim-api module, maven will load commons-collections:3.1. If you include another module with commons-collections dependency defined (such as rice-ksb-api) maven will load common-collections:3.2.1. To utilize a consistant and common current version in rice-kim-api, a dependency should be added which will inherit it's version number from the rice parent.
        Hide
        Corey Pedersen (Inactive) added a comment -

        Peter: This IS true in 2.1.3 and the change will be committed there.

        Show
        Corey Pedersen (Inactive) added a comment - Peter: This IS true in 2.1.3 and the change will be committed there.
        Hide
        Corey Pedersen (Inactive) added a comment -

        Committed revision 36174.

        Show
        Corey Pedersen (Inactive) added a comment - Committed revision 36174.

          People

          • Assignee:
            Corey Pedersen (Inactive)
            Reporter:
            Scott Gibson (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Structure Helper Panel