[REACT] propertyConfigNames should be processed by the wrapper

[jsakalos]

Forum post

I think we should also process propertyConfigNames in addition to propertyNames. The only difference between the two is that properties are not set during config time and propConfigs can be set both at config and run time.

The code should then read:

function shouldComponentUpdate(component: any, nextProps: Readonly, nextState: Readonly): boolean {
const { props, instance, propertyNames, propertyConfigNames } = component;

propertyNames.concat(propertyConfigNames).forEach((prop: string) => {
    if (props[prop] !== nextProps[prop]) {
        // Check if property is not a config
        applyPropValue(instance, prop, nextProps[prop], false);
    }
});

// Reflect JSX cell changes
return nextState?.generation !== component.state?.generation;

}
(untested)

I'll try the above and if our tests pass it'll be released in the next version.

[jsakalos]

See the post here: https://forum.bryntum.com/viewtopic.php?p=160211#p160211