Repository: wicket
Updated Branches:
  refs/heads/WICKET-6105-java.time e68844016 -> 3dc016119


redundant IDateConverter is removed


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/3dc01611
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/3dc01611
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/3dc01611

Branch: refs/heads/WICKET-6105-java.time
Commit: 3dc0161193cc938833ec3650c592b5b5a955229b
Parents: e688440
Author: Maxim Solodovnik <[email protected]>
Authored: Thu Oct 5 11:05:22 2017 +0700
Committer: Maxim Solodovnik <[email protected]>
Committed: Thu Oct 5 11:05:22 2017 +0700

----------------------------------------------------------------------
 .../markup/html/form/datetime/DateField.java    | 10 ++--
 .../html/form/datetime/IDateConverter.java      | 54 --------------------
 .../html/form/datetime/LocalDateConverter.java  |  3 +-
 .../html/form/datetime/LocalTimeConverter.java  |  3 +-
 .../markup/html/form/datetime/TimeField.java    | 10 ++--
 .../form/datetime/ZonedDateTimeConverter.java   |  6 +--
 6 files changed, 16 insertions(+), 70 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/3dc01611/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/DateField.java
----------------------------------------------------------------------
diff --git 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/DateField.java
 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/DateField.java
index 88d2637..895c0c6 100644
--- 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/DateField.java
+++ 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/DateField.java
@@ -153,7 +153,7 @@ public class DateField extends TextField<LocalDate> 
implements ITextFormatProvid
         *            the date converter
         * @return DateField
         */
-       public static DateField withConverter(String id, 
IDateConverter<LocalDate> converter)
+       public static DateField withConverter(String id, LocalDateConverter 
converter)
        {
                return withConverter(id, null, converter);
        }
@@ -169,7 +169,7 @@ public class DateField extends TextField<LocalDate> 
implements ITextFormatProvid
         *            the date converter
         * @return DateField
         */
-       public static DateField withConverter(String id, IModel<LocalDate> 
model, IDateConverter<LocalDate> converter)
+       public static DateField withConverter(String id, IModel<LocalDate> 
model, LocalDateConverter converter)
        {
                return new DateField(id, model, converter);
        }
@@ -177,7 +177,7 @@ public class DateField extends TextField<LocalDate> 
implements ITextFormatProvid
        /**
         * The converter for the TextField
         */
-       private final IDateConverter<LocalDate> converter;
+       private final LocalDateConverter converter;
 
        /**
         * Construct with a converter.
@@ -189,7 +189,7 @@ public class DateField extends TextField<LocalDate> 
implements ITextFormatProvid
         * @param converter
         *            The converter to use
         */
