NHibernate
  1. NHibernate
  2. NH-2660

Db.Time types are being set as DateTime

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: 3.1.0
    • Fix Version/s: None
    • Component/s: Core
    • Labels:
      None

      Description

      NHibernate.Type.TimeAsTimeSpanType and NHibernate.Type.TimeType add DateTime parameters to the commands passed to their Set() methods, but per this MSDN article: http://msdn.microsoft.com/en-us/library/cc716729.aspx , the .NET datatype that should be used for DbType.Time fields is TimeSpan. At least when trying to update TIME columns with SQL Server 2008, this causes an error.

      I believe that this can be fairly easily corrected by having the Set() methods save TimeSpans rather than DateTimes to the parameter list.

        Issue Links

          Activity

          Hide
          Neal Groothuis added a comment -

          Test case illustrating the problem.

          Show
          Neal Groothuis added a comment - Test case illustrating the problem.
          Hide
          Fabio Maulo added a comment -

          Don't worry for the "won't fix"... I'm on the workaround of the Ms bug.
          I'll introduce two new NH's types.

          • DateTimeToTimeSpanType
          • TimeSpanType (with a breaking change for the old one who will renamed to TimeSpanAsInte64Type)
          Show
          Fabio Maulo added a comment - Don't worry for the "won't fix"... I'm on the workaround of the Ms bug. I'll introduce two new NH's types. DateTimeToTimeSpanType TimeSpanType (with a breaking change for the old one who will renamed to TimeSpanAsInte64Type)

            People

            • Assignee:
              Unassigned
              Reporter:
              Neal Groothuis
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Who's Looking?