adding a patch that fixes it but I'm not sure if this will get past the integrator

NOTE TO INTEGRATOR - the above patch will cherry-pick cleanly onto 20Stable and 21Stable branches - it should also be applied there.

The old code uses the functions select_all_in/deselect_all_in which are old functions that should be deprec and replaced by yui3 stuff - the only reference I could find to using these functions in master was the choice mod - so we can probably remove those functions now.

This patch replaces the existing call to another function that should probably be deprec too: checkall/checknone - these functions are used in a handful of other places in Moodle but I couldn't find a new "core" replacement for it.

A new issue/bug could be created to convert all existing selectall/selectnone style calls in moodle to use a new YUI 3 function - but that probably isn't coming any time soon - I guess as this fixes a bug and uses similar code to other parts of moodle it should be ok to pass and integrate?

thanks.

Thanks Dan - this has been integrated now (merged to master - cherry-picked to 21, 20).

You are quite right really both of those function should be deprecated at some point and converted to a single YUI function.

Cheers
Sam

Hi Sam,

The patch for m2.0 is not integrate appropriately. It has control version character such as '<<<<<<< HEAD'. I'm attaching screenshot from git.moodle.org page.

The patch works great on m2.1 and master.

Wow thanks for spotting that Rosie!
I must've forgotten to add the fixed file before committing the merge and pushing it to integration.
I've now rectified that and its up there again

Cheers
Sam

Thanks for fixing this Sam.

Its all good now.

test passed.

Many thanks for the hard work developing and testing this. It has been spread to cvs and git upstream repositories.

Closing, ciao