-       public DateField(String id, IModel<LocalDate> model, 
IDateConverter<LocalDate> converter)
+       public DateField(String id, IModel<LocalDate> model, LocalDateConverter 
converter)
        {
                super(id, model, LocalDate.class);
 
@@ -205,7 +205,7 @@ public class DateField extends TextField<LocalDate> 
implements ITextFormatProvid
         * @param converter
         *            The converter to use
         */
-       public DateField(String id, IDateConverter<LocalDate> converter)
+       public DateField(String id, LocalDateConverter converter)
        {
                this(id, null, converter);
        }

http://git-wip-us.apache.org/repos/asf/wicket/blob/3dc01611/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/IDateConverter.java
----------------------------------------------------------------------
diff --git 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/IDateConverter.java
 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/IDateConverter.java
deleted file mode 100644
index 59351e7..0000000
--- 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/IDateConverter.java
+++ /dev/null
@@ -1,54 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.wicket.extensions.markup.html.form.datetime;
-
-import java.time.format.DateTimeFormatter;
-import java.time.temporal.Temporal;
-import java.util.Locale;
-
-import org.apache.wicket.util.convert.IConverter;
-
-
-/**
- * Base class for javax.time based date converters. It contains the logic to 
parse and format,
- * optionally taking the time zone difference between clients and the server 
into account.
- * <p>
- * Converters of this class are best suited for per-component use.
- * </p>
- * 
- * @author eelcohillenius
- */
-public interface IDateConverter<T extends Temporal> extends IConverter<T>
-{
-
-       T convertToObject(String value, DateTimeFormatter format, Locale 
locale);
-
-       /**
-        * @param locale
-        *            The locale used to convert the value
-        * @return Gets the pattern that is used for printing and parsing
-        */
-       String getPattern(Locale locale);
-
-       /**
-        * @param locale
-        *            The locale used to convert the value
-        * 
-        * @return formatter The formatter for the current conversion
-        */
-       DateTimeFormatter getFormat(Locale locale);
-}

http://git-wip-us.apache.org/repos/asf/wicket/blob/3dc01611/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalDateConverter.java
----------------------------------------------------------------------
diff --git 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalDateConverter.java
 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalDateConverter.java
index 28554bd..95e1a47 100644
--- 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalDateConverter.java
+++ 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalDateConverter.java
@@ -21,6 +21,7 @@ import java.time.format.DateTimeFormatter;
 import java.util.Locale;
 
 import org.apache.wicket.util.convert.ConversionException;
+import org.apache.wicket.util.convert.IConverter;
 import org.apache.wicket.util.lang.Args;
 import org.apache.wicket.util.string.Strings;
 
@@ -34,7 +35,7 @@ import org.apache.wicket.util.string.Strings;
  * 
  * @author eelcohillenius
  */
-public abstract class LocalDateConverter implements IDateConverter<LocalDate>
+public abstract class LocalDateConverter implements IConverter<LocalDate>
 {
        private static final long serialVersionUID = 1L;
 

http://git-wip-us.apache.org/repos/asf/wicket/blob/3dc01611/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalTimeConverter.java
----------------------------------------------------------------------
diff --git 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalTimeConverter.java
 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalTimeConverter.java
index a10a662..1597ab6 100644
--- 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalTimeConverter.java
+++ 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/LocalTimeConverter.java
@@ -21,6 +21,7 @@ import java.time.format.DateTimeFormatter;
 import java.util.Locale;
 
 import org.apache.wicket.util.convert.ConversionException;
+import org.apache.wicket.util.convert.IConverter;
 import org.apache.wicket.util.lang.Args;
 import org.apache.wicket.util.string.Strings;
 
@@ -34,7 +35,7 @@ import org.apache.wicket.util.string.Strings;
  * 
  * @author eelcohillenius
  */
-public abstract class LocalTimeConverter implements IDateConverter<LocalTime>
+public abstract class LocalTimeConverter implements IConverter<LocalTime>
 {
        private static final long serialVersionUID = 1L;
 

http://git-wip-us.apache.org/repos/asf/wicket/blob/3dc01611/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/TimeField.java
----------------------------------------------------------------------
diff --git 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/TimeField.java
 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/TimeField.java
index 51e63c3..68af251 100644
--- 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/TimeField.java
+++ 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/TimeField.java
@@ -201,7 +201,7 @@ public class TimeField extends 
FormComponentPanel<LocalTime> implements ITextFor
         *            the date converter
         * @return TimeField
         */
-       public static TimeField withConverter(String id, 
IDateConverter<LocalTime> converter)
+       public static TimeField withConverter(String id, LocalTimeConverter 
converter)
        {
                return withConverter(id, null, converter);
        }
@@ -217,7 +217,7 @@ public class TimeField extends 
FormComponentPanel<LocalTime> implements ITextFor
         *            the date converter
         * @return TimeField
         */
-       public static TimeField withConverter(String id, IModel<LocalTime> 
model, IDateConverter<LocalTime> converter)
+       public static TimeField withConverter(String id, IModel<LocalTime> 
model, LocalTimeConverter converter)
        {
                return new TimeField(id, model, converter);
        }
@@ -225,7 +225,7 @@ public class TimeField extends 
FormComponentPanel<LocalTime> implements ITextFor
        /**
         * The converter for the TextField
         */
-       private final IDateConverter<LocalTime> converter;
+       private final LocalTimeConverter converter;
 
        /**
         * Construct.
@@ -233,7 +233,7 @@ public class TimeField extends 
FormComponentPanel<LocalTime> implements ITextFor
         * @param id
         *      the component id
         */
-       public TimeField(String id, IDateConverter<LocalTime> converter)
+       public TimeField(String id, LocalTimeConverter converter)
        {
                this(id, null, converter);
        }
@@ -246,7 +246,7 @@ public class TimeField extends 
FormComponentPanel<LocalTime> implements ITextFor
         * @param model
         *      the component's model
         */
-       public TimeField(String id, IModel<LocalTime> model, 
IDateConverter<LocalTime> converter)
+       public TimeField(String id, IModel<LocalTime> model, LocalTimeConverter 
converter)
        {
                super(id, model);
 

http://git-wip-us.apache.org/repos/asf/wicket/blob/3dc01611/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/ZonedDateTimeConverter.java
----------------------------------------------------------------------
diff --git 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/ZonedDateTimeConverter.java
 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/ZonedDateTimeConverter.java
index 2d45aa0..9286f2e 100644
--- 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/ZonedDateTimeConverter.java
+++ 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/form/datetime/ZonedDateTimeConverter.java
@@ -26,6 +26,7 @@ import org.apache.wicket.Session;
 import org.apache.wicket.core.request.ClientInfo;
 import org.apache.wicket.protocol.http.request.WebClientInfo;
 import org.apache.wicket.util.convert.ConversionException;
+import org.apache.wicket.util.convert.IConverter;
 import org.apache.wicket.util.lang.Args;
 import org.apache.wicket.util.string.Strings;
 
@@ -39,7 +40,7 @@ import org.apache.wicket.util.string.Strings;
  * 
  * @author eelcohillenius
  */
-public abstract class ZonedDateTimeConverter implements 
IDateConverter<ZonedDateTime>
+public abstract class ZonedDateTimeConverter implements 
IConverter<ZonedDateTime>
 {
        private static final long serialVersionUID = 1L;
 
@@ -64,7 +65,6 @@ public abstract class ZonedDateTimeConverter implements 
IDateConverter<ZonedDate
                this.applyTimeZoneDifference = applyTimeZoneDifference;
        }
 
-       @Override
        public ZonedDateTime convertToObject(String value, DateTimeFormatter 
format, Locale locale) {
                try
                {
@@ -164,7 +164,6 @@ public abstract class ZonedDateTimeConverter implements 
IDateConverter<ZonedDate
         *            The locale used to convert the value
         * @return Gets the pattern that is used for printing and parsing
         */
-       @Override
        public abstract String getPattern(Locale locale);
 
        /**
@@ -189,7 +188,6 @@ public abstract class ZonedDateTimeConverter implements 
IDateConverter<ZonedDate
         * 
         * @return formatter The formatter for the current conversion
         */
-       @Override
        public abstract DateTimeFormatter getFormat(Locale locale);
 
        /**

Reply via email to