[pve-devel] applied: [PATCH widget-toolkit] htmlEncode DNS fields

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Oct 22 08:58:55 CEST 2018


On 10/19/18 12:36 PM, Dominik Csapak wrote:
> and change to a better style of the properties

slightly reworded to:
and change to a better code formatting of the properties

as I was confused why the GUI looked exactly the same :-)
thanks!

> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  node/DNSView.js | 22 ++++++++++++++++++----
>  1 file changed, 18 insertions(+), 4 deletions(-)
> 
> diff --git a/node/DNSView.js b/node/DNSView.js
> index b0f0973..29afe9f 100644
> --- a/node/DNSView.js
> +++ b/node/DNSView.js
> @@ -22,10 +22,24 @@ Ext.define('Proxmox.node.DNSView', {
>  	    interval: 1000,
>  	    run_editor: run_editor,
>  	    rows: {
> -		search: { header: 'Search domain', required: true },
> -		dns1: { header: gettext('DNS server') + " 1", required: true },
> -		dns2: { header: gettext('DNS server') + " 2" },
> -		dns3: { header: gettext('DNS server') + " 3" }
> +		search: {
> +		    header: 'Search domain',
> +		    required: true,
> +		    renderer: Ext.htmlEncode
> +		},
> +		dns1: {
> +		    header: gettext('DNS server') + " 1",
> +		    required: true,
> +		    renderer: Ext.htmlEncode
> +		},
> +		dns2: {
> +		    header: gettext('DNS server') + " 2",
> +		    renderer: Ext.htmlEncode
> +		},
> +		dns3: {
> +		    header: gettext('DNS server') + " 3",
> +		    renderer: Ext.htmlEncode
> +		}
>  	    },
>  	    tbar: [
>  		{
> 





More information about the pve-devel mailing list