From 4dca630057509164494b65c2deeb748a51928c73 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Fri, 11 Aug 2023 20:51:21 +0200 Subject: Add check for empty in We have a report of Deluxe failing a DCP because of this. --- src/verify.cc | 11 +++++++++++ src/verify.h | 5 +++++ 2 files changed, 16 insertions(+) (limited to 'src') diff --git a/src/verify.cc b/src/verify.cc index 8f318af9..42ee1921 100644 --- a/src/verify.cc +++ b/src/verify.cc @@ -1548,6 +1548,15 @@ verify_cpl( } } + for (auto version: cpl->content_versions()) { + if (version.label_text.empty()) { + notes.push_back( + dcp::VerificationNote(VerificationNote::Type::WARNING, VerificationNote::Code::EMPTY_CONTENT_VERSION_LABEL_TEXT, cpl->file().get()).set_id(cpl->id()) + ); + break; + } + } + if (dcp->standard() == Standard::SMPTE) { if (!cpl->annotation_text()) { notes.push_back({VerificationNote::Type::BV21_ERROR, VerificationNote::Code::MISSING_CPL_ANNOTATION_TEXT, cpl->id(), cpl->file().get()}); @@ -2091,6 +2100,8 @@ dcp::note_to_string (VerificationNote note) return String::compose("The SMPTE subtitle asset %1 has nodes but no node", note.id().get()); case VerificationNote::Code::MISMATCHED_ASSET_MAP_ID: return String::compose("The asset with ID %1 in the asset map actually has an id of %2", note.id().get(), note.other_id().get()); + case VerificationNote::Code::EMPTY_CONTENT_VERSION_LABEL_TEXT: + return String::compose("The in a in CPL %1 is empty", note.id().get()); } return ""; diff --git a/src/verify.h b/src/verify.h index 37a1fc11..fdf2d7ff 100644 --- a/src/verify.h +++ b/src/verify.h @@ -462,6 +462,11 @@ public: * other_id contains the ID from the file. */ MISMATCHED_ASSET_MAP_ID, + /** The inside a is empty + * note contains the CPL ID + * file contains the CPL filename + */ + EMPTY_CONTENT_VERSION_LABEL_TEXT, }; VerificationNote (Type type, Code code) -- cgit v1.2.3