Details

    • Type: Improvement Improvement
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.3rc1
    • Fix Version/s: 1.3.3rc1
    • Component/s: JAXB
    • Labels:
      None
    • Number of attachments :
      7

      Description

      The idea of this task is to improve the current implementation of jaxb by implementing missing functionality.

      1. CastorMarshallerTest.patch
        19 kB
        Jakub Narloch
      2. CastorUnmarshallerTest.patch
        20 kB
        Jakub Narloch
      3. Code_refactoring_and_documentation_.patch
        42 kB
        Jakub Narloch
      4. Renamed_the_fascade_classes.patch
        69 kB
        Jakub Narloch
      5. Revised_the_implementation.patch
        20 kB
        Jakub Narloch
      6. UPDATE_TO_CASTOR_1_3_2.patch
        1 kB
        Jakub Narloch
      7. Updated_Tests_to_JUnit_4_x.patch
        63 kB
        Jakub Narloch

        Activity

        Hide
        Jakub Narloch added a comment -

        Patch for CastorUnmarshaller has been prepared.

        Show
        Jakub Narloch added a comment - Patch for CastorUnmarshaller has been prepared.
        Hide
        Jakub Narloch added a comment -

        How I see this farther:

        • I want to remove the Marshaller, Unmarshaller, JAXBContext and JAXBIntrospector, but first I would have to deal with the tests.
        • Add extra tests that shows how much of the JAXB annotations is supported.
        • Implement the Marshaller properties support.
        • Implement validation agains the schema in both marshaller and unmarshaller.
        • Implement package registreation in JAXBContext.

        There two things that currently I am not sure how to deal with: XmlAdapter class and AttachmentMarshaller/Unmarshaller the second thing won't be trivial without 'native' support from Castor.

        Show
        Jakub Narloch added a comment - How I see this farther: I want to remove the Marshaller, Unmarshaller, JAXBContext and JAXBIntrospector, but first I would have to deal with the tests. Add extra tests that shows how much of the JAXB annotations is supported. Implement the Marshaller properties support. Implement validation agains the schema in both marshaller and unmarshaller. Implement package registreation in JAXBContext. There two things that currently I am not sure how to deal with: XmlAdapter class and AttachmentMarshaller/Unmarshaller the second thing won't be trivial without 'native' support from Castor.
        Hide
        Werner Guttmann added a comment -

        I want to remove the Marshaller, Unmarshaller, JAXBContext and JAXBIntrospector, but first I would have to deal with the tests.

        I guess that refers to to the classes for which you have already added their corresponding Castor- equivalents ?

        Show
        Werner Guttmann added a comment - I want to remove the Marshaller, Unmarshaller, JAXBContext and JAXBIntrospector, but first I would have to deal with the tests. I guess that refers to to the classes for which you have already added their corresponding Castor- equivalents ?
        Hide
        Werner Guttmann added a comment -

        Personally, I'd not look into Attachement* functionality for the time being, as I'd consider it to be the least important. Yes, we can have a go at this once everything else is in place.

        Show
        Werner Guttmann added a comment - Personally, I'd not look into Attachement* functionality for the time being, as I'd consider it to be the least important. Yes, we can have a go at this once everything else is in place.
        Hide
        Jakub Narloch added a comment -

        Thanks Werner

        I guess that refers to to the classes for which you have already added their corresponding Castor- equivalents ?

        Yes, that is correct, but I had to first modify the existing tests.

        Show
        Jakub Narloch added a comment - Thanks Werner I guess that refers to to the classes for which you have already added their corresponding Castor- equivalents ? Yes, that is correct, but I had to first modify the existing tests.

          People

          • Assignee:
            Jakub Narloch
            Reporter:
            Jakub Narloch
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 2 days, 35 minutes
              2d 35m
              Remaining:
              Remaining Estimate - 2 days, 15 minutes
              2d 15m
              Logged:
              Remaining Estimate - 2 days, 15 minutes
              20m