Hi,

wicket-util already has both converters.

Have fun
Sven


Am 26.09.2017 um 12:29 schrieb Maxim Solodovnik:
Hello Sven,

I have updated WICKET-6105 description with your improvement and have
started to work on it.

I got questions :))
I believe both DateConverter (to perform conversion to/from java.util.Date)
and LocalDateConverter should exist in wicket WDYT?

On Tue, Sep 26, 2017 at 3:08 PM, Maxim Solodovnik <solomax...@gmail.com>
wrote:

Now it works,
Thanks a lot Martin :)

On Tue, Sep 26, 2017 at 2:20 AM, Martin Grigorov <mgrigo...@apache.org>
wrote:

Do you have more than one JIRA users ?!
I've added you to all possible groups in WICKET's JIRA. You must have even
Admin rights.
I'm afraid you should ask Infra for more help if the issue persists.

Martin Grigorov
Wicket Training and Consulting
https://twitter.com/mtgrigorov

On Mon, Sep 25, 2017 at 6:06 PM, Maxim Solodovnik <solomax...@gmail.com>
wrote:

Thanks Martin,

but I'm still unable to change description or add comment :(

On Mon, Sep 25, 2017 at 1:17 PM, Martin Grigorov <mgrigo...@apache.org>
wrote:
On Sat, Sep 23, 2017 at 7:27 PM, Maxim Solodovnik <
solomax...@gmail.com>
wrote:

It seems I'm unable to edit
https://issues.apache.org/jira/browse/WICKET-6105 (to add points
proposed
be Sven)
Can someone please grant me rights?

Done!
I've re-assigned the ticket to you!
Thank you, Maxim!



On Sat, Sep 23, 2017 at 11:20 PM, Maxim Solodovnik <
solomax...@gmail.com>
wrote:

Thanks Sven,

Here is the PR with the current changes
Will try to work on it tomorrow

On Sat, Sep 23, 2017 at 3:57 PM, Sven Meier <s...@meiers.net>
wrote:
Hi,

there are many topics to improve on:

- DateConverter should be name DateTimeConverter and convert to
LocalDateTime
- DateField should work on a LocalDate (no longer extending
DateTimeField)
- without the DatePicker we could merge DateField and
DateTextField
- TimeField should work on a LocalTime (no longer extending
DateTimeField)
- for DateTimeField we should consider whether it works on
ZonedDateTime
or LocalDateTime - AFAIK most people use the former

Perhaps you could create a pull request instead, so we all have it
easier
to discuss the api changes.

Have fun
Sven


Am 23.09.2017 um 07:41 schrieb Maxim Solodovnik:

Local build is successful, could someone please review the
changes?
And I bet some additional classes/methods will be required ...

On Sat, Sep 23, 2017 at 12:22 AM, Maxim Solodovnik <
solomax...@gmail.com
wrote:

I have commited the code to WICKET-6105-java.time branch
The code is compilable (merged with master)
But tests fails, will try to continue this work on the weekend

On Fri, Sep 22, 2017 at 10:55 PM, Maxim Solodovnik <
solomax...@gmail.com>
wrote:

Just re-read description for WICKET-6105
<https://issues.apache.org/jira/browse/WICKET-6105>, will try
to
continue the work started by Martin in the same branch

On Fri, Sep 22, 2017 at 10:04 PM, Maxim Solodovnik <
solomax...@gmail.com>
wrote:

I read through http://markmail.org/thread/uqu6gfma6wihwcwt
And It seems all I need to do is to remove wicket-datetime
module
and
all references

Is this correct?

On Fri, Sep 22, 2017 at 9:47 PM, Maxim Solodovnik <
solomax...@gmail.com>
wrote:

Done: https://github.com/wicketstuff/core/commit/4f4a945d3fc
a0f856b8ec4f7e065da5f980feec6

On Fri, Sep 22, 2017 at 8:28 PM, Maxim Solodovnik <
solomax...@gmail.com

wrote:
Will perform further changes in wicketstuff
Will copy examples

On Fri, Sep 22, 2017 at 8:20 PM, Maxim Solodovnik <
solomax...@gmail.com> wrote:

Here is the wicketstuff module: https://github.com/wic
ketstuff/core/commit/6fd9af625
e7d7679ce438ad2df59bf47c1aa487b
Please review

Will start to work on wicket

On Fri, Sep 22, 2017 at 7:34 PM, Martin Makundi <
martin.maku...@koodaripalvelut.com> wrote:

+1 me too
2017-09-22 15:28 GMT+03:00 Andrea Del Bene <
an.delb...@gmail.com
:
+1 for me. But I doubt there will be something left in
wicket-datetime

when you will remove Joda :-)



On 22/09/2017 13:49, Maxim Solodovnik wrote:

OK, I'm ready to start :)
As far as I understand here is the plan:

1) move existing unchanged wicket-datetime to
wicketstuff
(8.0.0
only
change package)
2) remove joda-time from wicket-datetime (pom,
README.md)
3) use LocalDate/LocalDateTime where appropriate

Shall I proceed :))) ?

On Wed, Sep 20, 2017 at 7:53 PM, Martin Grigorov <

mgrigo...@apache.org>
wrote:
https://github.com/menodata/time4j - in case Java 8
Date
APIs
are not
good
enough

Martin Grigorov
Wicket Training and Consulting
https://twitter.com/mtgrigorov

On Tue, Sep 19, 2017 at 10:05 AM, Martin Makundi <
martin.maku...@koodaripalvelut.com> wrote:

