Stop assuming that the presence of FullContentTitleText means that there was CPL...

[libdcp.git] / src / verify.cc

diff --git a/src/verify.cc b/src/verify.cc
index bef05a16fe2e494a8a4490ffd91093bf6f2301be..e698920c11da3f8f1f49c3e43df16190c1921ba2 100644 (file)
--- a/src/verify.cc
+++ b/src/verify.cc
@@ -1534,10 +1534,7 @@ dcp::verify (
                                        notes.push_back ({VerificationNote::Type::BV21_ERROR, VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_LENGTH});
                                }
 
-                               if (!cpl->full_content_title_text()) {
-                                       /* Since FullContentTitleText is assumed always to exist if there's a CompositionMetadataAsset we
-                                        * can use it as a proxy for CompositionMetadataAsset's existence.
-                                        */
+                               if (!cpl->read_composition_metadata()) {
                                        notes.push_back ({VerificationNote::Type::BV21_ERROR, VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get()});
                                } else if (!cpl->version_number()) {
                                        notes.push_back ({VerificationNote::Type::BV21_ERROR, VerificationNote::Code::MISSING_CPL_METADATA_VERSION_NUMBER, cpl->id(), cpl->file().get()});