diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js index 0dbf4f938bb4cf26aed1ff976483a445219ab729..d5b30d6299829120584abdf27440c9c87f355191 100644 --- a/apps/files/js/filelist.js +++ b/apps/files/js/filelist.js @@ -1758,7 +1758,9 @@ updateRow: function($tr, fileInfo, options) { this.files.splice($tr.index(), 1); $tr.remove(); - $tr = this.add(fileInfo, _.extend({updateSummary: false, silent: true}, options)); + options = _.extend({silent: true}, options); + options = _.extend(options, {updateSummary: false}); + $tr = this.add(fileInfo, options); this.$fileList.trigger($.Event('fileActionsReady', {fileList: this, $files: $tr})); return $tr; }, diff --git a/apps/files/tests/js/filelistSpec.js b/apps/files/tests/js/filelistSpec.js index 9dfcdfead7fe9eab3fc930b5c475a8154adde275..e6046f2197e883a7a30b3788e08e1f9b89e6f036 100644 --- a/apps/files/tests/js/filelistSpec.js +++ b/apps/files/tests/js/filelistSpec.js @@ -812,6 +812,22 @@ describe('OCA.Files.FileList tests', function() { .toEqual(OC.imagePath('core', 'filetypes/text.svg')); }); }); + describe('Update file', function() { + it('does not change summary', function() { + var $summary = $('#filestable .summary'); + var fileData = new FileInfo({ + type: 'file', + name: 'test file', + }); + var $tr = fileList.add(fileData); + + expect($summary.find('.info').text()).toEqual('0 folders and 1 file'); + + var model = fileList.getModelForFile('test file'); + model.set({size: '100'}); + expect($summary.find('.info').text()).toEqual('0 folders and 1 file'); + }); + }) describe('List rendering', function() { it('renders a list of files using add()', function() { expect(fileList.files.length).toEqual(0);