🐛 Fix models converted to Blueprints becoming unsavable

If the default variant couldn't be found, the codec compile function would fail. I've changed the `Variant.getDefault` function to create a default variant if one does not already exist.

Author: SnaveSutit

src/components/keyframePanels/variantKeyframePanel.svelte

@@ -11,7 +11,7 @@
 	let executeCondition = keyframe?.execute_condition ?? ''
 
 	$: {
-		keyframe.variant = Variant.all.find(v => v.uuid === variantUuid) ?? Variant.getDefault()
+		keyframe.variant = Variant.getByUUID(variantUuid) ?? Variant.getDefault()
 		keyframe.execute_condition = executeCondition
 		Animator.preview()
 	}

src/formats/blueprint/codec.ts

@@ -309,8 +309,8 @@ export const BLUEPRINT_CODEC = registerCodec(
 			}
 
 			model.variants = {
-				default: Variant.all.find(v => v.isDefault)!.toJSON(),
-				list: Variant.all.filter(v => !v.isDefault).map(v => v.toJSON()),
+				default: Variant.getDefault().toJSON(),
+				list: Variant.allExcludingDefault().map(v => v.toJSON()),
 			}
 
 			const animationOptions = { bone_names: true, absolute_paths: options.absolute_paths }

src/formats/blueprint/index.ts

@@ -281,13 +281,6 @@ export const BLUEPRINT_FORMAT = registerModelFormat(
 
 			project.pluginMode = new Valuable(project.animated_java.enable_plugin_mode)
 
-			// project.variants ??= []
-			// if (Variant.all.length === 0) {
-			// 	console.warn('No variants found, creating default variant')
-			// 	new Variant('Default', true)
-			// }
-			// Variant.selectDefault()
-
 			initializeRenderBoxPreview()
 
 			requestAnimationFrame(() => {

src/mods/customKeyframes.ts

@@ -44,7 +44,7 @@ registerConditionalPropertyOverrideMod({
 		const uuid = this.data_points.at(0)?.[EFFECT_ANIMATOR_CHANNELS.VARIANT] as
 			| string
 			| undefined
-		if (uuid) return Variant.all.find(v => v.uuid === uuid)
+		if (uuid) return Variant.getByUUID(uuid)
 		console.error('Keyframe variant', uuid, 'not found!')
 	},
 

src/variants.ts

@@ -93,11 +93,13 @@ export class Variant {
 		this.textureMap = new TextureMap()
 		this.id = Variant.all.length
 		if (this.isDefault) {
+			if (Variant.hasDefault()) {
+				throw new Error('There can only be one default variant!')
+			}
 			this.displayName = 'Default'
 			this.name = 'default'
 		}
 		Variant.all.push(this)
-		// this.select()
 		EVENTS.CREATE_VARIANT.publish(this)
 	}
 
@@ -225,12 +227,23 @@ export class Variant {
 	}
 
 	static selectDefault() {
-		const variant = Variant.all.find(v => v.isDefault)
-		if (variant) variant.select()
+		Variant.getDefault().select()
+	}
+
+	static getByUUID(uuid: string): Variant | undefined {
+		return Variant.all.find(v => v.uuid === uuid)
+	}
+
+	static allExcludingDefault(): Variant[] {
+		return Variant.all.filter(v => !v.isDefault)
+	}
+
+	static hasDefault(): boolean {
+		return Variant.all.some(v => v.isDefault)
 	}
 
 	static getDefault(): Variant {
-		return Variant.all.find(v => v.isDefault) ?? Variant.all[0]
+		return Variant.all.find(v => v.isDefault) ?? new Variant('Default', true)
 	}
 }