Skip to content
Snippets Groups Projects
Commit 2912d0e3 authored by Thomas Müller's avatar Thomas Müller
Browse files

Merge pull request #19499 from owncloud/files-versions-properlyreloadafterrevert

Reload versions list after revert
parents f9093688 b6114ba2
Branches
No related tags found
No related merge requests found
...@@ -67,6 +67,11 @@ ...@@ -67,6 +67,11 @@
return this.fetch({remove: false}); return this.fetch({remove: false});
}, },
reset: function() {
this._currentIndex = 0;
OC.Backbone.Collection.prototype.reset.apply(this, arguments);
},
parse: function(result) { parse: function(result) {
var results = _.map(result.data.versions, function(version) { var results = _.map(result.data.versions, function(version) {
var revision = parseInt(version.version, 10); var revision = parseInt(version.version, 10);
......
...@@ -85,12 +85,18 @@ ...@@ -85,12 +85,18 @@
ev.preventDefault(); ev.preventDefault();
revision = $target.attr('data-revision'); revision = $target.attr('data-revision');
this.$el.find('.versions, .showMoreVersions').addClass('hidden');
var versionModel = this.collection.get(revision); var versionModel = this.collection.get(revision);
versionModel.revert({ versionModel.revert({
success: function() { success: function() {
// reset and re-fetch the updated collection // reset and re-fetch the updated collection
self.$versionsContainer.empty();
self.collection.setFileInfo(fileInfoModel); self.collection.setFileInfo(fileInfoModel);
self.collection.fetch(); self.collection.reset([], {silent: true});
self.collection.fetchNext();
self.$el.find('.versions').removeClass('hidden');
// update original model // update original model
fileInfoModel.trigger('busy', fileInfoModel, false); fileInfoModel.trigger('busy', fileInfoModel, false);
...@@ -157,7 +163,7 @@ ...@@ -157,7 +163,7 @@
if (fileInfo) { if (fileInfo) {
this.render(); this.render();
this.collection.setFileInfo(fileInfo); this.collection.setFileInfo(fileInfo);
this.collection.reset({silent: true}); this.collection.reset([], {silent: true});
this.nextPage(); this.nextPage();
} else { } else {
this.render(); this.render();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment