I've nudged the priority to "major" since it involves loss of functionality (albeit only in IE), even though the patch itself is incredibly trivial (quite literally just removing a comma).

Thanks for submitting that, Paul.

The change looks good. In JS, a trailing comma like that is an error, unlike in PHP.

However I was not able to trigger any errors in IE8 or IE9 by dragging and dropping sections (which is where I think the code is used).

Were you seeing actual errors?

Hi Michael,
The error only shows up in courses using the Weekly course format - and I was seeing it in IE8 (I'm not sure if IE9 still throws the error and aborts processing, as I don't have IE9 available) on page load. In a course using the Weekly format in IE8, the error caused JS to stop executing before the resource dragdrop was initialised - leaving all of the icons as the move up/down icon rather than the four-way drag icon.

-Paul

Thanks, Paul.

I was able to see an error when the browser was in compatibility view.

Pushing directly to integration.

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 (23 and master), thanks!

Tested as per the testing instructions and the test passed. Good work

Gutta cavat lapidem, non vi sed saepe cadendo - Ovidio

This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

Thanks!