New generated_vod text source not present in Track model

[martinbean]

I’ve been playing with auto-generated captions for on-demand video. I’ve managed to successfully generate a subtitles text track for a video asset, but then when using the Mux PHP SDK to list assets, I get the following error:

Invalid value 'generated_vod' for 'text_source', must be one of 'uploaded', 'embedded', 'generated_live', 'generated_live_final'

The error seems to stem from here:

mux-php/MuxPhp/Models/Track.php

Lines 790 to 799 in c69f2d4

	$allowedValues = $this->getTextSourceAllowableValues();
	if (!is_null($text_source) && !in_array($text_source, $allowedValues, true)) {
	throw new \InvalidArgumentException(
	sprintf(
	"Invalid value '%s' for 'text_source', must be one of '%s'",
	$text_source,
	implode("', '", $allowedValues)
	)
	);
	}

The issue seems to be, that there’s no constant representing the value generated_vod in the getTextSourceAllowableValues method: https://github.com/muxinc/mux-php/blob/c69f2d4c84f2be3d50f3f44895468535e0a7266d/MuxPhp/Models/Track.php#L352C21-L360

So seems this just needs updating with a new self::TEXT_SOURCE_GENERATED_VOD constant.

[jsanford8]

hey @martinbean - thanks for this! We're actively working on a release of our SDKs for these new features, and hope to have one out within a few days.

[dylanjha]

@martinbean is this resolved for you as of 3.12?

[martinbean]

@martinbean is this resolved for you as of 3.12?

Unfortunately not. A constant for generated_vod is still missing, even in v3.12.0:

https://github.com/muxinc/mux-php/blob/3.12.0/MuxPhp/Models/Track.php#L352-L360

[philcluff]

Hey @martinbean, sorry for the oversight here, I'm working on a release that'll address this over the next day or so.

Thanks

[philcluff]

Hey @martinbean, this is fixed in 3.12.1 - please can you upgrade and let us know if you continue to see issues.

Thanks!

[martinbean]

@philcluff Yup, that’s working now 👍

I would have submitted the code change myself, but thought I’d read the code was automatically generated from an API spec or something. Looking at the referenced PR, is that not the case?

[philcluff]

@martinbean Yep, you're spot on, most our SDKs are generated form our OpenAPI specification, so it's best we update them.

Thanks!