Make "Field Options" input required for type "Selectbox" #182

Activity

1. Ruslan-Aleev commented on May 21, 2020

   

   Ruslan-Aleev

   on May 21, 2020

   Last edited by Ruslan-Aleev

   ContributorAuthor

   More actions

   This can be solved in a crooked way, replace the name: 'options' block in the grid (https://github.com/modmore/ClientConfig/blob/master/assets/components/clientconfig/js/mgr/widgets/window.settings.js#L93) with:

   {
   xtype: 'textarea',
   id: config.id + '-options',
   name: 'options',
   fieldLabel: _('clientconfig.options') + '*',
   description: _('clientconfig.options.desc'),
   allowBlank: (config.record && (config.record.xtype === 'modx-combo')) ? false : true,
   anchor: '100%',
   hidden: (config.record && (config.record.xtype === 'modx-combo')) ? false : true,
   value: _('clientconfig.options_default')
   }
   

   And in the clientconfig.options_default lexicon set Yes==1||No==0.
   As a result, we have an example for the Selectbox for the user, which is convenient and the field is now required.
   However, for non-Selectbox, the options parameter is not empty.

   @Mark-H What do you think, should I do PR or are these corrections not particularly useful? Or is there another solution?