The default certificate does not have a name.

Reported-by: Dietmar Maurer <diet...@proxmox.com>
Signed-off-by: Maximiliano Sandoval <m.sando...@proxmox.com>
---
 src/panel/Certificates.js | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/panel/Certificates.js b/src/panel/Certificates.js
index a522ab6..f924512 100644
--- a/src/panel/Certificates.js
+++ b/src/panel/Certificates.js
@@ -237,10 +237,16 @@ Ext.define('Proxmox.panel.Certificates', {
            {
                xtype: 'proxmoxButton',
                text: gettext('Delete Custom Certificate'),
-               confirmMsg: rec => Ext.String.format(
-                   gettext('Are you sure you want to remove the certificate 
used for {0}'),
-                   me.certById[rec.id].name,
-               ),
+               confirmMsg: function(rec) {
+                   let cert = me.certById[rec.id];
+                   if (cert.name) {
+                       return Ext.String.format(
+                           gettext('Are you sure you want to remove the 
certificate used for {0}'),
+                           cert.name,
+                       );
+                   }
+                   return Ext.String.format(gettext('Are you sure you want to 
remove the certificate'));
+               },
                callback: () => me.reload(),
                selModel: me.selModel,
                disabled: true,
-- 
2.39.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to