Is there a more graceful way to enable/disable fields based on create/edit actions?

chris_n

Based on information in the forums, I am presently doing this sort of thing:

editFields = ["stock_num", "avg_cost", "uom"];

editor.on( 'initEdit', function () {
    for (var i=0; i<editFields.length; i++) {
        editor.disable(editFields[i]);
    }
} );

editor.on( 'initCreate', function () {
    for (var i=0; i<editFields.length; i++) {
        editor.enable(editFields[i]);
    }
} );

Is there a more graceful way to accomplish this?

allan

You don't need the loop - disable() and enable() will accept arrays:

editFields = ["stock_num", "avg_cost", "uom"];

editor
  .on( 'initEdit', function () { editor.disable( editFields ); } )
  .on( 'initCreate', function () { editor.enable( editFields ); } );

is about the most graceful way I can think of.

Allan

chris_n

Ahh... nice. And chaining++

allan

The only other trick I've got to clean it up a bit more is to use arrow functions from ES6:

editFields = ["stock_num", "avg_cost", "uom"];
 
editor
  .on( 'initEdit', () => editor.disable( editFields ) )
  .on( 'initCreate', () => editor.enable( editFields ) );

But you would need to use a cross compiler such as Babel JS to have it work in most current browsers...!

Ultimately perhaps Editor itself should have something like the dependent() method... (thinking on the keyboard... :-) ).

Allan

chris_n

Ultimately perhaps Editor itself should have something like the dependent() method... (thinking on the keyboard... :-) ).

That sounds like a very good idea.

Chris