The message 'By proceeding you won't be able to go back and edit your answers' is coming when clicking the 'Back' link in the last question form.I think the message should only come when we click on the 'Finish' button

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ultimike’s picture

I ran into this same bug. Patch is attached.

The patch just adds some jQuery to ensure that it was the "Finish" button that was clicked, and not the "Back" button.

Thanks,
-mike

ultimike’s picture

Actually, here's a better patch. I was testing the button value (which can be different depending on the language) when I should have been testing based on the button's ID.

Thanks,
-mike

sjager’s picture

Component: Code - other » Code - Quiz core
FileSize
909 bytes

I ran into this problem but I took a different approach by binding directly to the onclick action of the submit and submit as blank buttons. I also added a conditional inside of the submit button action to check the jump menu value since the question number jump menu (when enabled) will force a click on the `#edit-submit` button (and show the Confirmation Warning message)

eiriksm’s picture

Status: Active » Reviewed & tested by the community

Patch in #3 works great.

The last submitted patch, 1: back_button_bug-2182483-1.patch, failed testing.

The last submitted patch, 2: back_button_bug-2182483-2.patch, failed testing.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 3: back_button_bug-2182483-3.patch, failed testing.

eiriksm’s picture

Version: 7.x-4.0-beta2 » 7.x-4.x-dev
FileSize
896 bytes

Oh well, updated patch against 7.x-4.x then.

eiriksm’s picture

Status: Needs work » Needs review

Setting to needs review.

falcon’s picture

Status: Needs review » Fixed

Great job!

  • falcon committed 954db51 on 7.x-4.x authored by eiriksm
    Issue #2182483 by ultimike, eiriksm, sjager: Error in js confirm message...

Status: Fixed » Closed (fixed)

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