See attached small patch. Fixes the behaviour of forms giving aerrors when executed, because $form['#action'] isn't wrapped through url()..

CommentFileSizeAuthor
form-api-form-actions.patch3.87 KBStefan Nagtegaal

Comments

asimmonds’s picture

asimmonds commented

I applied this and works as advertised, tested with clean-urls enabled and disabled. Brings these #action uses in line with the other 8 or so in HEAD.

Tobias Maier’s picture

Tobias Maier commented
Status: Needs review » Reviewed & tested by the community

looks great to me!

junyor’s picture

junyor commented

Shouldn't this be handled in the form API when handling #action rather than every time #action appears?

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented

Adrian is aware of the problem, and your solution.. He told me that #action is going to be wrapped through url() by default in the forms api..

though I'm not sure there is a patch already...

junyor’s picture

junyor commented
Status: Reviewed & tested by the community » Needs work

Then it sounds like this still needs work.

asimmonds’s picture

asimmonds commented

It looks like Steven has committed this patch into HEAD http://drupal.org/cvs?commit=21271

chx’s picture

chx commented
Status: Needs work » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)