You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I recently switched CONFIG to use AnyConstructor more widely. I expect that this will be a positive change in most cases but for example CONFIG.Canvas.groups needs a specific constructor signature because of Canvas##createGroups
#createGroups(parentName,parent){for(const[name,config]ofObject.entries(CONFIG.Canvas.groups)){if(config.parent!==parentName)continue;constgroup=newconfig.groupClass();Object.defineProperty(this,name,{value: group,writable: false});// Reference on the CanvasObject.defineProperty(parent,name,{value: group,writable: false});// Reference on the parentparent.addChild(group);this.#createGroups(name,group);// Recursive}}
This specific case has of course been accounted for already. What's missing is doing this for all classes in CONFIG to ensure their valid constructors.
The text was updated successfully, but these errors were encountered:
I recently switched
CONFIG
to useAnyConstructor
more widely. I expect that this will be a positive change in most cases but for exampleCONFIG.Canvas.groups
needs a specific constructor signature because ofCanvas##createGroups
This specific case has of course been accounted for already. What's missing is doing this for all classes in CONFIG to ensure their valid constructors.
The text was updated successfully, but these errors were encountered: