search

javierdotnet / gwt-ext

Automatically exported from code.google.com/p/gwt-ext
1 stars 3 forks source link

actionComplete issue in Upload Form #375

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
From the forum:
http://gwt-ext.com/forum/viewtopic.php?f=8&t=2166

in extjs code if form is upload form, response does not contain status
field, but in Form.java
presents code:
Code:
function(frm, action) {
                    var responseText = '';
                    var status = 200;
                    if(action.response && action.response != null) {
                        responseText = action.response.responseText;
                        status = action.response.status;
                    }

listener.@com.gwtext.client.widgets.form.event.FormListener::onActionComplete(Lc
om/gwtext/client/widgets/form/Form;ILjava/lang/String;)(formJ,
status, responseText);
                }

need to change line
Code:
status = action.response.status;

to
Code:
status = action.response.status === null || action.response.status ===
undefined ? status : action.response.status;

Original issue reported on code.google.com by mlim1...@gmail.com on 23 Jul 2008 at 6:59

GoogleCodeExporter commented 9 years ago 
I have tried to add the proposed fix to trunk (r1799) and i'm happy to say that 
it
successfully fixed fileupload for me.

Before fix hosted mode browser would crash when doing fileupload.
(i'm on Mac os x with Gwt-ext 2.0.4 GWT1.5RC1 and extjs 2.0.2)

Original comment by nikolaj-...@49a.dk on 2 Aug 2008 at 9:20

GoogleCodeExporter commented 9 years ago 
Fixed in SVN...

Original comment by mlim1...@gmail.com on 2 Aug 2008 at 5:45

GoogleCodeExporter commented 9 years ago 
I'm not sure whether the fix should be applied to both actionComplete and
actionFailed or not?

Original comment by nikolaj-...@49a.dk on 3 Aug 2008 at 6:52