[INFO REQ] OnToggle in the CheckColumn not working

[henrique]

Yes, in the https://bryntum.com/examples/grid/basic/, using the sample in the issues tracker, the toast isn't showed.

[marcio]

Hello Henrique,

With the latest version, and the following configuration (replacing the Toast for a console.log, just for testing purposes), all worked as expected.

        {
            type            : 'check',
            field           : 'done',
            flex            : 1,
            align           : 'center',
            text            : 'CheckColumn',
            showCheckAll    : true,
            callOnFunctions : true,
            onToggle        : () => console.log('onToggle clicked'),
            onBeforeToggle  : () => console.log('onBeforeToggle clicked'),
            onToggleAll     : () => console.log('onToggleAll clicked')
        }

[henrique]

The callOnFunctions, this property I'm no using. When fill this property has "true", it's works! Thanks!

[henrique]

When I use only the "onToggle", give me an error:

grid.module.js?461600:10 
       Uncaught TypeError: Cannot read properties of undefined (reading 'substring')
    at CheckColumn.resolveCallback (grid.module.js?461600:10:33682)
    at CheckColumn.callback (grid.module.js?461600:10:33294)
    at CheckColumn.trigger (grid.module.js?461600:10:68268)
    at CheckColumn.onBeforeCheckboxChange (grid.module.js?461600:12:252387)
    at Checkbox.trigger (grid.module.js?461600:10:69674)
    at Checkbox.triggerChange (grid.module.js?461600:12:54411)
    at Checkbox.internalOnChange (grid.module.js?461600:12:54057)
    at HTMLInputElement.o (grid.module.js?461600:10:162798)
    

Using the example of the "Basic Demo".

        {
            type            : 'check',
            field           : 'done',
            flex            : 1,
            align           : 'center',
            text            : 'CheckColumn',
            showCheckAll    : true,
            callOnFunctions : true,
            onToggle        : () => console.log('onToggle clicked'),
//            onBeforeToggle  : () => console.log('onBeforeToggle clicked'),
//            onToggleAll     : () => console.log('onToggleAll clicked')
        }

[henrique]

Sill not working...

[marcio]

Hey henrique,

That's a bug! I created a ticket to fix it - https://github.com/bryntum/support/issues/5136

As a workaround for now you can set some empty functions for the onBeforeToggle, onToggle and onToggleAll

[henrique]

Yes, it's working now!