Fix for mixin constructor names

Author: Mike Bond

packages/dev/core/src/Materials/PBR/openPbrMaterial.ts

@@ -207,8 +207,6 @@ class OpenPBRMaterialDefinesBase extends UVDefinesMixin(MaterialDefines) {}
  * @internal
  */
 export class OpenPBRMaterialDefines extends ImageProcessingDefinesMixin(OpenPBRMaterialDefinesBase) {
-    public PBR = true;
-
     public NUM_SAMPLES = "0";
     public REALTIME_FILTERING = false;
     public IBL_CDF_FILTERING = false;
@@ -386,7 +384,6 @@ export class OpenPBRMaterialDefines extends ImageProcessingDefinesMixin(OpenPBRM
     public override reset(): void {
         super.reset();
         this.ALPHATESTVALUE = "0.5";
-        this.PBR = true;
         this.NORMALXYSCALE = true;
     }
 }
@@ -616,6 +613,12 @@ export class OpenPBRMaterial extends OpenPBRMaterialBase {
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     private _coatIor: Property<number> = new Property<number>("coat_ior", 1.5, "vCoatIor", 1, 0);
 
+    /**
+     * Specifies whether the coat roughness is taken from the
+     * same texture as the coat_weight.
+     */
+    public useCoatRoughnessFromWeightTexture: boolean = false;
+
     /**
      * Defines the normal of the material's geometry.
      * See OpenPBR's specs for geometry_normal

packages/dev/core/src/Materials/imageProcessing.ts

@@ -10,13 +10,13 @@ import type { ColorCurves } from "../Materials/colorCurves";
 // export type { Observer } from "../Misc/observable";
 // export type { ColorCurves } from "./colorCurves";
 
-type Constructor<T = {}> = new (...args: any[]) => T;
+type ImageProcessingMixinConstructor<T = {}> = new (...args: any[]) => T;
 
 /**
  * Mixin to add Image processing defines to your material defines
  * @internal
  */
-export function ImageProcessingMixin<Tbase extends Constructor>(base: Tbase) {
+export function ImageProcessingMixin<Tbase extends ImageProcessingMixinConstructor>(base: Tbase) {
     return class extends base {
         /**
          * Constructor for the ImageProcessingMixin.

packages/dev/core/src/Materials/imageProcessingConfiguration.defines.ts

@@ -22,13 +22,13 @@ export interface IImageProcessingConfigurationDefines {
     SKIPFINALCOLORCLAMP: boolean;
 }
 
-type Constructor<T = {}> = new (...args: any[]) => T;
+type ImageProcessingDefinesMixinConstructor<T = {}> = new (...args: any[]) => T;
 
 /**
  * Mixin to add Image processing defines to your material defines
  * @internal
  */
-export function ImageProcessingDefinesMixin<Tbase extends Constructor>(base: Tbase) {
+export function ImageProcessingDefinesMixin<Tbase extends ImageProcessingDefinesMixinConstructor>(base: Tbase) {
     return class extends base implements IImageProcessingConfigurationDefines {
         // Implement all members of IImageProcessingConfigurationDefines here
         public IMAGEPROCESSING = false;

packages/dev/core/src/Materials/uv.defines.ts

@@ -1,10 +1,10 @@
-type Constructor<T = {}> = new (...args: any[]) => T;
+type UVDefinesMixinConstructor<T = {}> = new (...args: any[]) => T;
 
 /**
  * Mixin to add UV defines to your material defines
  * @internal
  */
-export function UVDefinesMixin<Tbase extends Constructor>(base: Tbase) {
+export function UVDefinesMixin<Tbase extends UVDefinesMixinConstructor>(base: Tbase) {
     return class extends base {
         public MAINUV1 = false;
         public MAINUV2 = false;