From 16d96fc9320146356cc0c3d44852bd9e61d5b0bf Mon Sep 17 00:00:00 2001
From: Marcin Kuzminski <marcin@rhodecode.com>
Date: Sat, 12 Oct 2019 22:48:30 +0200
Subject: [PATCH] comments: fix JS for attachments for anonymous users.

---
 rhodecode/public/js/src/rhodecode/comments.js | 108 +++++++++---------
 1 file changed, 54 insertions(+), 54 deletions(-)

diff --git a/rhodecode/public/js/src/rhodecode/comments.js b/rhodecode/public/js/src/rhodecode/comments.js
index 2e9728d2..ed9ce4b6 100644
--- a/rhodecode/public/js/src/rhodecode/comments.js
+++ b/rhodecode/public/js/src/rhodecode/comments.js
@@ -680,69 +680,69 @@ var CommentsController = function() {
             {'repo_name': templateContext.repo_name,
                      'commit_id': templateContext.commit_data.commit_id})
 
-        var previewTmpl = $(formElement).find('.comment-attachment-uploader-template').get(0).innerHTML;
-        var selectLink = $(formElement).find('.pick-attachment').get(0);
-        $(formElement).find('.comment-attachment-uploader').dropzone({
-            url: storeUrl,
-            headers: {"X-CSRF-Token": CSRF_TOKEN},
-            paramName: function () {
-                return "attachment"
-            }, // The name that will be used to transfer the file
-            clickable: selectLink,
-            parallelUploads: 1,
-            maxFiles: 10,
-            maxFilesize: templateContext.attachment_store.max_file_size_mb,
-            uploadMultiple: false,
-            autoProcessQueue: true, // if false queue will not be processed automatically.
-            createImageThumbnails: false,
-            previewTemplate: previewTmpl,
+        var previewTmpl = $(formElement).find('.comment-attachment-uploader-template').get(0);
+        if (previewTmpl !== undefined){
+            var selectLink = $(formElement).find('.pick-attachment').get(0);
+            $(formElement).find('.comment-attachment-uploader').dropzone({
+                url: storeUrl,
+                headers: {"X-CSRF-Token": CSRF_TOKEN},
+                paramName: function () {
+                    return "attachment"
+                }, // The name that will be used to transfer the file
+                clickable: selectLink,
+                parallelUploads: 1,
+                maxFiles: 10,
+                maxFilesize: templateContext.attachment_store.max_file_size_mb,
+                uploadMultiple: false,
+                autoProcessQueue: true, // if false queue will not be processed automatically.
+                createImageThumbnails: false,
+                previewTemplate: previewTmpl.innerHTML,
 
-            accept: function (file, done) {
-                done();
-            },
-            init: function () {
+                accept: function (file, done) {
+                    done();
+                },
+                init: function () {
 
-                this.on("sending", function (file, xhr, formData) {
-                    $(formElement).find('.comment-attachment-uploader').find('.dropzone-text').hide();
-                    $(formElement).find('.comment-attachment-uploader').find('.dropzone-upload').show();
-                });
+                    this.on("sending", function (file, xhr, formData) {
+                        $(formElement).find('.comment-attachment-uploader').find('.dropzone-text').hide();
+                        $(formElement).find('.comment-attachment-uploader').find('.dropzone-upload').show();
+                    });
 
-                this.on("success", function (file, response) {
-                    $(formElement).find('.comment-attachment-uploader').find('.dropzone-text').show();
-                    $(formElement).find('.comment-attachment-uploader').find('.dropzone-upload').hide();
+                    this.on("success", function (file, response) {
+                        $(formElement).find('.comment-attachment-uploader').find('.dropzone-text').show();
+                        $(formElement).find('.comment-attachment-uploader').find('.dropzone-upload').hide();
 
-                    var isRendered = false;
-                    var ext = file.name.split('.').pop();
-                    var imageExts = templateContext.attachment_store.image_ext;
-                    if (imageExts.indexOf(ext) !== -1){
-                        isRendered = true;
-                    }
-
-                    insertAttachmentText(cm, file.name, response.repo_fqn_access_path, isRendered)
-                });
-
-                this.on("error", function (file, errorMessage, xhr) {
-                    $(formElement).find('.comment-attachment-uploader').find('.dropzone-upload').hide();
-
-                    var error = null;
-
-                    if (xhr !== undefined){
-                        var httpStatus = xhr.status + " " + xhr.statusText;
-                        if (xhr.status >= 500) {
-                            error = httpStatus;
+                        var isRendered = false;
+                        var ext = file.name.split('.').pop();
+                        var imageExts = templateContext.attachment_store.image_ext;
+                        if (imageExts.indexOf(ext) !== -1){
+                            isRendered = true;
                         }
-                    }
 
-                    if (error === null) {
-                        error = errorMessage.error || errorMessage || httpStatus;
-                    }
-                    $(file.previewElement).find('.dz-error-message').html('ERROR: {0}'.format(error));
+                        insertAttachmentText(cm, file.name, response.repo_fqn_access_path, isRendered)
+                    });
 
-                });
-            }
-        });
+                    this.on("error", function (file, errorMessage, xhr) {
+                        $(formElement).find('.comment-attachment-uploader').find('.dropzone-upload').hide();
 
+                        var error = null;
 
+                        if (xhr !== undefined){
+                            var httpStatus = xhr.status + " " + xhr.statusText;
+                            if (xhr.status >= 500) {
+                                error = httpStatus;
+                            }
+                        }
+
+                        if (error === null) {
+                            error = errorMessage.error || errorMessage || httpStatus;
+                        }
+                        $(file.previewElement).find('.dz-error-message').html('ERROR: {0}'.format(error));
+
+                    });
+                }
+            });
+        }
       return commentForm;
   };