2017-09-19 9:35 GMT+03:00 Martin Grigorov <
mgrigo...@apache.org

:
On Tue, Sep 19, 2017 at 8:12 AM, Martin Makundi <
martin.maku...@koodaripalvelut.com> wrote:

2017-09-19 7:16 GMT+03:00 Martin Grigorov <

martin.grigo...@gmail.com>:
On Sep 19, 2017 6:22 AM, "Martin Makundi" <
martin.maku...@koodaripalvelut.com> wrote:

IMO it's better to big mang whole wicket to drop
joda
date.
All
wicket
8

without joda.
Said the person who still uses Wicket 1.4 for his
apps...
Yes, incremental change is a pain so we big bang
1.4
->8.
Everyone wants to see the Big Bang and this might be
the
reason to
see
Wicket 8.0.0 in 2020.

Dropping joda should not significantly slow down
development.

True, if your name is Stephen Colebourne and you
know
both
APIs by
heart!
I personally had to ask Google and StackOverflow.com
several
questions
for
what I've done so far in the branch mentioned above.
My vote on whether to merge this rafectoring to
master
branch
will
depend
on the diff in wicket-examples.

Let's try it first. If it does not pass we can drop
joda
locally. So
long
as it is not a transient dependency it's doable.


**
Martin

2017-09-18 22:43 GMT+03:00 Martin Grigorov <

mgrigo...@apache.org>:
On Mon, Sep 18, 2017 at 6:50 AM, Martin Makundi <
martin.maku...@koodaripalvelut.com> wrote:

Is it possible to also drop joda from wicketstuff
for
consistency?

IMO, the WicketStuff module must be exactly the
same
as
the
one in
Wicket
7.x.

Any changes in the API should be done in Wicket
8.x.
This way when users migrate their apps to 8.x they
can
decide to
take
the
short route and use the WicketStuff module.
Later they can invest in reworking their code to
the
new
API and
use
again
org.apache.wicket:wicket-datetime
The WicketStuff module README should say that it
will be
available
only
for

Wicket 8.x lifecycle, to prevent double
maintenance.

2017-09-17 22:09 GMT+03:00 Andrea Del Bene <

an.delb...@gmail.com
:
+1 to move legacy code to WicketStuff. For 'legacy
code'
I
mean
all
the
code based on yui library and class
StyleDateConverter
which is
tailored
on

Joda time.


On 16/09/2017 14:13, Martin Grigorov wrote:

Hi Maxim,

Since I do not use wicket-datetime in my apps I
also
wanted to
break
it
hard by replacing Joda-Time with Java 8 classes.
But people who use it expressed concerns that it
will
be
hard
to
migrate
this way. Everyone will have to do the same for
his/her
application.

I remember Igor even suggested to have univeral
API
that
will

work
with
java.util.Date, Joda-Time and Java 8!
IMO the easiest thing that could be done is to
copy
the
current
code
to
WicketStuff, as https://github.com/wicketstuff
/wicket1.5-tree
,
and
rework
wicket-datetime to use Java 8 APIs.
Martin Grigorov
Wicket Training and Consulting
https://twitter.com/mtgrigorov

On Thu, Sep 14, 2017 at 7:11 AM, Maxim
Solodovnik
<
solomax...@gmail.com
wrote:

Hello Martin,
I tried to continue this work, but it seems I
need
some
information
before I can proceed
Could you please tell me What was the goal of
these
changes?
I would remove joda-time, and use LocalDate, LocalTime
instead
of
Date
As I can see you are using ZonedDateTime ...
On Mon, Sep 11, 2017 at 9:01 PM, Martin
Grigorov
<
mgrigo...@apache.org
wrote:
Yes, if finished on time!
Wicket 7 requires Java 7

Martin Grigorov
Wicket Training and Consulting
https://twitter.com/mtgrigorov

On Mon, Sep 11, 2017 at 3:34 PM, Rakesh A <

iamrakes...@rediffmail.com
wrote:
Hi,
Is it going to be for Wicket 8 only ?
Regards,
Rakesh.A

--
Sent from: http://apache-wicket.1842946.n

4.nabble.com/Users-forum-
f1842947.html
------------------------------
------------------------------
---------
To unsubscribe, e-mail: users-unsubscribe@wicket.
apache.org
For additional commands, e-mail:
users-h...@wicket.apache.org
--

WBR
Maxim aka solomax

------------------------------

------------------------------
---------
To unsubscribe, e-mail:
users-unsubscribe@wicket.apach
e.org
For additional commands, e-mail:
users-h...@wicket.apache.org
------------------------------

------------------------------
---------
To unsubscribe, e-mail:
users-unsubscribe@wicket.apach
e.org

For additional commands, e-mail:
users-h...@wicket.apache.org

------------------------------
------------------------------
---------

To unsubscribe, e-mail: users-unsubscribe@wicket.apach
e.org
For additional commands, e-mail:
users-h...@wicket.apache.org


--
WBR
Maxim aka solomax


--
WBR
Maxim aka solomax


--
WBR
Maxim aka solomax


--
WBR
Maxim aka solomax


--
WBR
Maxim aka solomax


--
WBR
Maxim aka solomax



------------------------------------------------------------
---------
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org



--
WBR
Maxim aka solomax



--
WBR
Maxim aka solomax



--
WBR
Maxim aka solomax

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org




--
WBR
Maxim aka solomax





---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org

Reply via email to