Uploaded image for project: 'JBRULES'
  1. JBRULES
  2. JBRULES-2840

Package compilation should present an error or warning when a file inside a directory does not declare itself to be in the same package as the other resources in that directory

This issue belongs to an archived project. You can view it, but you can't modify it. Learn more

    XMLWordPrintable

Details

    • Feature Request
    • Resolution: Done
    • Major
    • 5.2.0.M1
    • 5.1.0.FINAL
    • drools-ant, drools-api
    • None

    Description

      This is a request for an error/warning mechanism in the compiler task. This case presents itself when a person defines a package and makes a unintentional mistake by not defining all the files which are intended to be in the package to be part of the package. If familiarity with drools is low this issue can cause a lot of frustration from not being able to find the issue.

      If a package is composed of more than one rules source file (i.e a drl + xls + rf) there is an unreported error where the files do not all declare themselves to be of the same package. The compiler will only build the package to include some of the files.

      Given 3 files in the same directory to be compiled using the drools-ant task

      org.drools.contrib.DroolsCompilerAntTask

      process.rf [package rsarules.quote]
      start-process.drl [package rssrules.quote]
      quote.xls [package rsarules.quote]

      using

      <target name="buildPackage" >
      <compiler srcdir="${basedir}/src/test/resources/rules"
      tofile="${basedir}/src/test/resources/rsa.rules.pkg"
      classpathref="rsa.classpath"
      binformat="package" >
      <include name="*.xls"/>
      <include name="*.drl"/>
      <include name="*.rf"/>
      </compiler>
      </target>

      In the above example if start-process.drl was read first the resultant package would only contain the start-process.drl rule. If on the other hand the compiler was to read any of the other two files first the package created would not contain start-process.drl. In both cases the resultant rules application would not function as was intended. This is clearly an error but the compiler should flag this.

      Attachments

        Activity

          People

            rhn-support-tsurdilo Tihomir Surdilovic (Inactive)
            rhn-support-alazarot Alessandro Lazarotti
            Archiver:
            rhn-support-ceverson Clark Everson

            Dates

              Created:
              Updated:
              Resolved:
              Archived:

              PagerDuty