Fix #162 Add all serializers for org.joda.time.Hours\Minutes\Seconds\Months\Years\Weeks (#164)

Author: JooHyukKim

release-notes/CREDITS-2.x

@@ -112,4 +112,5 @@ Stephane Paulus (StephanePaulus@github)
 
 Joo Hyuk Kim (@JooHyukKim)
   * Fixed #160: Cannot serialize `org.joda.time.Days`
+  * Fixed #162: Cannot serialize `org.joda.time.Hours\Minutes\Seconds\Months\Years\Weeks`
   (2.18.4)

release-notes/VERSION-2.x

@@ -9,6 +9,9 @@ Project: jackson-datatype-joda
 #160: Cannot serialize `org.joda.time.Days`
  (reported by @bestbugwriter)
  (fixed by Joo-Hyuk K)
+#162: Cannot serialize `org.joda.time.Hours\Minutes\Seconds\Months\Years\Weeks`
+ (reported by @bestbugwriter)
+ (fixed by Joo-Hyuk K)
 
 2.18.3 (28-Feb-2025)
 2.18.2 (27-Nov-2024)

src/main/java/com/fasterxml/jackson/datatype/joda/JodaModule.java

@@ -39,14 +39,20 @@ public JodaModule()
         addSerializer(DateTimeZone.class, new DateTimeZoneSerializer());
         addSerializer(Days.class, new DaysSerializer()); // 2.18.4
         addSerializer(Duration.class, new DurationSerializer());
+        addSerializer(Hours.class, new HoursSerializer()); // 2.18.4
         addSerializer(Instant.class, new InstantSerializer());
-        addSerializer(LocalDateTime.class, new LocalDateTimeSerializer());
+        addSerializer(Interval.class, new IntervalSerializer());
         addSerializer(LocalDate.class, new LocalDateSerializer());
+        addSerializer(LocalDateTime.class, new LocalDateTimeSerializer());
         addSerializer(LocalTime.class, new LocalTimeSerializer());
-        addSerializer(Period.class, new PeriodSerializer());
-        addSerializer(Interval.class, new IntervalSerializer());
+        addSerializer(Minutes.class, new MinutesSerializer()); // 2.18.4
         addSerializer(MonthDay.class, new MonthDaySerializer());
+        addSerializer(Months.class, new MonthsSerializer()); // 2.18.4
+        addSerializer(Period.class, new PeriodSerializer());
+        addSerializer(Seconds.class, new SecondsSerializer()); // 2.18.4
+        addSerializer(Weeks.class, new WeeksSerializer()); // 2.18.4
         addSerializer(YearMonth.class, new YearMonthSerializer());
+        addSerializer(Years.class, new YearsSerializer()); // 2.18.4
 
         // then key deserializers
         addKeyDeserializer(DateTime.class, new DateTimeKeyDeserializer());

src/main/java/com/fasterxml/jackson/datatype/joda/ser/HoursSerializer.java

@@ -0,0 +1,29 @@
+package com.fasterxml.jackson.datatype.joda.ser;
+
+import java.io.IOException;
+
+import org.joda.time.Hours;
+
+import com.fasterxml.jackson.core.JsonGenerator;
+import com.fasterxml.jackson.databind.SerializerProvider;
+
+/**
+ * Serializer for Joda {@link Hours} class.
+ *
+ * @since 2.18.4
+ */
+public class HoursSerializer
+    extends JodaSerializerBase<Hours>
+{
+
+    private static final long serialVersionUID = 1L;
+
+    public HoursSerializer() {
+        super(Hours.class);
+    }
+
+    @Override
+    public void serialize(Hours value, JsonGenerator gen, SerializerProvider provider) throws IOException {
+        gen.writeNumber(value.getHours());
+    }
+}

src/main/java/com/fasterxml/jackson/datatype/joda/ser/MinutesSerializer.java

@@ -0,0 +1,29 @@
+package com.fasterxml.jackson.datatype.joda.ser;
+
+import java.io.IOException;
+
+import org.joda.time.Minutes;
+
+import com.fasterxml.jackson.core.JsonGenerator;
+import com.fasterxml.jackson.databind.SerializerProvider;
+
+/**
+ * Serializer for Joda {@link Minutes} class.
+ *
+ * @since 2.18.4
+ */
+public class MinutesSerializer
+    extends JodaSerializerBase<Minutes>
+{
+
+    private static final long serialVersionUID = 1L;
+
+    public MinutesSerializer() {
+        super(Minutes.class);
+    }
+
+    @Override
+    public void serialize(Minutes value, JsonGenerator gen, SerializerProvider provider) throws IOException {
+        gen.writeNumber(value.getMinutes());
+    }
+}

src/main/java/com/fasterxml/jackson/datatype/joda/ser/MonthsSerializer.java

