Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-6369

Change DateTextField to use java.time.LocalDate instead of java.util.Date

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: 8.0.0-M5
    • Fix Version/s: None
    • Component/s: wicket-extensions
    • Labels:
      None

      Description

      While working with Wicket 8.0 I noticed that DateTextField uses java.util.Date.
      I think that with Wicket 8 being created to use Java 8 features, it would be nice for DateTextField to use java.time.LocalDate.

        Issue Links

          Activity

          Hide
          mgrigorov Martin Grigorov added a comment -

          I'm closing this ticket.
          Please comment in WICKET-6105!
          Thank you!

          Show
          mgrigorov Martin Grigorov added a comment - I'm closing this ticket. Please comment in WICKET-6105 ! Thank you!
          Hide
          eximius Kamil added a comment -

          I think the best will be to create new "8" version of

          DateTextField

          , place it in new package (e.g wicket.java8) and deprecate the old one.
          This seems to be a good idea since - as you said - there are problems with supporting the old one

          Show
          eximius Kamil added a comment - I think the best will be to create new "8" version of DateTextField , place it in new package (e.g wicket.java8) and deprecate the old one. This seems to be a good idea since - as you said - there are problems with supporting the old one
          Hide
          mgrigorov Martin Grigorov added a comment -

          There is a ticket for this already.
          The problem with it is that most people want to preserve the support for j.u.Date and add j.t.LocalDate on top. And this proved to be not so simple.
          The second problem is that this widget uses YUI 2.x that is not supported for a long time.

          Show
          mgrigorov Martin Grigorov added a comment - There is a ticket for this already. The problem with it is that most people want to preserve the support for j.u.Date and add j.t.LocalDate on top. And this proved to be not so simple. The second problem is that this widget uses YUI 2.x that is not supported for a long time.

            People

            • Assignee:
              Unassigned
              Reporter:
              jaroslaw.ciarkowski Jaroslaw
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development