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

Method addOmittedFields in XStreamMarshaller should access XStream instance via getter.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.5
    • Component/s: OXM
    • Labels:
      None
    • Environment:
      Spring WS 1.5.4, JDK 5

      Description

      XStreamMarshaller consistently accesses to the XStream instance via getters. However the method addOmittedFields() contains the direct access to the XStream instance.
      Original code:
      public void addOmittedField(Class type, String fieldName)

      { xstream.omitField(type, fieldName); }

      This is inconsistent with the rest of the code in the class and prevents the subclasses of XStreamMarshaller to provide their own implementation of XStream.

        Activity

        novotnyr Robert Novotny created issue -
        arjen.poutsma Arjen Poutsma made changes -
        Field Original Value New Value
        Fix Version/s 1.5.5 [ 11067 ]
        arjen.poutsma Arjen Poutsma made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        arjen.poutsma Arjen Poutsma made changes -
        Resolution Fixed [ 1 ]
        Status In Progress [ 3 ] Resolved [ 5 ]
        Hide
        arjen.poutsma Arjen Poutsma added a comment -

        Closing old issues

        Show
        arjen.poutsma Arjen Poutsma added a comment - Closing old issues
        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 In Progress In Progress
        16h 1m 1 Arjen Poutsma 07/Oct/08 8:17 PM
        In Progress In Progress Resolved Resolved
        59s 1 Arjen Poutsma 07/Oct/08 8:18 PM
        Resolved Resolved Closed Closed
        1304d 10h 45m 1 Arjen Poutsma 04/May/12 7:03 AM

          People

          • Assignee:
            arjen.poutsma Arjen Poutsma
            Reporter:
            novotnyr Robert Novotny
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1h
              1h
              Remaining:
              Remaining Estimate - 1h
              1h
              Logged:
              Time Spent - Not Specified
              Not Specified