Camel
  1. Camel
  2. CAMEL-4011

type converters should return NULL for Double.NaN values instead of 0

    Details

      Description

      see this discussion...http://camel.465427.n5.nabble.com/XPath-for-an-Integer-td4422095.html

      Update the ObjectConverter.toXXX() methods to check for Double.NaN and return NULL instead of relying on Number.intValue()

        Activity

        Ben O'Day created issue -
        Hide
        Ben O'Day added a comment -

        patch for review...

        Show
        Ben O'Day added a comment - patch for review...
        Ben O'Day made changes -
        Field Original Value New Value
        Attachment CAMEL-4011.patch [ 12480320 ]
        Ben O'Day made changes -
        Attachment CAMEL-4011.patch [ 12480330 ]
        Ben O'Day made changes -
        Attachment CAMEL-4011.patch [ 12480320 ]
        Hide
        Ben O'Day added a comment -

        patch committed...

        Show
        Ben O'Day added a comment - patch committed...
        Ben O'Day made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Babak Vahdat added a comment -

        see my comments on the revision 1127744 here:

        http://camel.465427.n5.nabble.com/XPath-for-an-Integer-td4422095.html

        Show
        Babak Vahdat added a comment - see my comments on the revision 1127744 here: http://camel.465427.n5.nabble.com/XPath-for-an-Integer-td4422095.html
        Hide
        Ben O'Day added a comment -

        thanks, just added the check for Float.NaN as well

        Show
        Ben O'Day added a comment - thanks, just added the check for Float.NaN as well
        Daniel Kulp made changes -
        Fix Version/s 2.7.3 [ 12316520 ]
        Daniel Kulp made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1d 7h 4m 1 Ben O'Day 26/May/11 01:45
        Resolved Resolved Closed Closed
        152d 10h 50m 1 Daniel Kulp 25/Oct/11 12:36

          People

          • Assignee:
            Ben O'Day
            Reporter:
            Ben O'Day
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development