From: Roeland Jago Douma <roeland@famdouma.nl>
Date: Fri, 2 Nov 2018 21:45:25 +0000 (+0100)
Subject: Set the filemodel before rending the detailsview
X-Git-Tag: v14.0.4RC1~28^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2d153adb6465fe98af49045d37068b3efc43405d;p=nextcloud-server.git

Set the filemodel before rending the detailsview

fixes #10934

Else it triggers the rendering two times. Resulting is weird state in
for example the comments. Because the comments for OLD_FILEID are
retrieved but then the model is changed to NEW_FILEID. But the old
comments still get in and get parsed.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
---

diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js
index 42b80189287..a7dadbbab3e 100644
--- a/apps/files/js/filelist.js
+++ b/apps/files/js/filelist.js
@@ -568,8 +568,8 @@
 			}
 
 			this._currentFileModel = model;
-			this._detailsView.render();
 			this._detailsView.setFileInfo(model);
+			this._detailsView.render();
 			this._detailsView.$el.scrollTop(0);
 		},