/* Copyright (C) 2018-2021 Carl Hetherington This file is part of DCP-o-matic. DCP-o-matic is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation; either version 2 of the License, or (at your option) any later version. DCP-o-matic is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details. You should have received a copy of the GNU General Public License along with DCP-o-matic. If not, see . */ #include "verify_dcp_dialog.h" #include "wx_util.h" #include "lib/verify_dcp_job.h" #include #include #include LIBDCP_DISABLE_WARNINGS #include #include LIBDCP_ENABLE_WARNINGS #include using std::list; using std::map; using std::shared_ptr; using std::string; using std::vector; VerifyDCPDialog::VerifyDCPDialog (wxWindow* parent, shared_ptr job) : wxDialog (parent, wxID_ANY, _("DCP verification"), wxDefaultPosition, {600, 400}) { auto sizer = new wxBoxSizer (wxVERTICAL); auto notebook = new wxNotebook (this, wxID_ANY); sizer->Add (notebook, 1, wxEXPAND | wxALL, DCPOMATIC_DIALOG_BORDER); map pages; pages[dcp::VerificationNote::Type::ERROR] = new wxRichTextCtrl (notebook, wxID_ANY, wxEmptyString, wxDefaultPosition, {400, 300}, wxRE_READONLY); notebook->AddPage (pages[dcp::VerificationNote::Type::ERROR], _("Errors")); pages[dcp::VerificationNote::Type::BV21_ERROR] = new wxRichTextCtrl (notebook, wxID_ANY, wxEmptyString, wxDefaultPosition, {400, 300}, wxRE_READONLY); notebook->AddPage (pages[dcp::VerificationNote::Type::BV21_ERROR], _("SMPTE Bv2.1 errors")); pages[dcp::VerificationNote::Type::WARNING] = new wxRichTextCtrl (notebook, wxID_ANY, wxEmptyString, wxDefaultPosition, {400, 300}, wxRE_READONLY); notebook->AddPage (pages[dcp::VerificationNote::Type::WARNING], _("Warnings")); auto summary = new wxStaticText (this, wxID_ANY, wxT("")); sizer->Add (summary, 0, wxALL, DCPOMATIC_DIALOG_BORDER); auto buttons = CreateStdDialogButtonSizer (0); sizer->Add (CreateSeparatedSizer(buttons), wxSizerFlags().Expand().DoubleBorder()); buttons->SetAffirmativeButton (new wxButton (this, wxID_OK)); buttons->Realize (); SetSizer (sizer); sizer->Layout (); sizer->SetSizeHints (this); for (auto const& i: pages) { i.second->GetCaret()->Hide(); } if (job->finished_ok() && job->notes().empty()) { summary->SetLabel (_("DCP validates OK.")); return; } map counts; counts[dcp::VerificationNote::Type::WARNING] = 0; counts[dcp::VerificationNote::Type::BV21_ERROR] = 0; counts[dcp::VerificationNote::Type::ERROR] = 0; auto add_bullet = [&pages](dcp::VerificationNote::Type type, wxString message) { pages[type]->BeginStandardBullet(N_("standard/diamond"), 1, 50); pages[type]->WriteText (message); pages[type]->Newline (); pages[type]->EndStandardBullet (); }; int constexpr limit_per_type = 20; auto substitute = [](wxString message, dcp::VerificationNote const& note) { if (note.reference_hash()) { message.Replace("%reference_hash", std_to_wx(note.reference_hash().get())); } if (note.calculated_hash()) { message.Replace("%calculated_hash", std_to_wx(note.calculated_hash().get())); } if (note.frame()) { message.Replace("%frame", std_to_wx(dcp::raw_convert(note.frame().get()))); message.Replace( "%timecode", std_to_wx( dcp::Time(note.frame().get(), note.frame_rate().get(), note.frame_rate().get()).as_string(dcp::Standard::SMPTE) )); } if (note.note()) { message.Replace("%n", std_to_wx(note.note().get())); } if (note.file()) { message.Replace("%f", std_to_wx(note.file()->filename().string())); } if (note.line()) { message.Replace("%l", std_to_wx(dcp::raw_convert(note.line().get()))); } if (note.component()) { message.Replace("%component", std_to_wx(dcp::raw_convert(note.component().get()))); } if (note.size()) { message.Replace("%size", std_to_wx(dcp::raw_convert(note.size().get()))); } if (note.id()) { message.Replace("%id", std_to_wx(note.id().get())); } if (note.other_id()) { message.Replace("%other_id", std_to_wx(note.other_id().get())); } return message; }; auto add = [&counts, &add_bullet, &substitute](vector const& notes, wxString message, wxString more_message = {}) { int N = 0; for (auto const& note: notes) { add_bullet(note.type(), substitute(message, note)); counts[note.type()]++; if (++N >= limit_per_type) { break; } } if (notes.size() == limit_per_type && !more_message.IsEmpty()) { add_bullet(notes[0].type(), more_message); } }; if (job->finished_in_error() && job->error_summary() != "") { /* We have an error that did not come from dcp::verify */ add_bullet (dcp::VerificationNote::Type::ERROR, std_to_wx(job->error_summary())); ++counts[dcp::VerificationNote::Type::ERROR]; } /* Gather notes by code, discarding more than limit_per_type so we don't get overwhelmed if * every frame of a long DCP has a note. */ std::map> notes_by_code; for (auto const& note: job->notes()) { auto type_iter = notes_by_code.find(note.code()); if (type_iter != notes_by_code.end()) { if (type_iter->second.size() < limit_per_type) { type_iter->second.push_back(note); } } else { notes_by_code[note.code()] = { note }; } } for (auto const& i: notes_by_code) { switch (i.first) { case dcp::VerificationNote::Code::FAILED_READ: add(i.second, _("Could not read DCP (%n)")); break; case dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES: add(i.second, _("The hash (%reference_hash) of the CPL %n in the PKL does not agree with the CPL file (%calculated_hash). This probably means that the CPL file is corrupt.")); break; case dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE: add(i.second, _("The picture in a reel has a frame rate of %n, which is not valid.")); break; case dcp::VerificationNote::Code::INCORRECT_PICTURE_HASH: add(i.second, _("The hash (%calculated_hash) of the picture asset %f does not agree with the PKL file (%reference_hash). This probably means that the asset file is corrupt.")); break; case dcp::VerificationNote::Code::MISMATCHED_PICTURE_HASHES: add(i.second, _("The PKL and CPL hashes disagree for picture asset %f.")); break; case dcp::VerificationNote::Code::INCORRECT_SOUND_HASH: add(i.second, _("The hash (%calculated_hash) of the sound asset %f does not agree with the PKL file (%reference_hash). This probably means that the asset file is corrupt.")); break; case dcp::VerificationNote::Code::MISMATCHED_SOUND_HASHES: add(i.second, _("The PKL and CPL hashes disagree for sound asset %f.")); break; case dcp::VerificationNote::Code::EMPTY_ASSET_PATH: add(i.second, _("An asset has an empty path in the ASSETMAP.")); break; case dcp::VerificationNote::Code::MISSING_ASSET: add(i.second, _("The asset %f is missing.")); break; case dcp::VerificationNote::Code::MISMATCHED_STANDARD: add(i.second, _("Parts of the DCP are written according to the Interop standard and parts according to SMPTE.")); break; case dcp::VerificationNote::Code::INVALID_XML: for (auto const& note: i.second) { if (note.line()) { add({ note }, _("The XML in %f is malformed on line %l (%n).")); } else { add({ note }, _("The XML in %f is malformed (%n).")); } } break; case dcp::VerificationNote::Code::MISSING_ASSETMAP: add(i.second, _("No ASSETMAP or ASSETMAP.xml file was found.")); break; case dcp::VerificationNote::Code::INVALID_INTRINSIC_DURATION: add(i.second, _("The asset %n has an intrinsic duration of less than 1 second, which is invalid.")); break; case dcp::VerificationNote::Code::INVALID_DURATION: add(i.second, _("The asset %n has a duration of less than 1 second, which is invalid.")); break; case dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_SIZE_IN_BYTES: add( i.second, _("Frame %frame (timecode %timecode) in asset %f has an instantaneous bit rate that is over the limit of 250Mbit/s."), _("More frames (not listed) have an instantaneous bit rate that is over the limit of 250Mbit/s.") ); break; case dcp::VerificationNote::Code::NEARLY_INVALID_PICTURE_FRAME_SIZE_IN_BYTES: add( i.second, _("Frame %frame (timecode %timecode) in asset %f has an instantaneous bit rate that is close to the limit of 250Mbit/s."), _("More frames (not listed) have an instantaneous bit rate that is close to the limit of 250Mbit/s.") ); break; case dcp::VerificationNote::Code::EXTERNAL_ASSET: add(i.second, _("This DCP refers to at the asset %n in another DCP (and perhaps others), so it is a \"version file\" (VF)")); break; case dcp::VerificationNote::Code::THREED_ASSET_MARKED_AS_TWOD: add(i.second, _("The asset %f is 3D but its MXF is marked as 2D.")); break; case dcp::VerificationNote::Code::INVALID_STANDARD: add(i.second, _("This DCP uses the Interop standard, but it should be made with SMPTE.")); break; case dcp::VerificationNote::Code::INVALID_LANGUAGE: add(i.second, _("The invalid language tag %n is used.")); break; case dcp::VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS: add(i.second, _("The video asset %f uses the invalid image size %n.")); break; case dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_2K: add(i.second, _("The video asset %f uses the invalid frame rate %n.")); break; case dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_4K: add(i.second, _("The video asset %f uses the frame rate %n which is invalid for 4K video.")); break; case dcp::VerificationNote::Code::INVALID_PICTURE_ASSET_RESOLUTION_FOR_3D: add(i.second, _("The video asset %f uses the frame rate %n which is invalid for 3D video.")); break; case dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_XML_SIZE_IN_BYTES: add(i.second, _("The XML in the closed caption asset %f takes up %n bytes which is over the 256KB limit.")); break; case dcp::VerificationNote::Code::INVALID_TIMED_TEXT_SIZE_IN_BYTES: add(i.second, _("The timed text asset %f takes up %n bytes which is over the 115MB limit.")); break; case dcp::VerificationNote::Code::INVALID_TIMED_TEXT_FONT_SIZE_IN_BYTES: add(i.second, _("The fonts in the timed text asset %f take up %n bytes which is over the 10MB limit.")); break; case dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE: add(i.second, _("The subtitle asset %f contains no tag.")); break; case dcp::VerificationNote::Code::MISMATCHED_SUBTITLE_LANGUAGES: add(i.second, _("Not all subtitle assets specify the same tag.")); break; case dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME: add(i.second, _("The subtitle asset %f contains no tag.")); break; case dcp::VerificationNote::Code::INVALID_SUBTITLE_START_TIME: add(i.second, _("The subtitle asset %f has a which is not zero.")); break; case dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME: add(i.second, _("The first subtitle or closed caption happens before 4s into the first reel.")); break; case dcp::VerificationNote::Code::INVALID_SUBTITLE_DURATION: add(i.second, _("At least one subtitle lasts less than 15 frames.")); break; case dcp::VerificationNote::Code::INVALID_SUBTITLE_SPACING: add(i.second, _("At least one pair of subtitles is separated by less than 2 frames.")); break; case dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_COUNT: add(i.second, _("There are more than 3 subtitle lines in at least one place.")); break; case dcp::VerificationNote::Code::NEARLY_INVALID_SUBTITLE_LINE_LENGTH: add(i.second, _("There are more than 52 characters in at least one subtitle line.")); break; case dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_LENGTH: add(i.second, _("There are more than 79 characters in at least one subtitle line.")); break; case dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_COUNT: add(i.second, _("There are more than 3 closed caption lines in at least one place.")); break; case dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_LENGTH: add(i.second, _("There are more than 32 characters in at least one closed caption line.")); break; case dcp::VerificationNote::Code::INVALID_SOUND_FRAME_RATE: add(i.second, _("The sound asset %f has an invalid frame rate of %n.")); break; case dcp::VerificationNote::Code::MISSING_CPL_ANNOTATION_TEXT: add(i.second, _("The CPL %n has no tag.")); break; case dcp::VerificationNote::Code::MISMATCHED_CPL_ANNOTATION_TEXT: add(i.second, _("The CPL %n has an which is not the same as its .")); break; case dcp::VerificationNote::Code::MISMATCHED_ASSET_DURATION: add(i.second, _("At least one asset in a reel does not have the same duration as the others.")); break; case dcp::VerificationNote::Code::MISSING_MAIN_SUBTITLE_FROM_SOME_REELS: add(i.second, _("The DCP has subtitles but at least one reel has no subtitle asset.")); break; case dcp::VerificationNote::Code::MISMATCHED_CLOSED_CAPTION_ASSET_COUNTS: add(i.second, _("The DCP has closed captions but not every reel has the same number of closed caption assets.")); break; case dcp::VerificationNote::Code::MISSING_SUBTITLE_ENTRY_POINT: add(i.second, _("The subtitle asset %n has no tag.")); break; case dcp::VerificationNote::Code::INCORRECT_SUBTITLE_ENTRY_POINT: add(i.second, _("Subtitle asset %n has a non-zero .")); break; case dcp::VerificationNote::Code::MISSING_CLOSED_CAPTION_ENTRY_POINT: add(i.second, _("The closed caption asset %n has no tag.")); break; case dcp::VerificationNote::Code::INCORRECT_CLOSED_CAPTION_ENTRY_POINT: add(i.second, _("Closed caption asset %n has a non-zero .")); break; case dcp::VerificationNote::Code::MISSING_HASH: add(i.second, _("The asset %n has no in the CPL.")); break; case dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE: add(i.second, _("The DCP is a feature but has no FFEC (first frame of end credits) marker.")); break; case dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE: add(i.second, _("The DCP is a feature but has no FFMC (first frame of moving credits) marker.")); break; case dcp::VerificationNote::Code::MISSING_FFOC: add(i.second, _("The DCP has no FFOC (first frame of content) marker.")); break; case dcp::VerificationNote::Code::MISSING_LFOC: add(i.second, _("The DCP has no LFOC (last frame of content) marker.")); break; case dcp::VerificationNote::Code::INCORRECT_FFOC: add(i.second, _("The DCP has a FFOC of %n instead of 1.")); break; case dcp::VerificationNote::Code::INCORRECT_LFOC: add(i.second, _("The DCP has a LFOC of %n instead of the reel duration minus one.")); break; case dcp::VerificationNote::Code::MISSING_CPL_METADATA: add(i.second, _("The CPL %n has no CPL metadata tag.")); break; case dcp::VerificationNote::Code::MISSING_CPL_METADATA_VERSION_NUMBER: add(i.second, _("The CPL %n has no CPL metadata version number tag.")); break; case dcp::VerificationNote::Code::MISSING_EXTENSION_METADATA: add(i.second, _("The CPL %n has no CPL extension metadata tag.")); break; case dcp::VerificationNote::Code::INVALID_EXTENSION_METADATA: add(i.second, _("The CPL %f has an invalid CPL extension metadata tag (%n)")); break; case dcp::VerificationNote::Code::UNSIGNED_CPL_WITH_ENCRYPTED_CONTENT: add(i.second, _("The CPL %n has encrypted content but is not signed.")); break; case dcp::VerificationNote::Code::UNSIGNED_PKL_WITH_ENCRYPTED_CONTENT: add(i.second, _("The PKL %n has encrypted content but is not signed.")); break; case dcp::VerificationNote::Code::MISMATCHED_PKL_ANNOTATION_TEXT_WITH_CPL: add(i.second, _("The PKL %n has an which does not match its CPL's .")); break; case dcp::VerificationNote::Code::PARTIALLY_ENCRYPTED: add(i.second, _("The DCP has encrypted content, but not all its assets are encrypted.")); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_CODESTREAM: add( i.second, _("A picture frame has an invalid JPEG2000 codestream (%n)."), _("More picture frames (not listed) have invalid JPEG2000 codestreams.") ); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_GUARD_BITS_FOR_2K: add( i.second, _("A 2K JPEG2000 frame has %n guard bits instead of 1."), _("More 2K JPEG2000 frames (not listed) have an invalid number of guard bits.") ); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_GUARD_BITS_FOR_4K: add( i.second, _("A 4K JPEG2000 frame has %n guard bits instead of 2."), _("More 4K JPEG2000 frames (not listed) have an invalid number of guard bits.") ); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_SIZE: add( i.second, _("A JPEG2000 tile size does not match the image size."), _("More JPEG2000 tile sizes (not listed) do not match the image size.") ); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_CODE_BLOCK_WIDTH: add( i.second, _("A JPEG2000 frame has a code-block width of %n instead of 32."), _("More JPEG2000 frames (not listed) have an invalid code-block width.") ); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_CODE_BLOCK_HEIGHT: add( i.second, _("A JPEG2000 frame has a code-block height of %n instead of 32."), _("More JPEG2000 frames (not listed) have an invalid code-block height.") ); break; case dcp::VerificationNote::Code::INCORRECT_JPEG2000_POC_MARKER_COUNT_FOR_2K: add( i.second, _("A 2K JPEG2000 frame has %n POC marker(s) instead of 0."), _("More 2K JPEG2000 frames (not listed) have too many POC markers.") ); break; case dcp::VerificationNote::Code::INCORRECT_JPEG2000_POC_MARKER_COUNT_FOR_4K: add( i.second, _("A 4K JPEG2000 frame has %n POC marker(s) instead of 1."), _("More 4K JPEG2000 frames (not listed) have too many POC markers.") ); break; case dcp::VerificationNote::Code::INCORRECT_JPEG2000_POC_MARKER: add( i.second, _("A JPEG2000 frame contains an invalid POC marker (%n)."), _("More JPEG2000 frames (not listed) contain invalid POC markers.") ); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_POC_MARKER_LOCATION: add( i.second, _("A JPEG2000 frame contains a POC marker in an invalid location."), _("More JPEG2000 frames (not listed) contain POC markers in invalid locations.") ); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_PARTS_FOR_2K: add( i.second, _("A 2K JPEG2000 frame contains %n tile parts instead of 3."), _("More 2K JPEG2000 frames (not listed) contain the wrong number of tile parts.") ); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_PARTS_FOR_4K: add( i.second, _("A 4K JPEG2000 frame contains %n tile parts instead of 6."), _("More JPEG2000 frames (not listed) contain the wrong number of tile parts.") ); break; case dcp::VerificationNote::Code::MISSING_JPEG200_TLM_MARKER: add( i.second, _("A JPEG2000 frame has no TLM marker."), _("More JPEG2000 frames (not listed) have no TLM marker.") ); break; case dcp::VerificationNote::Code::SUBTITLE_OVERLAPS_REEL_BOUNDARY: add(i.second, _("A subtitle lasts longer than the reel it is in.")); break; case dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_RESOURCE_ID: add(i.second, _("The Resource ID in a timed text MXF did not match the ID of the contained XML.")); break; case dcp::VerificationNote::Code::INCORRECT_TIMED_TEXT_ASSET_ID: add(i.second, _("The Asset ID in a timed text MXF is the same as the Resource ID or that of the contained XML.")); break; case dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_DURATION: { for (auto const& note: i.second) { vector parts; boost::split(parts, note.note().get(), boost::is_any_of(" ")); add( { note }, wxString::Format( _("The reel duration (%s) of some timed text is not the same as the ContainerDuration (%s) of its MXF."), std_to_wx(parts[0]), std_to_wx(parts[1]) ) ); } break; } case dcp::VerificationNote::Code::MISSED_CHECK_OF_ENCRYPTED: add(i.second, _("Part of the DCP could not be checked because no KDM was available.")); break; case dcp::VerificationNote::Code::EMPTY_TEXT: add(i.second, _("At least one node in a subtitle or closed caption is empty.")); break; case dcp::VerificationNote::Code::MISMATCHED_CLOSED_CAPTION_VALIGN: add(i.second, _("Some closed or nodes have different vertical alignments within a .")); break; case dcp::VerificationNote::Code::INCORRECT_CLOSED_CAPTION_ORDERING: add(i.second, _("Some closed captions are not listed in the order of their vertical position.")); break; case dcp::VerificationNote::Code::UNEXPECTED_ENTRY_POINT: add(i.second, _("There is a tag inside a .")); break; case dcp::VerificationNote::Code::UNEXPECTED_DURATION: add(i.second, _("There is a tag inside a .")); break; case dcp::VerificationNote::Code::INVALID_CONTENT_KIND: add(i.second, _("An invalid %n has been used.")); break; case dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA: add(i.second, _("The is either not a multiple of 2, or is bigger than an asset.")); break; case dcp::VerificationNote::Code::DUPLICATE_ASSET_ID_IN_PKL: add(i.second, _("The PKL %n has more than one asset with the same ID.")); break; case dcp::VerificationNote::Code::DUPLICATE_ASSET_ID_IN_ASSETMAP: add(i.second, _("The ASSETMAP %n has more than one asset with the same ID.")); break; case dcp::VerificationNote::Code::MISSING_SUBTITLE: add(i.second, _("The subtitle asset %n contains no subtitles.")); break; case dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE: add(i.second, _(" has an invalid value %n")); break; case dcp::VerificationNote::Code::MISMATCHED_SOUND_CHANNEL_COUNTS: add(i.second, _("Sound assets do not all have the same channel count.")); break; case dcp::VerificationNote::Code::INVALID_MAIN_SOUND_CONFIGURATION: add(i.second, _(" describes incorrect number of channels (%n)")); break; case dcp::VerificationNote::Code::MISSING_FONT: add(i.second, _("The font file for font ID \"%n\" was not found, or was not referred to in the ASSETMAP.")); break; case dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_PART_SIZE: add( i.second, _("Frame %frame has an image component that is too large (component %component is %size bytes in size)."), _("More frames (not listed) have image components that are too large.") ); break; case dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT: add(i.second, _("The XML in the subtitle asset %n has more than one namespace declaration.")); break; case dcp::VerificationNote::Code::MISSING_LOAD_FONT_FOR_FONT: add(i.second, _("A subtitle or closed caption refers to a font with ID %id that does not have a corresponding node.")); break; case dcp::VerificationNote::Code::MISSING_LOAD_FONT: add(i.second, _("The SMPTE subtitle asset %id has nodes but no node")); break; case dcp::VerificationNote::Code::MISMATCHED_ASSET_MAP_ID: add(i.second, _("The asset with ID %id in the asset map actually has an id of %other_id")); break; case dcp::VerificationNote::Code::EMPTY_CONTENT_VERSION_LABEL_TEXT: add(i.second, _("The in a in CPL %id is empty")); break; case dcp::VerificationNote::Code::INVALID_CPL_NAMESPACE: add(i.second, _("The CPL %f has an invalid namespace %n")); break; case dcp::VerificationNote::Code::MISSING_CPL_CONTENT_VERSION: add(i.second, _("The CPL %n has no tag")); break; case dcp::VerificationNote::Code::INVALID_PKL_NAMESPACE: add(i.second, _("The PKL %f has an invalid namespace %n")); break; } } wxString summary_text; if (counts[dcp::VerificationNote::Type::ERROR] == 1) { /// TRANSLATORS: this will be used at the start of a string like "1 error, 2 Bv2.1 errors and 3 warnings." summary_text = _("1 error, "); } else { /// TRANSLATORS: this will be used at the start of a string like "1 error, 2 Bv2.1 errors and 3 warnings." summary_text = wxString::Format("%d errors, ", counts[dcp::VerificationNote::Type::ERROR]); } if (counts[dcp::VerificationNote::Type::BV21_ERROR] == 1) { /// TRANSLATORS: this will be used in the middle of a string like "1 error, 2 Bv2.1 errors and 3 warnings." summary_text += _("1 Bv2.1 error, "); } else { /// TRANSLATORS: this will be used in the middle of a string like "1 error, 2 Bv2.1 errors and 3 warnings." summary_text += wxString::Format("%d Bv2.1 errors, ", counts[dcp::VerificationNote::Type::BV21_ERROR]); } if (counts[dcp::VerificationNote::Type::WARNING] == 1) { /// TRANSLATORS: this will be used at the end of a string like "1 error, 2 Bv2.1 errors and 3 warnings." summary_text += _("and 1 warning."); } else { /// TRANSLATORS: this will be used at the end of a string like "1 error, 2 Bv2.1 errors and 3 warnings." summary_text += wxString::Format("and %d warnings.", counts[dcp::VerificationNote::Type::WARNING]); } summary->SetLabel(summary_text); if (counts[dcp::VerificationNote::Type::ERROR] == 0) { add_bullet (dcp::VerificationNote::Type::ERROR, _("No errors found.")); } if (counts[dcp::VerificationNote::Type::BV21_ERROR] == 0) { add_bullet (dcp::VerificationNote::Type::BV21_ERROR, _("No SMPTE Bv2.1 errors found.")); } if (counts[dcp::VerificationNote::Type::WARNING] == 0) { add_bullet (dcp::VerificationNote::Type::WARNING, _("No warnings found.")); } }