Patch committed to 2.0 and HEAD Oleg
On Tue, 2004-01-27 at 16:50, [EMAIL PROTECTED] wrote: > > > Oleg, > > Yeah, the patch below will work. > > As for the two bugs, the first one doesn't really interest me, but the > second one interests me a little. I was going to implement it (kind of) > using separate classes, etc. Now, seeing the bug report, I might change it > a bit to be included in the classes. > > Thanks. > > Robert Lasch > Follett Library and School Group > > > > > > > > "Kalnichevski, Oleg" > > <[EMAIL PROTECTED] To: "Commons HttpClient > Project" > gpoint.com> <[EMAIL PROTECTED]> > > cc: > > 01/27/2004 07:42 AM Subject: RE: [PATCH] > FilePart class > Please respond to > > "Commons HttpClient > > Project" > > > > > > > > > > Robert, > > Could you live with something like that (see patch below)? > > Oleg > > PS: Two feature requests may be of interest to you. BTW, we happily accept > contributions ;-) > > http://nagoya.apache.org/bugzilla/show_bug.cgi?id=14036 > http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20288 > > > Index: FilePart.java > =================================================================== > RCS file: > /home/cvspublic/jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/methods/multipart/FilePart.java,v > > retrieving revision 1.14.2.1 > diff -u -r1.14.2.1 FilePart.java > --- FilePart.java 10 Oct 2003 04:16:03 -0000 1.14.2.1 > +++ FilePart.java 27 Jan 2004 13:34:20 -0000 > @@ -264,6 +264,16 @@ > } > } > > + /** > + * Returns the source of the file part. > + * > + * @return The source. > + */ > + protected PartSource getSource() { > + LOG.trace("enter getSource()"); > + return this.source; > + } > + > /** > * Return the length of the data. > * @return The length. > > > > > -----Original Message----- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > Sent: Monday, January 26, 2004 23:53 > To: [EMAIL PROTECTED] > Subject: [PATCH] FilePart class > > > > > > > Hi All, > > I'm currently working on a subclass of > org.apache.commons.httpclient.methods.multipart.FilePart that allows the > multipart post to be stopped prematurely. It would be really nice to have > protected access on the member variable source in FilePart, so I only have > to override the sendData function as opposed to most of the class. Any > chance of getting this changed? > > --- FilePart.java.orig Mon Jan 26 16:40:39 2004 > +++ FilePart.java Mon Jan 26 16:41:38 2004 > @@ -109,7 +109,7 @@ > EncodingUtil.getAsciiBytes(FILE_NAME); > > /** Source of the file part. */ > - private PartSource source; > + protected PartSource source; > > /** > * FilePart Constructor. > > > Thanks. > - Robert Lasch > Follett Library and School Group > > > --------------------------------------------------------------------- > 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] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]