Done: https://issues.apache.org/jira/browse/WICKET-2469

Thanks,

Esteban

Igor Vaynberg escribió:
No chance without a Jira issue.

-igor

On 9/15/09, nico2884 <[email protected]> wrote:
I am using GAE + Wicket too, and I also had to copy/paste that class.
Any chances of making a refactor in a future release?

Thanks a lot! :)

Nico


Esteban Masoero-2 wrote:
Hi there:

In MultipartServletWebRequest constructor we have:
...
(line 93) DiskFileItemFactory factory = new DiskFileItemFactory();
...
which inhibits anyone from reusing MultipartServletWebRequest  with a
different factory. The only solution is generate a new class that is a
copy of this one, with that line changed.

Would it be possible to extract this code in  a method like:

    /** Creates and returns a new file item factory. Subclasses can
override this method to use a different factory
     * @returns a recently created FileItemFactory object
     */
    protected FileItemFactory getNewFileItemFactory() {
        return new DiskFileItemFactory();
    }

This way, we could just extend this class when there's a need to use a
different factory (as in GAE applications)

Thanks in advance,

Esteban

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



--
View this message in context:
http://www.nabble.com/Extract-method-suggestion-in-MultipartServletWebRequest-class-tp25390796p25456459.html
Sent from the Wicket - User mailing list archive at Nabble.com.


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to