Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix: don't need to reload the whole page to refresh_children from the container view

Browse Source
This commit is contained in:
Braden MacDonald
2014-12-10 20:59:49 -08:00
committed by E. Kolpakov
parent e498872ab1
commit 904007a9e4
1 changed files with 12 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
common/lib/xmodule/xmodule/public/js/library_content_edit.js
View File

@@ -1,10 +1,11 @@
/* JavaScript for special editing operations that can be done on LibraryContentXBlock */
window.LibraryContentAuthorView = function (runtime, element) {
"use strict";
var usage_id = $(element).data('usage-id');
var $element = $(element);
var usage_id = $element.data('usage-id');
// The "Update Now" button is not a child of 'element', as it is in the validation message area
// But it is still inside this xblock's wrapper element, which we can easily find:
var $wrapper = $(element).parents('*[data-locator="'+usage_id+'"]');
var $wrapper = $element.parents('*[data-locator="'+usage_id+'"]');
// We can't bind to the button itself because in the bok choy test environment,
// it may not yet exist at this point in time... not sure why.
@@ -21,12 +22,15 @@ window.LibraryContentAuthorView = function (runtime, element) {
state: 'end',
element: element
});
// runtime.refreshXBlock(element);
// The above does not work, because this XBlock's runtime has no reference
// to the page (XBlockContainerPage). Only the Vertical XBlock's runtime has
// a reference to the page, and we have no way of getting a reference to it.
// So instead we:
location.reload();
if ($element.closest('.wrapper-xblock').is(':not(.level-page)')) {
// We are on a course unit page. The notify('save') should refresh this block,
// but that is only working on the container page view of this block.
// Why? On the unit page, this XBlock's runtime has no reference to the
// XBlockContainerPage - only the top-level XBlock (a vertical) runtime does.
// But unfortunately there is no way to get a reference to our parent block's
// JS 'runtime' object. So instead we must refresh the whole page:
location.reload();
}
});
});
};