Don't use a formatter with an optional value when producing times - fixes mauricio#142

mauricio · mauricio · commit 8d2050075fe2 · 2015-07-10T22:15:03.000-03:00
diff --git a/db-async-common/src/main/scala/com/github/mauricio/async/db/column/TimeEncoderDecoder.scala b/db-async-common/src/main/scala/com/github/mauricio/async/db/column/TimeEncoderDecoder.scala
@@ -33,14 +33,16 @@ class TimeEncoderDecoder extends ColumnEncoderDecoder {
     .appendOptional(optional)
     .toFormatter
 
+  final private val printer = new DateTimeFormatterBuilder()
+    .appendPattern("HH:mm:ss.SSSSSS")
+    .toFormatter
+
   def formatter = format
 
-  override def decode(value: String): LocalTime = {
+  override def decode(value: String): LocalTime =
     format.parseLocalTime(value)
-  }
 
-  override def encode(value: Any): String = {
-    this.format.print(value.asInstanceOf[LocalTime])
-  }
+  override def encode(value: Any): String =
+    this.printer.print(value.asInstanceOf[LocalTime])
 
 }