@@ -0,0 +1,29 @@
+package com.fasterxml.jackson.datatype.joda.ser;
+
+import java.io.IOException;
+
+import org.joda.time.Months;
+
+import com.fasterxml.jackson.core.JsonGenerator;
+import com.fasterxml.jackson.databind.SerializerProvider;
+
+/**
+ * Serializer for Joda {@link Months} class.
+ *
+ * @since 2.18.4
+ */
+public class MonthsSerializer
+    extends JodaSerializerBase<Months>
+{
+
+    private static final long serialVersionUID = 1L;
+
+    public MonthsSerializer() {
+        super(Months.class);
+    }
+
+    @Override
+    public void serialize(Months value, JsonGenerator gen, SerializerProvider provider) throws IOException {
+        gen.writeNumber(value.getMonths());
+    }
+}

src/main/java/com/fasterxml/jackson/datatype/joda/ser/SecondsSerializer.java

@@ -0,0 +1,29 @@
+package com.fasterxml.jackson.datatype.joda.ser;
+
+import java.io.IOException;
+
+import org.joda.time.Seconds;
+
+import com.fasterxml.jackson.core.JsonGenerator;
+import com.fasterxml.jackson.databind.SerializerProvider;
+
+/**
+ * Serializer for Joda {@link Seconds} class.
+ *
+ * @since 2.18.4
+ */
+public class SecondsSerializer
+    extends JodaSerializerBase<Seconds>
+{
+
+    private static final long serialVersionUID = 1L;
+
+    public SecondsSerializer() {
+        super(Seconds.class);
+    }
+
+    @Override
+    public void serialize(Seconds value, JsonGenerator gen, SerializerProvider provider) throws IOException {
+        gen.writeNumber(value.getSeconds());
+    }
+}

src/main/java/com/fasterxml/jackson/datatype/joda/ser/WeeksSerializer.java

@@ -0,0 +1,29 @@
+package com.fasterxml.jackson.datatype.joda.ser;
+
+import java.io.IOException;
+
+import org.joda.time.Weeks;
+
+import com.fasterxml.jackson.core.JsonGenerator;
+import com.fasterxml.jackson.databind.SerializerProvider;
+
+/**
+ * Serializer for Joda {@link Weeks} class.
+ *
+ * @since 2.18.4
+ */
+public class WeeksSerializer
+    extends JodaSerializerBase<Weeks>
+{
+
+    private static final long serialVersionUID = 1L;
+
+    public WeeksSerializer() {
+        super(Weeks.class);
+    }
+
+    @Override
+    public void serialize(Weeks value, JsonGenerator gen, SerializerProvider provider) throws IOException {
+        gen.writeNumber(value.getWeeks());
+    }
+}

src/main/java/com/fasterxml/jackson/datatype/joda/ser/YearsSerializer.java

@@ -0,0 +1,29 @@
+package com.fasterxml.jackson.datatype.joda.ser;
+
+import java.io.IOException;
+
+import org.joda.time.Years;
+
+import com.fasterxml.jackson.core.JsonGenerator;
+import com.fasterxml.jackson.databind.SerializerProvider;
+
+/**
+ * Serializer for Joda {@link Years} class.
+ *
+ * @since 2.18.4
+ */
+public class YearsSerializer
+    extends JodaSerializerBase<Years>
+{
+
+    private static final long serialVersionUID = 1L;
+
+    public YearsSerializer() {
+        super(Years.class);
+    }
+
+    @Override
+    public void serialize(Years value, JsonGenerator gen, SerializerProvider provider) throws IOException {
+        gen.writeNumber(value.getYears());
+    }
+}

src/test/java/com/fasterxml/jackson/datatype/joda/ser/JodaSerializationTest.java

@@ -12,6 +12,7 @@
 import org.joda.time.format.ISODateTimeFormat;
 
 import java.io.IOException;
+import java.util.Random;
 
 public class JodaSerializationTest extends JodaTestBase
 {
@@ -248,4 +249,47 @@ public void testCustomYearMonthSer() throws Exception
         String json = MAPPER.writeValueAsString(new FormattedYearMonth(yearMonth));
         assertEquals(a2q("{'value':'2013/08'}"), json);
     }
+
+    public void testHoursSer() throws Exception
+    {
+        Hours hours = Hours.hours(1);
+        String json = MAPPER.writeValueAsString(hours);
+        assertEquals("1", json);
+    }
+
+    public void testMinutesSer() throws Exception
+    {
+        Minutes minutes = Minutes.minutes(2);
+        String json = MAPPER.writeValueAsString(minutes);
+        assertEquals("2", json);
+    }
+
+    public void testSecondsSer() throws Exception
+    {
+        Seconds seconds = Seconds.seconds(3);
+        String json = MAPPER.writeValueAsString(seconds);
+        assertEquals("3", json);
+    }
+
+    public void testMonthsSer() throws Exception
+    {
+        Months months = Months.months(4);
+        String json = MAPPER.writeValueAsString(months);
+        assertEquals("4", json);
+    }
+
+    public void testYearsSer() throws Exception
+    {
+        Years years = Years.years(5);
+        String json = MAPPER.writeValueAsString(years);
+        assertEquals("5", json);
+    }
+
+    public void testWeeksSer() throws Exception
+    {
+        Weeks weeks = Weeks.weeks(6);
+        String json = MAPPER.writeValueAsString(weeks);
+        assertEquals("6", json);
+    }
+
 }