When unlimited retakes of quiz is allowed, confirmation message implies answers can no longer be changed.

I've tested a patch which shows a different confirmation message (attached) if the quiz has unlimited attempts or if the user hasn't yet exhausted their attempts.

I've published the patch here: https://github.com/mackensen/moodle/commit/136667bd8e9b4ff9efb0c3ca474827d1a839d7ee.

I'm afraid I don't really like the change. The more words you have in an "Are you sure" message, the less likely anyone is to read an understand it.

So I think we should be aiming for something shorter, while still being clear. I would also like to use the same message in all situations, if possible.

Also, some technical comments on your patch:
1. Don't put comments like MDL-123 in the code. It is just clutter. If you need that information, use git blame.
2. You commit message is not in the right format. You must right the issue number with a -, or it will not automatically be made into a link. You are not specific which messages is being improved.
3. A very trivial change has added two new local variables. There must be a neater way to write that code.

Thanks for responding, Tim. I think we do better to err on the side of an informative message. The users who don't read messages won't be affected either way; the users who do will appreciate the information. As for the local variables; this was my thinking:
1. I didn't want to make two calls to the same class variable twice in the same if statement.
2. I didn't want to split up the add_confirm_action() logic.

Anyway, I've rewritten the patch to address the stylistic concerns and to eliminate one of the local variables. I don't see how the other one can be eliminated without an evil looking ternary function, and above all code like this needs to be human-readable. New patch is here: https://github.com/mackensen/moodle/commit/465fc3c0e1e31e7f4c859052494d56eb0d5a17de.

Here is my version of a simplified and non-confusing submit confirmation. Comments please.

Note that the change for this bug relies on the change in MDL-28195 - you need to apply that patch first.

I also made a similar change to the start attempt are-you-sure. (I changed the 'Yes' button to 'Start attempt'.)

Comments please. I will submit this for integration after MDL-28195 is approved.

This works for me (and I also like the change in MDL-28195).

Rebased, due to changes in MDL-28195. That has been approved, at least in principle, so I am submitting this for integration too now.

Please can we have this on both the 2.1 and master branches.

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

TIA and ciao

Integrated, thanks (master and 21)!

Looks like a good change to me

Congratulations - this fix has just been released in the weeklies.