You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@wicket.apache.org by Ernesto Reinaldo Barreiro <re...@gmail.com> on 2019/12/03 08:56:29 UTC
FileUpload class still implements IClusterable
Hi,
FileUpload still implements IClusterable but it contains a field of type
FileItem: which is no longer Serializable. Should IClusterable be dropped?
--
Regards - Ernesto Reinaldo Barreiro
Re: FileUpload class still implements IClusterable
Posted by Sven Meier <sv...@meiers.net>.
This will be fixed with Wicket 9.
Have fun
Sven
On 30.03.20 19:32, natefki wrote:
> FileUpload class despite everything actualizes IClusterable however it
> contains a field of type FileItem: which is never again Serializable, Drop
> IClusterable.
>
> --
> Sent from: http://apache-wicket.1842946.n4.nabble.com/Users-forum-f1842947.html
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org
Re: FileUpload class still implements IClusterable
Posted by natefki <zi...@outlook.com>.
FileUpload class despite everything actualizes IClusterable however it
contains a field of type FileItem: which is never again Serializable, Drop
IClusterable.
--
Sent from: http://apache-wicket.1842946.n4.nabble.com/Users-forum-f1842947.html
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org
Re: FileUpload class still implements IClusterable
Posted by Ernesto Reinaldo Barreiro <re...@gmail.com>.
Done
On Tue, Dec 3, 2019 at 6:19 PM Martin Grigorov <mg...@apache.org> wrote:
> On Tue, Dec 3, 2019 at 3:42 PM Ernesto Reinaldo Barreiro <
> reiern70@gmail.com>
> wrote:
>
> > Hi Martin,
> >
> >
> > On Tue, Dec 3, 2019 at 3:09 PM Martin Grigorov <mg...@apache.org>
> > wrote:
> >
> > > Hi Ernesto,
> > >
> > > Yes, I think FileUpload should not be Serializable.
> > > FileUploadField uses transient reference to the list of file uploads
> for
> > > this reason:
> > >
> > >
> >
> https://github.com/apache/wicket/blob/ad6ecac7fdebefc25d310361f3a92aa481c36b1f/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/FileUploadField.java#L50
> >
> >
> > Yes I'm aware of this behaviour: I just fixed some related issue in our
> > project and while doing that I noticed mentioned problem? Shall I create
> an
> > issue for this or just a PR?
> >
>
> Issue + commit (you have the permissions).
> But maybe only in Wicket 9.x because otherwise Clirr plugin will complain
> that it is an API break.
>
>
> >
> > --
> > Regards - Ernesto Reinaldo Barreiro
> >
>
--
Regards - Ernesto Reinaldo Barreiro
Re: FileUpload class still implements IClusterable
Posted by Ernesto Reinaldo Barreiro <re...@gmail.com>.
Done
On Tue, Dec 3, 2019 at 6:19 PM Martin Grigorov <mg...@apache.org> wrote:
> On Tue, Dec 3, 2019 at 3:42 PM Ernesto Reinaldo Barreiro <
> reiern70@gmail.com>
> wrote:
>
> > Hi Martin,
> >
> >
> > On Tue, Dec 3, 2019 at 3:09 PM Martin Grigorov <mg...@apache.org>
> > wrote:
> >
> > > Hi Ernesto,
> > >
> > > Yes, I think FileUpload should not be Serializable.
> > > FileUploadField uses transient reference to the list of file uploads
> for
> > > this reason:
> > >
> > >
> >
> https://github.com/apache/wicket/blob/ad6ecac7fdebefc25d310361f3a92aa481c36b1f/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/FileUploadField.java#L50
> >
> >
> > Yes I'm aware of this behaviour: I just fixed some related issue in our
> > project and while doing that I noticed mentioned problem? Shall I create
> an
> > issue for this or just a PR?
> >
>
> Issue + commit (you have the permissions).
> But maybe only in Wicket 9.x because otherwise Clirr plugin will complain
> that it is an API break.
>
>
> >
> > --
> > Regards - Ernesto Reinaldo Barreiro
> >
>
--
Regards - Ernesto Reinaldo Barreiro
Re: FileUpload class still implements IClusterable
Posted by Martin Grigorov <mg...@apache.org>.
On Tue, Dec 3, 2019 at 3:42 PM Ernesto Reinaldo Barreiro <re...@gmail.com>
wrote:
> Hi Martin,
>
>
> On Tue, Dec 3, 2019 at 3:09 PM Martin Grigorov <mg...@apache.org>
> wrote:
>
> > Hi Ernesto,
> >
> > Yes, I think FileUpload should not be Serializable.
> > FileUploadField uses transient reference to the list of file uploads for
> > this reason:
> >
> >
> https://github.com/apache/wicket/blob/ad6ecac7fdebefc25d310361f3a92aa481c36b1f/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/FileUploadField.java#L50
>
>
> Yes I'm aware of this behaviour: I just fixed some related issue in our
> project and while doing that I noticed mentioned problem? Shall I create an
> issue for this or just a PR?
>
Issue + commit (you have the permissions).
But maybe only in Wicket 9.x because otherwise Clirr plugin will complain
that it is an API break.
>
> --
> Regards - Ernesto Reinaldo Barreiro
>
Re: FileUpload class still implements IClusterable
Posted by Martin Grigorov <mg...@apache.org>.
On Tue, Dec 3, 2019 at 3:42 PM Ernesto Reinaldo Barreiro <re...@gmail.com>
wrote:
> Hi Martin,
>
>
> On Tue, Dec 3, 2019 at 3:09 PM Martin Grigorov <mg...@apache.org>
> wrote:
>
> > Hi Ernesto,
> >
> > Yes, I think FileUpload should not be Serializable.
> > FileUploadField uses transient reference to the list of file uploads for
> > this reason:
> >
> >
> https://github.com/apache/wicket/blob/ad6ecac7fdebefc25d310361f3a92aa481c36b1f/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/FileUploadField.java#L50
>
>
> Yes I'm aware of this behaviour: I just fixed some related issue in our
> project and while doing that I noticed mentioned problem? Shall I create an
> issue for this or just a PR?
>
Issue + commit (you have the permissions).
But maybe only in Wicket 9.x because otherwise Clirr plugin will complain
that it is an API break.
>
> --
> Regards - Ernesto Reinaldo Barreiro
>
Re: FileUpload class still implements IClusterable
Posted by Ernesto Reinaldo Barreiro <re...@gmail.com>.
Hi Martin,
On Tue, Dec 3, 2019 at 3:09 PM Martin Grigorov <mg...@apache.org> wrote:
> Hi Ernesto,
>
> Yes, I think FileUpload should not be Serializable.
> FileUploadField uses transient reference to the list of file uploads for
> this reason:
>
> https://github.com/apache/wicket/blob/ad6ecac7fdebefc25d310361f3a92aa481c36b1f/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/FileUploadField.java#L50
Yes I'm aware of this behaviour: I just fixed some related issue in our
project and while doing that I noticed mentioned problem? Shall I create an
issue for this or just a PR?
--
Regards - Ernesto Reinaldo Barreiro
Re: FileUpload class still implements IClusterable
Posted by Ernesto Reinaldo Barreiro <re...@gmail.com>.
Hi Martin,
On Tue, Dec 3, 2019 at 3:09 PM Martin Grigorov <mg...@apache.org> wrote:
> Hi Ernesto,
>
> Yes, I think FileUpload should not be Serializable.
> FileUploadField uses transient reference to the list of file uploads for
> this reason:
>
> https://github.com/apache/wicket/blob/ad6ecac7fdebefc25d310361f3a92aa481c36b1f/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/FileUploadField.java#L50
Yes I'm aware of this behaviour: I just fixed some related issue in our
project and while doing that I noticed mentioned problem? Shall I create an
issue for this or just a PR?
--
Regards - Ernesto Reinaldo Barreiro
Re: FileUpload class still implements IClusterable
Posted by Martin Grigorov <mg...@apache.org>.
Hi Ernesto,
Yes, I think FileUpload should not be Serializable.
FileUploadField uses transient reference to the list of file uploads for
this reason:
https://github.com/apache/wicket/blob/ad6ecac7fdebefc25d310361f3a92aa481c36b1f/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/FileUploadField.java#L50
On Tue, Dec 3, 2019 at 10:56 AM Ernesto Reinaldo Barreiro <
reiern70@gmail.com> wrote:
> Hi,
>
> FileUpload still implements IClusterable but it contains a field of type
> FileItem: which is no longer Serializable. Should IClusterable be dropped?
>
> --
> Regards - Ernesto Reinaldo Barreiro
>
Re: FileUpload class still implements IClusterable
Posted by Martin Grigorov <mg...@apache.org>.
Hi Ernesto,
Yes, I think FileUpload should not be Serializable.
FileUploadField uses transient reference to the list of file uploads for
this reason:
https://github.com/apache/wicket/blob/ad6ecac7fdebefc25d310361f3a92aa481c36b1f/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/FileUploadField.java#L50
On Tue, Dec 3, 2019 at 10:56 AM Ernesto Reinaldo Barreiro <
reiern70@gmail.com> wrote:
> Hi,
>
> FileUpload still implements IClusterable but it contains a field of type
> FileItem: which is no longer Serializable. Should IClusterable be dropped?
>
> --
> Regards - Ernesto Reinaldo Barreiro
>