Camel
  1. Camel
  2. CAMEL-6041

Bindy does not handle separator at the end of quoted field, appends next field

    Details

    • Estimated Complexity:
      Unknown

      Description

      Address fields quite often contain comma at the end of an addressLine1 field.
      Example - "PROBLEM SOLVER","8506 SIX FORKS ROAD,","SUITE 104","RALEIGH","NC","27615","US"
      Bindy will append next field to such one shifting all others to the left.

        Activity

        Slava created issue -
        Hide
        Slava added a comment - - edited

        Test project with suggested patch(for version 2.10.3).
        Situations with separator at the beginning of field still requires attention. Though in real life such use cases are probably rare.

        Show
        Slava added a comment - - edited Test project with suggested patch(for version 2.10.3). Situations with separator at the beginning of field still requires attention. Though in real life such use cases are probably rare.
        Slava made changes -
        Field Original Value New Value
        Attachment bindy-cvsissue-test.zip [ 12568244 ]
        Slava made changes -
        Attachment bindy-cvsissue-patch.txt [ 12568247 ]
        Slava made changes -
        Labels patch
        Slava made changes -
        Labels patch maven patch test
        Willem Jiang made changes -
        Assignee Willem Jiang [ njiang ]
        Claus Ibsen made changes -
        Fix Version/s 2.10.5 [ 12324024 ]
        Fix Version/s 2.11.1 [ 12323967 ]
        Fix Version/s 2.12.0 [ 12323968 ]
        Hide
        Claus Ibsen added a comment -

        Thanks for the patch. Willem can you take a look?

        Show
        Claus Ibsen added a comment - Thanks for the patch. Willem can you take a look?
        Willem Jiang made changes -
        Fix Version/s 2.11.0 [ 12321695 ]
        Fix Version/s 2.11.1 [ 12323967 ]
        Fix Version/s 2.12.0 [ 12323968 ]
        Hide
        Willem Jiang added a comment -

        Applied the patch into trunk and merged back into camel-2.10.x branch.

        Show
        Willem Jiang added a comment - Applied the patch into trunk and merged back into camel-2.10.x branch.
        Willem Jiang made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]

          People

          • Assignee:
            Willem Jiang
            Reporter:
            Slava
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development