Uploaded image for project: 'Spring Web Services'
  1. Spring Web Services
  2. SWS-353

Schema inlining results in duplicate inlining of files referenced from multiple files

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.1
    • Fix Version/s: 1.5.2
    • Component/s: Core
    • Labels:
      None
    • Environment:
      Any

      Description

      Suppose you have two files foo-response.xsd and bar-response.xsd referenceing a common domain.xsd the Dynamic WSDL creation will cause the domain.xsd to be inlined twice in the WSDL.

      This causes trouble to JAX-WS wsimport utility as elements appera twice in the same namespace.

        Activity

        olivergierke Oliver Gierke created issue -
        Hide
        olivergierke Oliver Gierke added a comment -

        Added patched version of CommonsXsdSchemaCollection that does not include or import external XSDs twice.

        Show
        olivergierke Oliver Gierke added a comment - Added patched version of CommonsXsdSchemaCollection that does not include or import external XSDs twice.
        olivergierke Oliver Gierke made changes -
        Field Original Value New Value
        Attachment CommonsXsdSchemaCollection.java [ 14040 ]
        arjen.poutsma Arjen Poutsma made changes -
        Fix Version/s 1.5.2 [ 10966 ]
        Hide
        arjen.poutsma Arjen Poutsma added a comment -

        Fixed, thanks for the patch!

        Show
        arjen.poutsma Arjen Poutsma added a comment - Fixed, thanks for the patch!
        arjen.poutsma Arjen Poutsma made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Hide
        arjen.poutsma Arjen Poutsma added a comment -

        One more thing, Oliver. Since you stripped the Apache preamble from the patch, I have to ask you explicitly:

        Do you agree to share the attached patch under the terms of the Apache License, version 2?

        Show
        arjen.poutsma Arjen Poutsma added a comment - One more thing, Oliver. Since you stripped the Apache preamble from the patch, I have to ask you explicitly: Do you agree to share the attached patch under the terms of the Apache License, version 2?
        Hide
        olivergierke Oliver Gierke added a comment -

        Excuse me, Arjen. Of course I do. I tried to fix the issue by subclassing at first for the project i needed the functionality for but ended up copyng the entire class as extending it didn't work as well as expected. By copying the entire code I seem to have missed the license declaration.

        Regards,
        Ollie

        Show
        olivergierke Oliver Gierke added a comment - Excuse me, Arjen. Of course I do. I tried to fix the issue by subclassing at first for the project i needed the functionality for but ended up copyng the entire class as extending it didn't work as well as expected. By copying the entire code I seem to have missed the license declaration. Regards, Ollie
        Hide
        arjen.poutsma Arjen Poutsma added a comment -

        Closing issues in 1.5.2

        Show
        arjen.poutsma Arjen Poutsma added a comment - Closing issues in 1.5.2
        arjen.poutsma Arjen Poutsma made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        18h 40m 1 Arjen Poutsma 08/May/08 10:13 PM
        Resolved Resolved Closed Closed
        73d 23h 53m 1 Arjen Poutsma 21/Jul/08 10:06 PM

          People

          • Assignee:
            arjen.poutsma Arjen Poutsma
            Reporter:
            olivergierke Oliver Gierke
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: