quarantine list: cope with undefined mail value

[pmg-gui.git] / js / SpamQuarantineOptions.js

diff --git a/js/SpamQuarantineOptions.js b/js/SpamQuarantineOptions.js
index 0a3cff78a0f1f8c54fd6ff3d01ba9b497e222353..353fe7e6219214828507b04f94470a9955ee8ede 100644 (file)
--- a/js/SpamQuarantineOptions.js
+++ b/js/SpamQuarantineOptions.js
@@ -1,4 +1,3 @@
-/*global Proxmox*/
 Ext.define('PMG.SpamQuarantineOptions', {
     extend: 'Proxmox.grid.ObjectGrid',
     alias: ['widget.pmgSpamQuarantineOptions'],
@@ -8,65 +7,64 @@ Ext.define('PMG.SpamQuarantineOptions', {
     authmodeTextHash: {
        ticket: 'Ticket',
        ldap: 'LDAP',
-       ldapticket: 'LDAP or Ticket'
+       ldapticket: 'LDAP or Ticket',
     },
 
     reportstyleTextHash: {
        none: gettext('No Reports'),
        'short': gettext('Short'),
        verbose: gettext('Verbose'),
-       outlook: gettext('Verbose') + ' (' + gettext('Outlook 2007') + ')',
-       custom: gettext('Custom')
+       custom: gettext('Custom'),
     },
-    
-    initComponent : function() {
+
+    initComponent: function() {
        var me = this;
 
        me.add_integer_row('lifetime', gettext('Lifetime (days)'),
-                          { minValue: 1, defaultValue: 7,
-                            deleteEmpty: true });
+                          {
+ minValue: 1, defaultValue: 7,
+                            deleteEmpty: true,
+});
 
        var render_authmode = function(value) {
            return me.authmodeTextHash[value] || value;
        };
 
-       /*jslint confusion: true*/
-       /* defaultValue is a string and a number*/
        me.add_combobox_row('authmode', gettext('Authentication mode'), {
            defaultValue: 'ticket',
            renderer: render_authmode,
            comboItems: [
-               ['ticket', render_authmode('ticket') ],
-               ['ldap', render_authmode('ldap') ],
-               ['ldapticket', render_authmode('ldapticket') ]]
+               ['ticket', render_authmode('ticket')],
+               ['ldap', render_authmode('ldap')],
+               ['ldapticket', render_authmode('ldapticket')]],
        });
 
        var render_reportstyle = function(value) {
            return me.reportstyleTextHash[value] || value;
        };
 
-       me.add_combobox_row('reportstyle', gettext('Report Style'), {
+       me.add_combobox_row('reportstyle', gettext('User Spamreport Style'), {
            defaultValue: 'verbose',
            renderer: render_reportstyle,
            comboItems: [
-               ['none', render_reportstyle('none') ],
-               ['short', render_reportstyle('short') ],
-               ['verbose', render_reportstyle('verbose') ],
-               ['outlook', render_reportstyle('outlook') ],
-               ['custom', render_reportstyle('custom') ]]
+               ['none', render_reportstyle('none')],
+               ['short', render_reportstyle('short')],
+               ['verbose', render_reportstyle('verbose')],
+               ['custom', render_reportstyle('custom')]],
        });
-       /*jslint confusion: false*/
 
        me.add_text_row('hostname', gettext('Quarantine Host'),
                        { deleteEmpty: true, defaultValue: Proxmox.Utils.noneText });
+       me.add_integer_row('port', gettext('Quarantine port'),
+                       { deleteEmpty: true, defaultValue: Proxmox.Utils.defaultText });
        me.add_text_row('mailfrom', gettext("EMail 'From:'"),
                        { deleteEmpty: true, defaultValue: Proxmox.Utils.noneText });
 
        me.add_boolean_row('viewimages', gettext('View images'),
-                          { defaultValue: 1});
-       
+                          { defaultValue: 1 });
+
        me.add_boolean_row('allowhrefs', gettext('Allow HREFs'),
-                          {defaultValue: 1 });
+                          { defaultValue: 1 });
 
        var baseurl = '/config/spamquar';
 
@@ -78,17 +76,18 @@ Ext.define('PMG.SpamQuarantineOptions', {
                xtype: 'proxmoxButton',
                disabled: true,
                handler: function() { me.run_editor(); },
-               selModel: me.selModel
+               selModel: me.selModel,
            }],
            url: '/api2/json' + baseurl,
            editorConfig: {
-               url: '/api2/extjs' + baseurl
+               url: '/api2/extjs' + baseurl,
+               onlineHelp: 'pmgconfig_spamdetector_quarantine',
            },
            interval: 5000,
            cwidth1: 200,
            listeners: {
-               itemdblclick: me.run_editor
-           }
+               itemdblclick: me.run_editor,
+           },
        });
 
        me.callParent();
@@ -96,5 +95,5 @@ Ext.define('PMG.SpamQuarantineOptions', {
        me.on('activate', me.rstore.startUpdate);
        me.on('destroy', me.rstore.stopUpdate);
        me.on('deactivate', me.rstore.stopUpdate);
-    }
+    },
 });