gorp_test: update timestamp parsing code to match reality

gorp_test.go

@@ -2036,6 +2036,9 @@ func TestNullTime(t *testing.T) {
 
 	// if time is not null
 	ts, err := time.Parse(time.Stamp, "Jan 2 15:04:05")
+	if err != nil {
+		t.Errorf("failed to parse time %s: %s", time.Stamp, err.Error())
+	}
 	ent = &WithNullTime{
 		Id: 1,
 		Time: gorp.NullTime{

nulltypes.go

@@ -13,6 +13,7 @@ package gorp
 
 import (
 	"database/sql/driver"
+	"log"
 	"time"
 )
 
@@ -24,26 +25,39 @@ type NullTime struct {
 
 // Scan implements the Scanner interface.
 func (nt *NullTime) Scan(value interface{}) error {
+	log.Printf("Time scan value is: %#v", value)
 	switch t := value.(type) {
 	case time.Time:
 		nt.Time, nt.Valid = t, true
 	case []byte:
-		nt.Valid = false
-		for _, dtfmt := range []string{
-			"2006-01-02 15:04:05.999999999",
-			"2006-01-02T15:04:05.999999999",
-			"2006-01-02 15:04:05",
-			"2006-01-02T15:04:05",
-			"2006-01-02 15:04",
-			"2006-01-02T15:04",
-			"2006-01-02",
-			"2006-01-02 15:04:05-07:00",
-		} {
-			var err error
-			if nt.Time, err = time.Parse(dtfmt, string(t)); err == nil {
-				nt.Valid = true
-				break
-			}
+		v := strToTime(string(t))
+		if v != nil {
+			nt.Valid = true
+			nt.Time = *t
+		}
+	case string:
+		v := strToTime(t)
+		if v != nil {
+			nt.Valid = true
+			nt.Time = *t
+		}
+	}
+	return nil
+}
+
+func strToTime(v string) *time.Time {
+	for _, dtfmt := range []string{
+		"2006-01-02 15:04:05.999999999",
+		"2006-01-02T15:04:05.999999999",
+		"2006-01-02 15:04:05",
+		"2006-01-02T15:04:05",
+		"2006-01-02 15:04",
+		"2006-01-02T15:04",
+		"2006-01-02",
+		"2006-01-02 15:04:05-07:00",
+	} {
+		if t, err := time.Parse(dtfmt, v); err == nil {
+			return &t
 		}
 	}
 	return nil