Hi Derrell,
Ok, I now have added "this"
this.tableModel = new qx.ui.table.model.Simple();
and it works now. Great.
I also tried again your solution with only one addListener:
var form = new
mobilewebjobticket.UploadForm('uploadFrm','/uploads/upload.php');
this.getTableData("form", form);
and
form.addListener('completed',function(e) {
this.debug('completed');
file.setFieldValue('');
//var response = this.getIframeHtmlContent();
var response = this.getTableData("form").getIframeHtmlContent();
this.debug(response);
wm1.close();
}, this);
but the error message is now:
this.getTableData is undefined
which means line
var response = this.getTableData("form").getIframeHtmlContent();
best regards
Hansjoerg
2010/3/8 Derrell Lipman <[email protected]>
> On Mon, Mar 8, 2010 at 09:47, Qoodary Doo <[email protected]> wrote:
>
>> Hi Derrell,
>>
>> now it works!
>>
> Great! But let's talk about your solution...
>
>
>> I am so happy. Thanks a lot for your help and your patience.
>>
> You're welcome.
>
>
>> I have now added two addListener. One for callling my getTableData
>> function and one for getIframeContent
>>
>> * form.addListener('completed',function(e) {
>> this.getTableData();
>> }, this);
>> *
>>
>>
>> * form.addListener('completed',function(e) {
>>
>> this.debug('completed');
>> file.setFieldValue('');
>> var response = this.getIframeHtmlContent();
>> this.debug(response);
>> wm1.close();
>> });
>> *
>
>
> Although this works, it's (a) not the most efficient way to do it because
> two separate event handlers must be called for the same event, and (b) it
> makes your code a bit more difficult to read than necessary, since you're
> handing a "completed" event in two different places. Using either of the
> methods I described in a previous message (closure of form, or saving the
> form variable as user data in the Application object) would allow you to use
> only a single event and handler function.
>
> **Then I have changed
>> var tableModel = new qx.ui.table.model.Simple();
>>
>> to
>>
>> tableModel = new qx.ui.table.model.Simple();
>>
>> and in the function from
>> this.tableModel.setData(content);
>> to
>> tableModel.setData(content);
>>
>> I do not know if this is ok or if this is proper code but for the moment
>> it works finde :-)
>>
>
> Nope, you definitely don't want to do this. What you've done is changed
> tableModel from being a variable local to your method of the Application
> class, to being a GLOBAL variable. Bad, bad, bad. :-) For many reasons, you
> should try to avoid global variables nearly always. All of qooxdoo has only
> a single global variable, qx, and thus qooxdoo plays well with other
> well-written JavaScript packages that also avoid polluting the global
> namespace.
>
> You were on the right track before you changed your function. Replace
>
> var tableModel = new qx.ui.table.model.Simple();
> with
> this.tableModel = new qx.ui.table.model.Simple();
>
> which allows your function to access this.tableModel.setData().
>
> It sounds like you're well on your way!
>
> Derrell
>
>
>
> ------------------------------------------------------------------------------
> Download Intel® Parallel Studio Eval
> Try the new software tools for yourself. Speed compiling, find bugs
> proactively, and fine-tune applications for parallel performance.
> See why Intel Parallel Studio got high marks during beta.
> http://p.sf.net/sfu/intel-sw-dev
> _______________________________________________
> qooxdoo-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
>
>
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
