From 00b886c4921ee395f9f9d9a08c33489c8ecd6536 Mon Sep 17 00:00:00 2001
From: Haocen Xu <admin@haocen.me>
Date: Sun, 25 Aug 2019 12:00:59 -0400
Subject: [PATCH] Refactor attachment editable logic to avoid undesired
 assumption

---
 js/privatebin.js | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/js/privatebin.js b/js/privatebin.js
index 6cde5dc0..b141779b 100644
--- a/js/privatebin.js
+++ b/js/privatebin.js
@@ -2145,18 +2145,6 @@ jQuery.PrivateBin = (function($, RawDeflate) {
             return isPreview;
         };
 
-        /**
-         * gets the visibility of the editor
-         *
-         * @name   Editor.isHidden
-         * @function
-         * @return {bool}
-         */
-        me.isHidden = function()
-        {
-            return $message.hasClass('hidden');
-        };
-
         /**
          * reset the Editor view
          *
@@ -2855,7 +2843,7 @@ jQuery.PrivateBin = (function($, RawDeflate) {
                 evt.stopPropagation();
                 evt.preventDefault();
 
-                if (Editor.isHidden()) {
+                if (TopNav.isAttachmentReadonly()) {
                     return false;
                 }
 
@@ -2873,7 +2861,7 @@ jQuery.PrivateBin = (function($, RawDeflate) {
 
             $(document).draghover().on({
                 'draghoverstart': function(e) {
-                    if (Editor.isHidden()) {
+                    if (TopNav.isAttachmentReadonly()) {
                         e.stopPropagation();
                         e.preventDefault();
                         return false;
@@ -2903,7 +2891,7 @@ jQuery.PrivateBin = (function($, RawDeflate) {
          */
         function addClipboardEventHandler() {
             $(document).on('paste', function (event) {
-                if (Editor.isHidden()) {
+                if (TopNav.isAttachmentReadonly()) {
                     event.stopPropagation();
                     event.preventDefault();
                     return false;
@@ -3845,6 +3833,18 @@ jQuery.PrivateBin = (function($, RawDeflate) {
             $formatter.parent().find(`a[data-format="${format}"]`).click();
         }
 
+        /**
+         * returns if attachment dropdown is readonly, not editable
+         * 
+         * @name   TopNav.isAttachmentReadonly
+         * @function
+         * @return {bool}
+         */
+        me.isAttachmentReadonly = function()
+        {
+            return $attach.hasClass('hidden');
+        }
+
         /**
          * init navigation manager
          *