Expected: After saving form controller form, go back to the original form that I was on that had the 'Configure form' link.
Actual: Just sits there on the form controller form.

CommentFileSizeAuthor
#2 642936-formcontroller-destination-D6.patch1.27 KBdave reid
#1 642936-formcontroller-destination-D6.patch966 bytesdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 642936-formcontroller-destination-D6.patch966 bytes

Patch that adds the destination query to the 'Configure this form' links.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 642936-formcontroller-destination-D6.patch1.27 KB

New patch that also adds a 'Cancel' link that will go back to the form.

hefox’s picture

hefox commented

working fine for me :)

mrfelton’s picture

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

good.

jcmarco’s picture

jcmarco commented

Great

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Project: Form controller » Form
Version: 6.x-2.x-dev » 6.x-1.x-dev
Status: Reviewed & tested by the community » Needs work
sun’s picture

sun
Primary language German
Location Karlsruhe
commented 
+++ form_controller.module	28 Nov 2009 18:58:12 -0000
@@ -118,6 +118,12 @@ function form_controller_form(&$form_sta
+  if (isset($_REQUEST['destination'])) {

This should be $_GET.

This review is powered by Dreditor.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Fixed

Adjusted to $_GET and committed to CVS: http://drupal.org/cvs?commit=299410

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.