Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6
    • Labels:
      None

      Description

      The method toMap(ResultSet rs)
      should use getColumnLabel()

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        245d 17h 51m 1 William R. Speirs 13/May/13 20:55
        Resolved Resolved Closed Closed
        432d 12h 52m 1 Benedikt Ritter 20/Jul/14 09:48
        Benedikt Ritter made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        William R. Speirs made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.6 [ 12322483 ]
        Resolution Fixed [ 1 ]
        Hide
        William R. Speirs added a comment -

        Fix applied in r1482051.

        Show
        William R. Speirs added a comment - Fix applied in r1482051.
        Hide
        Michael J Chane added a comment -

        Hi Simone,
        it's org.apache.commons.dbutils.BasicRowProcessor.toMap(ResultSet).
        you have changed in org.apache.commons.dbutils.BeanProcessor.mapColumnsToProperties(ResultSetMetaData, PropertyDescriptor[])
        from java.sql.ResultSetMetaData.getColumnName(int) to java.sql.ResultSetMetaData.getColumnLabel(int).
        in version 1.3, for bug #57

        Show
        Michael J Chane added a comment - Hi Simone, it's org.apache.commons.dbutils.BasicRowProcessor.toMap(ResultSet). you have changed in org.apache.commons.dbutils.BeanProcessor.mapColumnsToProperties(ResultSetMetaData, PropertyDescriptor[]) from java.sql.ResultSetMetaData.getColumnName(int) to java.sql.ResultSetMetaData.getColumnLabel(int). in version 1.3, for bug #57
        Hide
        Simone Tripodi added a comment -

        Ni-hao Xiaofei,

        apologize but I didn't find intuitive to understand what you mean. Which class(es) are involved? And what is the reason of method changing? Any chance you submit a patch with a testcase that demonstrates the theory?

        Xie xie,
        -Simo

        Show
        Simone Tripodi added a comment - Ni-hao Xiaofei, apologize but I didn't find intuitive to understand what you mean. Which class(es) are involved? And what is the reason of method changing? Any chance you submit a patch with a testcase that demonstrates the theory? Xie xie, -Simo
        xiaofei.xu made changes -
        Description The method toMap(ResultSet rs)
        should use getColumnLabel()
        xiaofei.xu made changes -
        Field Original Value New Value
        Environment The method toMap(ResultSet rs)
        You know what I mean
        xiaofei.xu created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            xiaofei.xu
          • Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development