On 5/7/20 2:00 PM, Dominik Csapak wrote:
> if the 'default' account exists, make the name field required and
> remove the emptytext
> 
> get the information by querying the grid store. this may be not
> up-to-date, but it is less intrusive that an extra api call that
> blocks the window
> 
> Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
> ---
>  www/manager6/dc/ACMEClusterView.js | 3 +++
>  www/manager6/node/ACME.js          | 8 ++++++--
>  2 files changed, 9 insertions(+), 2 deletions(-)
> 
> diff --git a/www/manager6/dc/ACMEClusterView.js 
> b/www/manager6/dc/ACMEClusterView.js
> index 00db8e9f..aaf2fc19 100644
> --- a/www/manager6/dc/ACMEClusterView.js
> +++ b/www/manager6/dc/ACMEClusterView.js
> @@ -29,7 +29,10 @@ Ext.define('PVE.dc.ACMEAccountView', {
>  
>       addAccount: function() {
>           let me = this;
> +         let view = me.getView();
> +         let defaultExists = view.getStore().findExact('name', 'default') 
> !== -1;
>           Ext.create('PVE.node.ACMEAccountCreate', {
> +             defaultExists,
>               taskDone: function() {
>                   me.reload();
>               },
> diff --git a/www/manager6/node/ACME.js b/www/manager6/node/ACME.js
> index 0418f406..fd33333b 100644
> --- a/www/manager6/node/ACME.js
> +++ b/www/manager6/node/ACME.js
> @@ -1,5 +1,6 @@
>  Ext.define('PVE.node.ACMEAccountCreate', {
>      extend: 'Proxmox.window.Edit',
> +    mixins: ['Proxmox.Mixin.CBind'],
>  
>      width: 400,
>      title: gettext('Register Account'),
> @@ -8,14 +9,17 @@ Ext.define('PVE.node.ACMEAccountCreate', {
>      submitText: gettext('Register'),
>      url: '/cluster/acme/account',
>      showTaskViewer: true,
> +    defaultExists: false,
>  
>      items: [
>       {
>           xtype: 'proxmoxtextfield',
>           fieldLabel: gettext('Name'),
>           name: 'name',
> -         emptyText: 'default',
> -         allowBlank: true,
> +         cbind: {
> +             emptyText: (get) => get('defaultExists') ? '' : 'default',
> +             allowBlank: (get) => !get('defaultExists'),
> +         },
>       },
>       {
>           xtype: 'proxmoxComboGrid',
> 

applied-series, thanks!

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

Reply via email to