diff options
| author | Roeland Jago Douma <roeland@famdouma.nl> | 2017-05-04 13:23:04 +0200 |
|---|---|---|
| committer | Roeland Jago Douma <roeland@famdouma.nl> | 2017-05-04 13:29:49 +0200 |
| commit | 8d66e325a9375dc06c8105a30e795965e1b73773 (patch) | |
| tree | cabee441f557b6b61529da375443e777c7a60298 /apps/files/tests | |
| parent | b04102aab364a62c7a7c3f5cbae9fd0d672842f5 (diff) | |
| download | nextcloud-server-8d66e325a9375dc06c8105a30e795965e1b73773.tar.gz nextcloud-server-8d66e325a9375dc06c8105a30e795965e1b73773.zip | |
Do not reload the filelist on first open
Fixes: #4644
Without this patch the filelist would always reload. However since not
all the correct data was set yet it would often:
1. fireoff a propfind to ../webdav/
2. fireoff a propfind to ../webdav/<PATH>
When just opening the file list those are the same so the result is just
fine. However if opening a direct link it means that there is a race
condition on which finishes first.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Diffstat (limited to 'apps/files/tests')
| -rw-r--r-- | apps/files/tests/js/filelistSpec.js | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/apps/files/tests/js/filelistSpec.js b/apps/files/tests/js/filelistSpec.js index 934aa054fb6..6b403e7fa85 100644 --- a/apps/files/tests/js/filelistSpec.js +++ b/apps/files/tests/js/filelistSpec.js @@ -1509,6 +1509,12 @@ describe('OCA.Files.FileList tests', function() { }); it('reloads the list when leaving hidden state', function() { var reloadStub = sinon.stub(fileList, 'reload'); + + // First show should not trigger + $('#app-content-files').trigger(new $.Event('show')); + expect(reloadStub.calledOnce).toEqual(false); + + // Second show should! $('#app-content-files').trigger(new $.Event('show')); expect(reloadStub.calledOnce).toEqual(true); reloadStub.restore(); |