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

DefaultConcretePartProvider is not allowing a blank suffix, due to improper assertion

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.0.3
    • Component/s: Core
    • Labels:

      Description

      The allowance for a blank ("") suffix is esential to be able to maintain compatability w/ legacy or handwritten WSDLs.

      should read as:

      /** Sets the suffix to append to the port type name to obtain the binding name. */
      public void setBindingSuffix(String bindingSuffix)

      { Assert.notNull(bindingSuffix, "'bindingSuffix' must not be null"); this.bindingSuffix = bindingSuffix; }

      currently as:

      public void setBindingSuffix(String bindingSuffix)

      { Assert.hasText(bindingSuffix, "'bindingSuffix' must not be null"); this.bindingSuffix = bindingSuffix; }

      patch forthcomming....

        Activity

        jpyeron Jason Pyeron created issue -
        Hide
        jpyeron Jason Pyeron added a comment -

        Permission is given for inclusion or any other use.

        Show
        jpyeron Jason Pyeron added a comment - Permission is given for inclusion or any other use.
        jpyeron Jason Pyeron made changes -
        Field Original Value New Value
        Attachment patch-SWS-724-spring-nonnull-bindingsuffix-issue.patch [ 18552 ]
        arjen.poutsma Arjen Poutsma made changes -
        Fix Version/s 2.0.3 [ 12197 ]
        Assignee Arjen Poutsma [ arjen.poutsma ]
        arjen.poutsma Arjen Poutsma made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        arjen.poutsma Arjen Poutsma made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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!
        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
        55d 17h 24m 1 Arjen Poutsma 06/Oct/11 3:21 AM
        In Progress In Progress Resolved Resolved
        1m 43s 1 Arjen Poutsma 06/Oct/11 3:23 AM
        Resolved Resolved Closed Closed
        211d 3h 40m 1 Arjen Poutsma 04/May/12 7:03 AM

          People

          • Assignee:
            arjen.poutsma Arjen Poutsma
            Reporter:
            jpyeron Jason Pyeron
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: