Uploaded image for project: 'Drools'
  1. Drools
  2. DROOLS-1335

Classpath is not checked for Kie Module ReleaseId

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • Major
    • 7.0.0.Beta3
    • 6.4.0.Final, 6.5.0.CR2, 7.0.0.Beta2
    • kie server
    • None
    • Hide

      In a CDI container,

      • have the following:
        @Inject
        @KReleaseId(groupId = "org.group", artifactId = "artifact", version = "1.0")
        @KSession("defaultSession")
        private KieSession kSession;
      • have org.group:artifact:1.0 on the classpath without depending on kie-ci.

      You will notice that kSession is not injected.

      Show
      In a CDI container, have the following: @Inject @KReleaseId(groupId = "org.group", artifactId = "artifact", version = "1.0") @KSession("defaultSession") private KieSession kSession; have org.group:artifact:1.0 on the classpath without depending on kie-ci. You will notice that kSession is not injected.
    • NEW
    • NEW

    Description

      Injecting a ksession annotated with a particular @KReleaseId is not able to read the passed release id from the classpath (the part that does that is not implemented).

      The bug is here: https://github.com/droolsjbpm/drools/blob/6.4.0.Final/drools-compiler/src/main/java/org/drools/compiler/kie/builder/impl/KieRepositoryImpl.java#L123-L126

      Patch attached.

      Attachments

        Issue Links

          is related to

          Bug - A problem that impairs or prevents the functions of the product. DROOLS-1351 Calling KieRepository.getKieModule() for a GAV of a non KJar throws Exception instead of returning null as per API contract.

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Closed

          Activity

            People

              mfusco@redhat.com Mario Fusco
              chadiem Chadi El Masri (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: