Just added a default for combined feedback field shownumcorrect
line is
$mform->setDefault('shownumcorrect', true);

Confirmed expected functionality by:
Created a new Matching question. shownumcorrect default is now checked be default. Saved question. Opened it unchecked and saved. Reopened. field still unchecked.

+1, thanks Colin. Submitting for integration.

Should this go into 2.4? I think it is a small improvement and it would do no harm to add it to 2.4. It really depends how strict the integrators want to be with the 'bug-fixes only in stable' policy.

(Colin, For future reference, it would be better if the first line of the commit comment was a bit shortere. If you look in github https://github.com/colchambers/moodle/compare/master...MDL-37465, try to avoid triggering the ... thing in github's display.)

Rebased on latest master and pushed. Thanks for comments tim.

Tried a shorter commit message. Fits but is it acceptable? You call.

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

Thanks Colin. That is better.

Integrated (master only), thanks!

I know it's a minor and safe change, but recently we have decided to become more seriously-rigid, so this cannot land to stables, sorry.

FYI, just some hours ago, the new "seriously-rigid" approach has been documented (after agreement by the iTeam) at:

http://docs.moodle.org/dev/Integration_Review#Process_for_requesting_a_non_bug-fix_backport

Thanks and ciao

Test result: Success!

Tested in Master only.

A brilliant future is awaiting us out there, better with your code. Let's look towards the future together, this is now closed.

(and won't be revisiting it unless some regression is found)

Thanks and ciao