Apps > Profiling - disabled button event bug fixed

minjk-bl · minjk-bl · commit 5ca771da596d · 2021-09-27T13:28:23.000+09:00
diff --git a/src/common/vpProfiling.js b/src/common/vpProfiling.js
@@ -253,13 +253,11 @@ define([
                         if (!checking) {
                             return;
                         }
-                        if (msg.content['name'] == 'stderr') {
-                            if (msg.content['text'].includes('not found')) {
-                                $(that.wrapSelector('.' + VP_PF_INSTALL_BTN)).text('Install');
-                                // set enabled
-                                if ($(that.wrapSelector('.' + VP_PF_INSTALL_BTN)).hasClass('disabled')) {
-                                    $(that.wrapSelector('.' + VP_PF_INSTALL_BTN)).removeClass('disabled');
-                                }
+                        if (msg.content['name'] == 'stderr' || msg.content['text'].includes('not found')) {
+                            $(that.wrapSelector('.' + VP_PF_INSTALL_BTN)).text('Install');
+                            // set enabled
+                            if ($(that.wrapSelector('.' + VP_PF_INSTALL_BTN)).hasClass('disabled')) {
+                                $(that.wrapSelector('.' + VP_PF_INSTALL_BTN)).removeClass('disabled');
                             }
                         } else {
                             $(that.wrapSelector('.' + VP_PF_INSTALL_BTN)).text('Installed');
@@ -351,7 +349,7 @@ define([
         });
 
         // click install
-        $(document).on('click', this.wrapSelector('.' + VP_PF_INSTALL_BTN), function(event) {
+        $(document).on('click', this.wrapSelector('.' + VP_PF_INSTALL_BTN + ':not(.disabled)'), function(event) {
             vpCommon.cellExecute([{command: '!pip install pandas-profiling', exec:true, type:'code'}]);
         });
 
