[OPEN][2.*/3.0] Standard submit of form does not set form action in dom

[surfyogi]

Looks like a bug, where I have to set the action attribute of the form in dom before submitting or there will be no form action assigned and the form post will fail:

items: [{
    xtype:"button",
    text: 'Find',
    minWidth: 100,
    style:'padding:0px 0px 0px 140px;margin:0px 10px 30px 10px',
    handler: function(){ 
                                    
        air_rt_form.getForm().el.dom.action=air_rt_form.getForm().url;
        air_rt_form.getForm().submit(); 
    }

                                   }]     // end inside items array

does the same in FF and IE, and I'm using the ext-base.js; easy work around but this should probably get fixed so other's don't run into the same thing.

thx! (Ext is the BEST man)

[Animal]

To expand, where the BasicForm config standardSubmit is used to request a browser submission of the form as opposed to an Ajax submission, the url config should be stamped into the generated DOM form element as the action attribute value.

[Animal]

I suggest this fix (to which I, herinafter known as the herinafter do perperually grant rights, notwithstanding... (continued page 94)):

    initEl : function(el){
        this.el = Ext.get(el);
        this.id = this.el.id || Ext.id();
        if(this.standardSubmit){
            this.el.dom.action = this.url;
        } else {
            this.el.on('submit', this.onSubmit, this);
        }
        this.el.addClass('x-form');
    },

[Animal]

Although this thread might provide a different option if <form> as body is implemented: http://extjs.com/forum/showthread.php?t=28125

[4him]

How about this other fix in 'FormPanel' (Form.js) in the 'onRender' method:

var o = {
  tag: 'form',
  action : this.url, // <-- Fix here 
  method : this.method || 'POST',
  id : this.formId || Ext.id()
};

It works for my use case...

[Animal]

Yes that will work for you, but the fix must really go into BasicForm because that is what gets submitted, and it can be used independently of FormPanel.

[Condor]

Yes that will work for you, but the fix must really go into BasicForm because that is what gets submitted, and it can be used independently of FormPanel.

I don't agree, BasicForm never creates a FORM element, so why should it set the action attribute?

Also, AJAX submit uses the FORM action attribute if the url of BasicForm isn't set.

ps. Shouldn't your code be:

this.el.dom.action = this.url;

[Animal]

No, it doesn't create the form, it wraps a form. But the point remains that it can be used without a FormPanel, so in this case, it must still honour the url config by setting the action attribute if standardSubmit is specified.

(I know, if they're using a BasicForm to wrap an existing DOM form, they're likely to have set action, but BasicForm has to be sure)

(Yes, of course... I missed the "dom" bit. I'll edit)

[mjlecomte]

Any plans to implement a patch for this in future version?

For now, one possible solution has been added to the FAQ:
http://extjs.com/learn/Ext_FAQ_Forms#Standard_Submit

Suggest to patch the source, or supplement the Docs to alert people what needs to be done.

[Animal]

Shameless BUMP!

When is this ancient bug with a very simple fix going to be fixed? The bug was reported in 2.1 and got sent out in 2.2!

People are still being bitten by this when using standardSubmit: true