Moodle

When the section is highlighted, background update on section move fails.

Details

Description

That is why we need functional tests.

It appeared that if we have a highlighted section, the moves that affect the highlighted section location fails on the background. See AJAX response for details.

Issue Links

required writing of QA test

Functional Test - A test of a single Moodle feature MDLQA-2396 CLONE - A highlighted section with no activities or resources can be moved using drag and drop in IE8.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2397 CLONE - A highlighted section with no activities or resources can be moved using drag and drop in IE9.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2398 CLONE - A highlighted section with no activities or resources can be moved using drag and drop in Firefox.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2399 CLONE - A highlighted section with no activities or resources can be moved using drag and drop in Google Chrome.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2400 CLONE - A highlighted section with no activities or resources can be moved using drag and drop in Safari.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2401 CLONE - A highlighted section with no activities or resources can be moved using drag and drop in Opera.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2402 CLONE - A highlighted section with a folder resource can be moved using drag and drop in IE8.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2403 CLONE - A highlighted section with a folder resource can be moved using drag and drop in IE9.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2404 CLONE - A highlighted section with a folder resource can be moved using drag and drop in Firefox.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2405 CLONE - A highlighted section with a folder resource can be moved using drag and drop in Google Chrome.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2406 CLONE - A highlighted section with a folder resource can be moved using drag and drop in Safari.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2407 CLONE - A highlighted section with a folder resource can be moved using drag and drop in Opera.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2408 CLONE - A highlighted section with multiple resources and activities can be moved using drag and drop in IE8.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2410 CLONE - A highlighted section with multiple resources and activities can be moved using drag and drop in IE9.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2411 CLONE - A highlighted section with multiple resources and activities can be moved using drag and drop in Firefox.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2412 CLONE - A highlighted section with multiple resources and activities can be moved using drag and drop in Google Chrome.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2413 CLONE - A highlighted section with multiple resources and activities can be moved using drag and drop in Safari.

  • Passed - The test passed

Functional Test - A test of a single Moodle feature MDLQA-2414 CLONE - A highlighted section with multiple resources and activities can be moved using drag and drop in Opera.

  • Passed - The test passed
will help resolve

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-32742 Course highlight/visibility issues with drag-drop

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Ruslan Kabalin added a comment -

Making it major and added instructions. Patch is coming.

Show
Ruslan Kabalin added a comment - Making it major and added instructions. Patch is coming.
Hide
Permalink
Ruslan Kabalin added a comment -

Andrew, can you please review this.

Show
Ruslan Kabalin added a comment - Andrew, can you please review this.
Hide
Permalink
Andrew Nicols added a comment -

Looks good to me.
Submit for peer review when ready.

Show
Andrew Nicols added a comment - Looks good to me. Submit for peer review when ready.
Hide
Permalink
Ruslan Kabalin added a comment -

Ready for integration

Show
Ruslan Kabalin added a comment - Ready for integration
Hide
Permalink
Dan Poltawski added a comment -

Perhaps you can help with the functional tests which are being written by writing some tests and adding the qa_test_required label.

Show
Dan Poltawski added a comment - Perhaps you can help with the functional tests which are being written by writing some tests and adding the qa_test_required label.
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

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

Show
Eloy Lafuente (stronk7) added a comment - 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
Hide
Permalink
Dan Poltawski added a comment -

Adding the QA test required label so Tim can check if we coverage for this, if not hopefully Ruslan can help with doing that.

Show
Dan Poltawski added a comment - Adding the QA test required label so Tim can check if we coverage for this, if not hopefully Ruslan can help with doing that.
Hide
Permalink
Dan Poltawski added a comment -

Integrated, thanks

Show
Dan Poltawski added a comment - Integrated, thanks
Hide
Permalink
Ankit Agarwal added a comment -

working as expected
Passing.
Thanks

Show
Ankit Agarwal added a comment - working as expected Passing. Thanks
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

U P S T R E A M I Z E D !

Many thanks for the hard work, closing this as fixed.

Ciao

Show
Eloy Lafuente (stronk7) added a comment - U P S T R E A M I Z E D ! Many thanks for the hard work, closing this as fixed. Ciao
Hide
Permalink
Ruslan Kabalin added a comment -

Thanks Eloy!

Show
Ruslan Kabalin added a comment - Thanks Eloy!

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: