Uploaded image for project: 'WINDUP - Red Hat Application Migration Toolkit'
  1. WINDUP - Red Hat Application Migration Toolkit
  2. WINDUP-3668

JPA XML discovery: add further namespaces

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Done
    • Icon: Undefined Undefined
    • MTR-1.0.2
    • None
    • None
    • None
    • False
    • None
    • False
    • Hide
      * *Why we missed the bug?*
       ** Pick to proper answer from drop-down field upper.
       +_Additional comments:_+


       * *What is required:*
       ** Pick to proper answer from drop-down field upper.
       +_Additional comments:_+
      Show
      * *Why we missed the bug?*  ** Pick to proper answer from drop-down field upper.  +_Additional comments:_+  * *What is required:*  ** Pick to proper answer from drop-down field upper.  +_Additional comments:_+
    • ---
    • ---
    • AMM Sprint 232, AMM Sprint 231
    • None

      Currently, the "DiscoverJpaConfigurationXmlRuleProvider" rule searches only for the "http://java.sun.com/xml/ns/persistence" namespace [0].
      As stated in [1]

      Starting with the 2.1 version, the Java Persistence API Schemas share the namespace, http://xmlns.jcp.org/xml/ns/persistence/. Previous versions used the namespace http://java.sun.com/xml/ns/persistence/
      

      Lately, with Jakarta EE, the "https://jakarta.ee/xml/ns/persistence" namespace has been added as well.

      So namespace's URIs:

      must be added to that rule's when condition.

      [0] https://github.com/windup/windup/blob/72135fa04aa7a70b80ab38cfc9b9e6e1de4d4842/rules-java-ee/addon/src/main/java/org/jboss/windup/rules/apps/javaee/rules/DiscoverJpaConfigurationXmlRuleProvider.java
      [1] http://www.oracle.com/webfolder/technetwork/jsc/xml/ns/persistence/index.html#1.0

              mrizzi@redhat.com Marco Rizzi
              mrizzi@redhat.com Marco Rizzi
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: