Refactor sequence validation: Make sequence validation conditional on project scheduling being enabled
This commit is contained in:
@@ -6418,32 +6418,35 @@ export function validateSceneDocument(
|
|||||||
validateProjectAsset(asset, path, diagnostics);
|
validateProjectAsset(asset, path, diagnostics);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (validateProjectScheduling) {
|
for (const [sequenceKey, sequence] of Object.entries(
|
||||||
for (const [sequenceKey, sequence] of Object.entries(
|
document.sequences.sequences
|
||||||
document.sequences.sequences
|
)) {
|
||||||
)) {
|
const path = `sequences.sequences.${sequenceKey}`;
|
||||||
const path = `sequences.sequences.${sequenceKey}`;
|
|
||||||
|
|
||||||
if (sequence.id !== sequenceKey) {
|
if (sequence.id !== sequenceKey) {
|
||||||
diagnostics.push(
|
diagnostics.push(
|
||||||
createDiagnostic(
|
createDiagnostic(
|
||||||
"error",
|
"error",
|
||||||
"sequence-id-mismatch",
|
"sequence-id-mismatch",
|
||||||
"Sequence ids must match their registry key.",
|
"Sequence ids must match their registry key.",
|
||||||
`${path}.id`
|
`${path}.id`
|
||||||
)
|
)
|
||||||
);
|
|
||||||
}
|
|
||||||
|
|
||||||
registerAuthoredId(sequence.id, path, seenIds, diagnostics);
|
|
||||||
validateProjectSequence(
|
|
||||||
sequence,
|
|
||||||
path,
|
|
||||||
{ scenes: {}, currentSceneEntities: document.entities },
|
|
||||||
projectSchedulerValidationContext,
|
|
||||||
diagnostics
|
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
registerAuthoredId(sequence.id, path, seenIds, diagnostics);
|
||||||
|
|
||||||
|
if (!validateProjectScheduling) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
validateProjectSequence(
|
||||||
|
sequence,
|
||||||
|
path,
|
||||||
|
{ scenes: {}, currentSceneEntities: document.entities },
|
||||||
|
projectSchedulerValidationContext,
|
||||||
|
diagnostics
|
||||||
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
for (const [brushKey, brush] of Object.entries(document.brushes)) {
|
for (const [brushKey, brush] of Object.entries(document.brushes)) {
|
||||||
|
|||||||
Reference in New Issue
Block a user