baldimir commented on issue #403:
URL: 
https://github.com/apache/incubator-kie-issues/issues/403#issuecomment-2039239829

   When I model a decision (from scratch using the new DMN editor), that uses 
an imported BKM, imported using the empty name (1.5 feature), it also doesn't 
work with the runner. I see the potential problem could be, how the editor adds 
the knowledgeRequirement: 
   ```
   <knowledgeRequirement id="_F4D15B4D-63AF-4956-9C39-96A1D2334146">
       <requiredKnowledge 
href="https://kie.org/dmn/_CDC1C967-6384-4BA9-989A-AC5385DDDFF7#_05703942-965D-4D2A-9579-E23A91DBB373";
 />
   </knowledgeRequirement>
   ```
   It is added with the namespace prefix (namespace before #). The namespace 
shouldn't be there, because the import with empty name imports into the default 
namespace of the model. This is from spec: 
   
   > When the import name attribute is an empty string, the elements are 
imported in the default namespace of the
   model. When a name collision occurs between an element in the default 
namespace and an imported element,
   the imported element does not replace the one already in the default 
namespace while the elements without
   name collision are imported.
   
   So it should be used like this:
   ```
   <knowledgeRequirement id="_F4D15B4D-63AF-4956-9C39-96A1D2334146">
       <requiredKnowledge href="#_05703942-965D-4D2A-9579-E23A91DBB373" />
   </knowledgeRequirement>
   ```
   @tiagobento @jomarko @ljmotta, should I file this as a separate issue 
please? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to