src/wx/verify_dcp_dialog.cc

   1 /*
   2     Copyright (C) 2018-2021 Carl Hetherington <cth@carlh.net>
   3
   4     This file is part of DCP-o-matic.
   5
   6     DCP-o-matic is free software; you can redistribute it and/or modify
   7     it under the terms of the GNU General Public License as published by
   8     the Free Software Foundation; either version 2 of the License, or
   9     (at your option) any later version.
  10
  11     DCP-o-matic is distributed in the hope that it will be useful,
  12     but WITHOUT ANY WARRANTY; without even the implied warranty of
  13     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  14     GNU General Public License for more details.
  15
  16     You should have received a copy of the GNU General Public License
  17     along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
  18
  19 */
  20
  21
  22 #include "verify_dcp_dialog.h"
  23 #include "wx_util.h"
  24 #include "lib/verify_dcp_job.h"
  25 #include <dcp/raw_convert.h>
  26 #include <dcp/verify.h>
  27 #include <dcp/warnings.h>
  28 LIBDCP_DISABLE_WARNINGS
  29 #include <wx/richtext/richtextctrl.h>
  30 #include <wx/notebook.h>
  31 LIBDCP_ENABLE_WARNINGS
  32 #include <boost/algorithm/string.hpp>
  33
  34
  35 using std::list;
  36 using std::map;
  37 using std::shared_ptr;
  38 using std::string;
  39 using std::vector;
  40
  41
  42 VerifyDCPDialog::VerifyDCPDialog (wxWindow* parent, shared_ptr<VerifyDCPJob> job)
  43         : wxDialog (parent, wxID_ANY, _("DCP verification"), wxDefaultPosition, {600, 400})
  44 {
  45         auto sizer = new wxBoxSizer (wxVERTICAL);
  46         auto notebook = new wxNotebook (this, wxID_ANY);
  47         sizer->Add (notebook, 1, wxEXPAND | wxALL, DCPOMATIC_DIALOG_BORDER);
  48
  49         map<dcp::VerificationNote::Type, wxRichTextCtrl*> pages;
  50         pages[dcp::VerificationNote::Type::ERROR] = new wxRichTextCtrl (notebook, wxID_ANY, wxEmptyString, wxDefaultPosition, {400, 300}, wxRE_READONLY);
  51         notebook->AddPage (pages[dcp::VerificationNote::Type::ERROR], _("Errors"));
  52         pages[dcp::VerificationNote::Type::BV21_ERROR] = new wxRichTextCtrl (notebook, wxID_ANY, wxEmptyString, wxDefaultPosition, {400, 300}, wxRE_READONLY);
  53         notebook->AddPage (pages[dcp::VerificationNote::Type::BV21_ERROR], _("SMPTE Bv2.1 errors"));
  54         pages[dcp::VerificationNote::Type::WARNING] = new wxRichTextCtrl (notebook, wxID_ANY, wxEmptyString, wxDefaultPosition, {400, 300}, wxRE_READONLY);
  55         notebook->AddPage (pages[dcp::VerificationNote::Type::WARNING], _("Warnings"));
  56
  57         auto summary = new wxStaticText (this, wxID_ANY, wxT(""));
  58         sizer->Add (summary, 0, wxALL, DCPOMATIC_DIALOG_BORDER);
  59
  60         auto buttons = CreateStdDialogButtonSizer (0);
  61         sizer->Add (CreateSeparatedSizer(buttons), wxSizerFlags().Expand().DoubleBorder());
  62         buttons->SetAffirmativeButton (new wxButton (this, wxID_OK));
  63         buttons->Realize ();
  64
  65         SetSizer (sizer);
  66         sizer->Layout ();
  67         sizer->SetSizeHints (this);
  68
  69         for (auto const& i: pages) {
  70                 i.second->GetCaret()->Hide();
  71         }
  72
  73         if (job->finished_ok() && job->notes().empty()) {
  74                 summary->SetLabel (_("DCP validates OK."));
  75                 return;
  76         }
  77
  78         map<dcp::VerificationNote::Type, int> counts;
  79         counts[dcp::VerificationNote::Type::WARNING] = 0;
  80         counts[dcp::VerificationNote::Type::BV21_ERROR] = 0;
  81         counts[dcp::VerificationNote::Type::ERROR] = 0;
  82
  83         auto add_bullet = [&pages](dcp::VerificationNote::Type type, wxString message) {
  84                 pages[type]->BeginStandardBullet(N_("standard/diamond"), 1, 50);
  85                 pages[type]->WriteText (message);
  86                 pages[type]->Newline ();
  87                 pages[type]->EndStandardBullet ();
  88         };
  89
  90         auto add = [&counts, &add_bullet](dcp::VerificationNote note, wxString message) {
  91                 if (note.reference_hash()) {
  92                         message.Replace("%reference_hash", std_to_wx(note.reference_hash().get()));
  93                 }
  94                 if (note.calculated_hash()) {
  95                         message.Replace("%calculated_hash", std_to_wx(note.calculated_hash().get()));
  96                 }
  97                 if (note.frame()) {
  98                         message.Replace("%frame", std_to_wx(dcp::raw_convert<string>(note.frame().get())));
  99                         message.Replace(
 100                                 "%timecode",
 101                                 std_to_wx(
 102                                         dcp::Time(note.frame().get(), note.frame_rate().get(), note.frame_rate().get()).as_string(dcp::Standard::SMPTE)
 103                                         ));
 104                 }
 105                 if (note.note()) {
 106                         message.Replace("%n", std_to_wx(note.note().get()));
 107                 }
 108                 if (note.file()) {
 109                         message.Replace("%f", std_to_wx(note.file()->filename().string()));
 110                 }
 111                 if (note.line()) {
 112                         message.Replace("%l", std_to_wx(dcp::raw_convert<string>(note.line().get())));
 113                 }
 114                 if (note.component()) {
 115                         message.Replace("%component", std_to_wx(dcp::raw_convert<string>(note.component().get())));
 116                 }
 117                 if (note.size()) {
 118                         message.Replace("%size", std_to_wx(dcp::raw_convert<string>(note.size().get())));
 119                 }
 120                 if (note.id()) {
 121                         message.Replace("%id", std_to_wx(note.id().get()));
 122                 }
 123                 if (note.other_id()) {
 124                         message.Replace("%other_id", std_to_wx(note.other_id().get()));
 125                 }
 126                 add_bullet (note.type(), message);
 127                 counts[note.type()]++;
 128         };
 129
 130         if (job->finished_in_error() && job->error_summary() != "") {
 131                 /* We have an error that did not come from dcp::verify */
 132                 add_bullet (dcp::VerificationNote::Type::ERROR, std_to_wx(job->error_summary()));
 133                 ++counts[dcp::VerificationNote::Type::ERROR];
 134         }
 135
 136         for (auto i: job->notes()) {
 137                 switch (i.code()) {
 138                 case dcp::VerificationNote::Code::FAILED_READ:
 139                         add (i, _("Could not read DCP (%n)"));
 140                         break;
 141                 case dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES:
 142                         add(i, _("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."));
 143                         break;
 144                 case dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE:
 145                         add(i, _("The picture in a reel has a frame rate of %n, which is not valid."));
 146                         break;
 147                 case dcp::VerificationNote::Code::INCORRECT_PICTURE_HASH:
 148                         add(i, _("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."));
 149                         break;
 150                 case dcp::VerificationNote::Code::MISMATCHED_PICTURE_HASHES:
 151                         add(i, _("The PKL and CPL hashes disagree for picture asset %f."));
 152                         break;
 153                 case dcp::VerificationNote::Code::INCORRECT_SOUND_HASH:
 154                         add(i, _("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."));
 155                         break;
 156                 case dcp::VerificationNote::Code::MISMATCHED_SOUND_HASHES:
 157                         add(i, _("The PKL and CPL hashes disagree for sound asset %f."));
 158                         break;
 159                 case dcp::VerificationNote::Code::EMPTY_ASSET_PATH:
 160                         add(i, _("An asset has an empty path in the ASSETMAP."));
 161                         break;
 162                 case dcp::VerificationNote::Code::MISSING_ASSET:
 163                         add(i, _("The asset %f is missing."));
 164                         break;
 165                 case dcp::VerificationNote::Code::MISMATCHED_STANDARD:
 166                         add(i, _("Parts of the DCP are written according to the Interop standard and parts according to SMPTE."));
 167                         break;
 168                 case dcp::VerificationNote::Code::INVALID_XML:
 169                         if (i.line()) {
 170                                 add(i, _("The XML in %f is malformed on line %l (%n)."));
 171                         } else {
 172                                 add(i, _("The XML in %f is malformed (%n)."));
 173                         }
 174                         break;
 175                 case dcp::VerificationNote::Code::MISSING_ASSETMAP:
 176                         add(i, _("No ASSETMAP or ASSETMAP.xml file was found."));
 177                         break;
 178                 case dcp::VerificationNote::Code::INVALID_INTRINSIC_DURATION:
 179                         add(i, _("The asset %n has an intrinsic duration of less than 1 second, which is invalid."));
 180                         break;
 181                 case dcp::VerificationNote::Code::INVALID_DURATION:
 182                         add(i, _("The asset %n has a duration of less than 1 second, which is invalid."));
 183                         break;
 184                 case dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_SIZE_IN_BYTES:
 185                         add(i, _("At least one frame of the video asset %f is over the limit of 250Mbit/s."));
 186                         break;
 187                 case dcp::VerificationNote::Code::NEARLY_INVALID_PICTURE_FRAME_SIZE_IN_BYTES:
 188                         add(i, _("Frame %frame (timecode %timecode) in asset %f has an instantaneous bit rate that is close to the limit of 250Mbit/s."));
 189                         break;
 190                 case dcp::VerificationNote::Code::EXTERNAL_ASSET:
 191                         add(i, _("This DCP refers to at the asset %n in another DCP (and perhaps others), so it is a \"version file\" (VF)"));
 192                         break;
 193                 case dcp::VerificationNote::Code::THREED_ASSET_MARKED_AS_TWOD:
 194                         add(i, _("The asset %f is 3D but its MXF is marked as 2D."));
 195                         break;
 196                 case dcp::VerificationNote::Code::INVALID_STANDARD:
 197                         add(i, _("This DCP uses the Interop standard, but it should be made with SMPTE."));
 198                         break;
 199                 case dcp::VerificationNote::Code::INVALID_LANGUAGE:
 200                         add(i, _("The invalid language tag %n is used."));
 201                         break;
 202                 case dcp::VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS:
 203                         add(i, _("The video asset %f uses the invalid image size %n."));
 204                         break;
 205                 case dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_2K:
 206                         add(i, _("The video asset %f uses the invalid frame rate %n."));
 207                         break;
 208                 case dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_4K:
 209                         add(i, _("The video asset %f uses the frame rate %n which is invalid for 4K video."));
 210                         break;
 211                 case dcp::VerificationNote::Code::INVALID_PICTURE_ASSET_RESOLUTION_FOR_3D:
 212                         add(i, _("The video asset %f uses the frame rate %n which is invalid for 3D video."));
 213                         break;
 214                 case dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_XML_SIZE_IN_BYTES:
 215                         add(i, _("The XML in the closed caption asset %f takes up %n bytes which is over the 256KB limit."));
 216                         break;
 217                 case dcp::VerificationNote::Code::INVALID_TIMED_TEXT_SIZE_IN_BYTES:
 218                         add(i, _("The timed text asset %f takes up %n bytes which is over the 115MB limit."));
 219                         break;
 220                 case dcp::VerificationNote::Code::INVALID_TIMED_TEXT_FONT_SIZE_IN_BYTES:
 221                         add(i, _("The fonts in the timed text asset %f take up %n bytes which is over the 10MB limit."));
 222                         break;
 223                 case dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE:
 224                         add(i, _("The subtitle asset %f contains no <Language> tag."));
 225                         break;
 226                 case dcp::VerificationNote::Code::MISMATCHED_SUBTITLE_LANGUAGES:
 227                         add(i, _("Not all subtitle assets specify the same <Language> tag."));
 228                         break;
 229                 case dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME:
 230                         add(i, _("The subtitle asset %f contains no <StartTime> tag."));
 231                         break;
 232                 case dcp::VerificationNote::Code::INVALID_SUBTITLE_START_TIME:
 233                         add(i, _("The subtitle asset %f has a <StartTime> which is not zero."));
 234                         break;
 235                 case dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME:
 236                         add(i, _("The first subtitle or closed caption happens before 4s into the first reel."));
 237                         break;
 238                 case dcp::VerificationNote::Code::INVALID_SUBTITLE_DURATION:
 239                         add(i, _("At least one subtitle lasts less than 15 frames."));
 240                         break;
 241                 case dcp::VerificationNote::Code::INVALID_SUBTITLE_SPACING:
 242                         add(i, _("At least one pair of subtitles is separated by less than 2 frames."));
 243                         break;
 244                 case dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_COUNT:
 245                         add(i, _("There are more than 3 subtitle lines in at least one place."));
 246                         break;
 247                 case dcp::VerificationNote::Code::NEARLY_INVALID_SUBTITLE_LINE_LENGTH:
 248                         add(i, _("There are more than 52 characters in at least one subtitle line."));
 249                         break;
 250                 case dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_LENGTH:
 251                         add(i, _("There are more than 79 characters in at least one subtitle line."));
 252                         break;
 253                 case dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_COUNT:
 254                         add(i, _("There are more than 3 closed caption lines in at least one place."));
 255                         break;
 256                 case dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_LENGTH:
 257                         add(i, _("There are more than 32 characters in at least one closed caption line."));
 258                         break;
 259                 case dcp::VerificationNote::Code::INVALID_SOUND_FRAME_RATE:
 260                         add(i, _("The sound asset %f has an invalid frame rate of %n."));
 261                         break;
 262                 case dcp::VerificationNote::Code::MISSING_CPL_ANNOTATION_TEXT:
 263                         add(i, _("The CPL %n has no <AnnotationText> tag."));
 264                         break;
 265                 case dcp::VerificationNote::Code::MISMATCHED_CPL_ANNOTATION_TEXT:
 266                         add(i, _("The CPL %n has an <AnnotationText> which is not the same as its <ContentTitleText>."));
 267                         break;
 268                 case dcp::VerificationNote::Code::MISMATCHED_ASSET_DURATION:
 269                         add(i, _("At least one asset in a reel does not have the same duration as the others."));
 270                         break;
 271                 case dcp::VerificationNote::Code::MISSING_MAIN_SUBTITLE_FROM_SOME_REELS:
 272                         add(i, _("The DCP has subtitles but at least one reel has no subtitle asset."));
 273                         break;
 274                 case dcp::VerificationNote::Code::MISMATCHED_CLOSED_CAPTION_ASSET_COUNTS:
 275                         add(i, _("The DCP has closed captions but not every reel has the same number of closed caption assets."));
 276                         break;
 277                 case dcp::VerificationNote::Code::MISSING_SUBTITLE_ENTRY_POINT:
 278                         add(i, _("The subtitle asset %n has no <EntryPoint> tag."));
 279                         break;
 280                 case dcp::VerificationNote::Code::INCORRECT_SUBTITLE_ENTRY_POINT:
 281                         add(i, _("Subtitle asset %n has a non-zero <EntryPoint>."));
 282                         break;
 283                 case dcp::VerificationNote::Code::MISSING_CLOSED_CAPTION_ENTRY_POINT:
 284                         add(i, _("The closed caption asset %n has no <EntryPoint> tag."));
 285                         break;
 286                 case dcp::VerificationNote::Code::INCORRECT_CLOSED_CAPTION_ENTRY_POINT:
 287                         add(i, _("Closed caption asset %n has a non-zero <EntryPoint>."));
 288                         break;
 289                 case dcp::VerificationNote::Code::MISSING_HASH:
 290                         add(i, _("The asset %n has no <Hash> in the CPL."));
 291                         break;
 292                 case dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE:
 293                         add(i, _("The DCP is a feature but has no FFEC (first frame of end credits) marker."));
 294                         break;
 295                 case dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE:
 296                         add(i, _("The DCP is a feature but has no FFMC (first frame of moving credits) marker."));
 297                         break;
 298                 case dcp::VerificationNote::Code::MISSING_FFOC:
 299                         add(i, _("The DCP has no FFOC (first frame of content) marker."));
 300                         break;
 301                 case dcp::VerificationNote::Code::MISSING_LFOC:
 302                         add(i, _("The DCP has no LFOC (last frame of content) marker."));
 303                         break;
 304                 case dcp::VerificationNote::Code::INCORRECT_FFOC:
 305                         add(i, _("The DCP has a FFOC of %n instead of 1."));
 306                         break;
 307                 case dcp::VerificationNote::Code::INCORRECT_LFOC:
 308                         add(i, _("The DCP has a LFOC of %n instead of the reel duration minus one."));
 309                         break;
 310                 case dcp::VerificationNote::Code::MISSING_CPL_METADATA:
 311                         add(i, _("The CPL %n has no CPL metadata tag."));
 312                         break;
 313                 case dcp::VerificationNote::Code::MISSING_CPL_METADATA_VERSION_NUMBER:
 314                         add(i, _("The CPL %n has no CPL metadata version number tag."));
 315                         break;
 316                 case dcp::VerificationNote::Code::MISSING_EXTENSION_METADATA:
 317                         add(i, _("The CPL %n has no CPL extension metadata tag."));
 318                         break;
 319                 case dcp::VerificationNote::Code::INVALID_EXTENSION_METADATA:
 320                         add(i, _("The CPL %f has an invalid CPL extension metadata tag (%n)"));
 321                         break;
 322                 case dcp::VerificationNote::Code::UNSIGNED_CPL_WITH_ENCRYPTED_CONTENT:
 323                         add(i, _("The CPL %n has encrypted content but is not signed."));
 324                         break;
 325                 case dcp::VerificationNote::Code::UNSIGNED_PKL_WITH_ENCRYPTED_CONTENT:
 326                         add(i, _("The PKL %n has encrypted content but is not signed."));
 327                         break;
 328                 case dcp::VerificationNote::Code::MISMATCHED_PKL_ANNOTATION_TEXT_WITH_CPL:
 329                         add(i, _("The PKL %n has an <AnnotationText> which does not match its CPL's <ContentTitleText>."));
 330                         break;
 331                 case dcp::VerificationNote::Code::PARTIALLY_ENCRYPTED:
 332                         add(i, _("The DCP has encrypted content, but not all its assets are encrypted."));
 333                         break;
 334                 case dcp::VerificationNote::Code::INVALID_JPEG2000_CODESTREAM:
 335                         add(i, _("A picture frame has an invalid JPEG2000 codestream (%n)"));
 336                         break;
 337                 case dcp::VerificationNote::Code::INVALID_JPEG2000_GUARD_BITS_FOR_2K:
 338                         add(i, _("A 2K JPEG2000 frame has %n guard bits instead of 1."));
 339                         break;
 340                 case dcp::VerificationNote::Code::INVALID_JPEG2000_GUARD_BITS_FOR_4K:
 341                         add(i, _("A 4K JPEG2000 frame has %n guard bits instead of 2."));
 342                         break;
 343                 case dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_SIZE:
 344                         add(i, _("A JPEG2000 tile size does not match the image size."));
 345                         break;
 346                 case dcp::VerificationNote::Code::INVALID_JPEG2000_CODE_BLOCK_WIDTH:
 347                         add(i, _("A JPEG2000 frame has a code-block width of %n instead of 32."));
 348                         break;
 349                 case dcp::VerificationNote::Code::INVALID_JPEG2000_CODE_BLOCK_HEIGHT:
 350                         add(i, _("A JPEG2000 frame has a code-block height of %n instead of 32."));
 351                         break;
 352                 case dcp::VerificationNote::Code::INCORRECT_JPEG2000_POC_MARKER_COUNT_FOR_2K:
 353                         add(i, _("A 2K JPEG2000 frame has %n POC marker(s) instead of 0."));
 354                         break;
 355                 case dcp::VerificationNote::Code::INCORRECT_JPEG2000_POC_MARKER_COUNT_FOR_4K:
 356                         add(i, _("A 4K JPEG2000 frame has %n POC marker(s) instead of 1."));
 357                         break;
 358                 case dcp::VerificationNote::Code::INCORRECT_JPEG2000_POC_MARKER:
 359                         add(i, _("A JPEG2000 frame contains an invalid POC marker (%n)."));
 360                         break;
 361                 case dcp::VerificationNote::Code::INVALID_JPEG2000_POC_MARKER_LOCATION:
 362                         add(i, _("A JPEG2000 frame contains POC marker in an invalid location."));
 363                         break;
 364                 case dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_PARTS_FOR_2K:
 365                         add(i, _("A 2K JPEG2000 frame contains %n tile parts instead of 3."));
 366                         break;
 367                 case dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_PARTS_FOR_4K:
 368                         add(i, _("A 4K JPEG2000 frame contains %n tile parts instead of 6."));
 369                         break;
 370                 case dcp::VerificationNote::Code::MISSING_JPEG200_TLM_MARKER:
 371                         add(i, _("A JPEG2000 frame has no TLM marker."));
 372                         break;
 373                 case dcp::VerificationNote::Code::SUBTITLE_OVERLAPS_REEL_BOUNDARY:
 374                         add(i, _("A subtitle lasts longer than the reel it is in."));
 375                         break;
 376                 case dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_RESOURCE_ID:
 377                         add(i, _("The Resource ID in a timed text MXF did not match the ID of the contained XML."));
 378                         break;
 379                 case dcp::VerificationNote::Code::INCORRECT_TIMED_TEXT_ASSET_ID:
 380                         add(i, _("The Asset ID in a timed text MXF is the same as the Resource ID or that of the contained XML."));
 381                         break;
 382                 case dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_DURATION:
 383                 {
 384                         vector<string> parts;
 385                         boost::split (parts, i.note().get(), boost::is_any_of(" "));
 386                         add(i, 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])));
 387                         break;
 388                 }
 389                 case dcp::VerificationNote::Code::MISSED_CHECK_OF_ENCRYPTED:
 390                         add(i, _("Part of the DCP could not be checked because no KDM was available."));
 391                         break;
 392                 case dcp::VerificationNote::Code::EMPTY_TEXT:
 393                         add(i, _("At least one <Text> node in a subtitle or closed caption is empty."));
 394                         break;
 395                 case dcp::VerificationNote::Code::MISMATCHED_CLOSED_CAPTION_VALIGN:
 396                         add(i, _("Some closed <Text> or <Image> nodes have different vertical alignments within a <Subtitle>."));
 397                         break;
 398                 case dcp::VerificationNote::Code::INCORRECT_CLOSED_CAPTION_ORDERING:
 399                         add(i, _("Some closed captions are not listed in the order of their vertical position."));
 400                         break;
 401                 case dcp::VerificationNote::Code::UNEXPECTED_ENTRY_POINT:
 402                         add(i, _("There is a <EntryPoint> tag inside a <MainMarkers>."));
 403                         break;
 404                 case dcp::VerificationNote::Code::UNEXPECTED_DURATION:
 405                         add(i, _("There is a <Duration> tag inside a <MainMarkers>."));
 406                         break;
 407                 case dcp::VerificationNote::Code::INVALID_CONTENT_KIND:
 408                         add(i, _("An invalid <ContentKind> %n has been used."));
 409                         break;
 410                 case dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA:
 411                         add(i, _("The <MainPictureActiveArea> is either not a multiple of 2, or is bigger than an asset."));
 412                         break;
 413                 case dcp::VerificationNote::Code::DUPLICATE_ASSET_ID_IN_PKL:
 414                         add(i, _("The PKL %n has more than one asset with the same ID."));
 415                         break;
 416                 case dcp::VerificationNote::Code::DUPLICATE_ASSET_ID_IN_ASSETMAP:
 417                         add(i, _("The ASSETMAP %n has more than one asset with the same ID."));
 418                         break;
 419                 case dcp::VerificationNote::Code::MISSING_SUBTITLE:
 420                         add(i, _("The subtitle asset %n contains no subtitles."));
 421                         break;
 422                 case dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE:
 423                         add(i, _("<IssueDate> has an invalid value %n"));
 424                         break;
 425                 case dcp::VerificationNote::Code::MISMATCHED_SOUND_CHANNEL_COUNTS:
 426                         add(i, _("Sound assets do not all have the same channel count."));
 427                         break;
 428                 case dcp::VerificationNote::Code::INVALID_MAIN_SOUND_CONFIGURATION:
 429                         add(i, _("<MainSoundConfiguration> describes incorrect number of channels (%n)"));
 430                         break;
 431                 case dcp::VerificationNote::Code::MISSING_FONT:
 432                         add(i, _("The font file for font ID \"%n\" was not found, or was not referred to in the ASSETMAP."));
 433                         break;
 434                 case dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_PART_SIZE:
 435                         add(i, _("Frame %frame has an image component that is too large (component %component is %size bytes in size)."));
 436                         break;
 437                 case dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT:
 438                         add(i, _("The XML in the subtitle asset %n has more than one namespace declaration."));
 439                         break;
 440                 case dcp::VerificationNote::Code::MISSING_LOAD_FONT_FOR_FONT:
 441                         add(i, _("A subtitle or closed caption refers to a font with ID %id that does not have a corresponding <LoadFont> node."));
 442                         break;
 443                 case dcp::VerificationNote::Code::MISSING_LOAD_FONT:
 444                         add(i, _("The SMPTE subtitle asset %id has <Text> nodes but no <LoadFont> node"));
 445                         break;
 446                 case dcp::VerificationNote::Code::MISMATCHED_ASSET_MAP_ID:
 447                         add(i, _("The asset with ID %id in the asset map actually has an id of %other_id"));
 448                         break;
 449                 case dcp::VerificationNote::Code::EMPTY_CONTENT_VERSION_LABEL_TEXT:
 450                         add(i, _("The <LabelText> in a <ContentVersion> in CPL %id is empty"));
 451                         break;
 452                 case dcp::VerificationNote::Code::INVALID_CPL_NAMESPACE:
 453                         add(i, _("The CPL %f has an invalid namespace %n"));
 454                         break;
 455                 case dcp::VerificationNote::Code::MISSING_CPL_CONTENT_VERSION:
 456                         add(i, _("The CPL %n has no <ContentVersion> tag"));
 457                         break;
 458                 }
 459         }
 460
 461         wxString summary_text;
 462
 463         if (counts[dcp::VerificationNote::Type::ERROR] == 1) {
 464                 /// TRANSLATORS: this will be used at the start of a string like "1 error, 2 Bv2.1 errors and 3 warnings."
 465                 summary_text = _("1 error, ");
 466         } else {
 467                 /// TRANSLATORS: this will be used at the start of a string like "1 error, 2 Bv2.1 errors and 3 warnings."
 468                 summary_text = wxString::Format("%d errors, ", counts[dcp::VerificationNote::Type::ERROR]);
 469         }
 470
 471         if (counts[dcp::VerificationNote::Type::BV21_ERROR] == 1) {
 472                 /// TRANSLATORS: this will be used in the middle of a string like "1 error, 2 Bv2.1 errors and 3 warnings."
 473                 summary_text += _("1 Bv2.1 error, ");
 474         } else {
 475                 /// TRANSLATORS: this will be used in the middle of a string like "1 error, 2 Bv2.1 errors and 3 warnings."
 476                 summary_text += wxString::Format("%d Bv2.1 errors, ", counts[dcp::VerificationNote::Type::BV21_ERROR]);
 477         }
 478
 479         if (counts[dcp::VerificationNote::Type::WARNING] == 1) {
 480                 /// TRANSLATORS: this will be used at the end of a string like "1 error, 2 Bv2.1 errors and 3 warnings."
 481                 summary_text += _("and 1 warning.");
 482         } else {
 483                 /// TRANSLATORS: this will be used at the end of a string like "1 error, 2 Bv2.1 errors and 3 warnings."
 484                 summary_text += wxString::Format("and %d warnings.", counts[dcp::VerificationNote::Type::WARNING]);
 485         }
 486
 487         summary->SetLabel(summary_text);
 488
 489         if (counts[dcp::VerificationNote::Type::ERROR] == 0) {
 490                 add_bullet (dcp::VerificationNote::Type::ERROR, _("No errors found."));
 491         }
 492
 493         if (counts[dcp::VerificationNote::Type::BV21_ERROR] == 0) {
 494                 add_bullet (dcp::VerificationNote::Type::BV21_ERROR, _("No SMPTE Bv2.1 errors found."));
 495         }
 496
 497         if (counts[dcp::VerificationNote::Type::WARNING] == 0) {
 498                 add_bullet (dcp::VerificationNote::Type::WARNING, _("No warnings found."));
 499         }
 500 }