From beba93b147eb8a6644f7621f7de1512c9a29e8a7 Mon Sep 17 00:00:00 2001 From: Emmanuel Kasper Date: Wed, 14 Jun 2017 11:22:20 +0200 Subject: [PATCH] Fix SafeDestroy when entering a numeric resource ID In a previous commit, the inputField of SafeDestroy.js was converted from a numberfield to a textfield to allow ceph pools names to be entered. This means the type of 'value' changed from number to string and the strong equal test === was failing against a numeric id. To solve this, we now always stringify the resource id before comparing. --- www/manager6/window/SafeDestroy.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/window/SafeDestroy.js b/www/manager6/window/SafeDestroy.js index 5093dbf1..23e4c04a 100644 --- a/www/manager6/window/SafeDestroy.js +++ b/www/manager6/window/SafeDestroy.js @@ -31,7 +31,7 @@ Ext.define('PVE.window.SafeDestroy', { change: function(f, value) { var view = this.getView(); var removeButton = this.lookupReference('removeButton'); - if (value === view.getItem().id) { + if (value === view.getItem().id.toString()) { removeButton.enable(); } else { removeButton.disable();