continued:-)
I meant the code

             if (!upload_->isUploaded())

does not work right, it should be

            if (upload_->isUploaded()) // although it did not meant literally

then can do right. so I always used

            if (upload_->isUploaded())

before 2.99.2 to finish uploadNow() work. and now, I should use

            if (upload_->canUpload())

to do before work. so I suggest the sentence of ReleaseNotes:

    You should replace any call to isUploaded() with the new method !canUpload()

changes to

    You should replace any call to isUploaded() with the new method canUpload()

Regards
Zhimin



2009/11/8 Zhimin Song <[email protected]>

> en, in fact I meant the following:
>
> before Wt-2.99.2, I used the sample code from example Composer like this :
>
> 00096 {
> 00097   /*
> 00098    * See if this attachment still needs to be uploaded,
> 00099    * and return if a new asyncrhonous upload is started.
> 00100    */
> 00101   if (upload_) {
> 00102     if (!upload_->isUploaded()) {
> 00103       upload_->upload();
> 00104       return true;
> 00105     } else
> 00106       return false;
> 00107   } else
> 00108     return false;
> 00109 }
>
> but after Wt-2.99.2, I use the code like this:
>
> 00096 {
> 00097   /*
> 00098    * See if this attachment still needs to be uploaded,
> 00099    * and return if a new asynchronous upload is started.
> 00100    */
> 00101   if (upload_ 
> <http://www.webtoolkit.eu/wt/doc/examples/html/classAttachmentEdit.html#a9a3945293cf0a30d35e8bc56312d430>)
>  {
> 00102     if (upload_ 
> <http://www.webtoolkit.eu/wt/doc/examples/html/classAttachmentEdit.html#a9a3945293cf0a30d35e8bc56312d430>->canUpload
>  
> <http://www.webtoolkit.eu/wt/doc/reference/html/classWt_1_1WFileUpload.html#66376fb4668a037760a91abad9f83f8a>())
>  {
> 00103       upload_ 
> <http://www.webtoolkit.eu/wt/doc/examples/html/classAttachmentEdit.html#a9a3945293cf0a30d35e8bc56312d430>->upload
>  
> <http://www.webtoolkit.eu/wt/doc/reference/html/classWt_1_1WFileUpload.html#c17094f2fd6ae8cac1035161f7d80b9c>();
> 00104       return true;
> 00105     } else
> 00106       return false;
> 00107   } else
> 00108     return false;
> 00109 }
>
> you can see, in fact, we directly replace the isUploaded() with canUpload()
> instead of !canUpload() :-)
>
> Regards
> zhimin
>
> 2009/11/8 OvermindDL1 <[email protected]>
>
> On Fri, Nov 6, 2009 at 9:08 PM, Zhimin Song <[email protected]> wrote:
>> > hi,
>> > I think the contents about isUploaded() in the RealeaseNotes are still
>> not
>> > well understood,
>> >
>> > this "You should replace any call to isUploaded() with the new method
>> > !canUpload() (note the inversion!." should change into "You should
>> replace
>> > any call to isUploaded() with the new method canUpload() (note the
>> > inversion!.", cause in fact the old version isUploaded() never be or
>> could
>> > not be used in the situation which its name expressed.
>> >
>> > right?
>>
>> Er, you forgot a ! before canUpload()?
>>
>>
>> ------------------------------------------------------------------------------
>> Let Crystal Reports handle the reporting - Free Crystal Reports 2008
>> 30-Day
>> trial. Simplify your report design, integration and deployment - and focus
>> on
>> what you do best, core application coding. Discover what's new with
>> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
>> _______________________________________________
>> witty-interest mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/witty-interest
>>
>
>
------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
witty-interest mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/witty-interest

Reply via email to