#include "verify_dcp_dialog.h"
#include "wx_util.h"
+#include "lib/verify_dcp_job.h"
+#include "lib/warnings.h"
#include <dcp/verify.h>
+DCPOMATIC_DISABLE_WARNINGS
#include <wx/richtext/richtextctrl.h>
+DCPOMATIC_ENABLE_WARNINGS
#include <boost/foreach.hpp>
using std::list;
+using std::shared_ptr;
-VerifyDCPDialog::VerifyDCPDialog (wxWindow* parent, list<dcp::VerificationNote> notes)
+VerifyDCPDialog::VerifyDCPDialog (wxWindow* parent, shared_ptr<VerifyDCPJob> job)
: wxDialog (parent, wxID_ANY, _("DCP verification"))
{
wxBoxSizer* sizer = new wxBoxSizer (wxVERTICAL);
_text->GetCaret()->Hide ();
- if (notes.empty ()) {
+ if (job->finished_ok() && job->notes().empty()) {
_text->BeginStandardBullet (N_("standard/circle"), 1, 50);
_text->WriteText (_("DCP validates OK."));
_text->EndStandardBullet ();
return;
}
- BOOST_FOREACH (dcp::VerificationNote i, notes) {
+ /* We might have an error that did not come from dcp::verify; report it if so */
+ if (job->finished_in_error() && job->error_summary() != "") {
+ _text->BeginSymbolBullet (N_("!"), 1, 50);
+ _text->WriteText(std_to_wx(job->error_summary()));
+ _text->Newline();
+ }
+
+ BOOST_FOREACH (dcp::VerificationNote i, job->notes()) {
switch (i.type()) {
- case dcp::VerificationNote::VERIFY_NOTE:
- _text->BeginStandardBullet (N_("standard/circle"), 1, 50);
- break;
case dcp::VerificationNote::VERIFY_WARNING:
_text->BeginStandardBullet (N_("standard/diamond"), 1, 50);
break;
break;
}
- _text->WriteText (std_to_wx (i.note()));
+ wxString text;
+ switch (i.code()) {
+ case dcp::VerificationNote::GENERAL_READ:
+ text = std_to_wx(*i.note());
+ break;
+ case dcp::VerificationNote::CPL_HASH_INCORRECT:
+ text = _("The hash of the CPL in the PKL does not agree with the CPL file. This probably means that the CPL file is corrupt.");
+ break;
+ case dcp::VerificationNote::INVALID_PICTURE_FRAME_RATE:
+ text = _("The picture in a reel has an invalid frame rate");
+ break;
+ case dcp::VerificationNote::PICTURE_HASH_INCORRECT:
+ text = wxString::Format(
+ _("The hash of the picture asset %s does not agree with the PKL file. This probably means that the asset file is corrupt."),
+ std_to_wx(i.file()->filename().string())
+ );
+ break;
+ case dcp::VerificationNote::PKL_CPL_PICTURE_HASHES_DISAGREE:
+ text = _("The PKL and CPL hashes disagree for a picture asset.");
+ break;
+ case dcp::VerificationNote::SOUND_HASH_INCORRECT:
+ text = wxString::Format(
+ _("The hash of the sound asset %s does not agree with the PKL file. This probably means that the asset file is corrupt."),
+ std_to_wx(i.file()->filename().string())
+ );
+ break;
+ case dcp::VerificationNote::PKL_CPL_SOUND_HASHES_DISAGREE:
+ text = _("The PKL and CPL hashes disagree for a sound asset.");
+ break;
+ case dcp::VerificationNote::EMPTY_ASSET_PATH:
+ text = _("An asset has an empty path in the ASSETMAP.");
+ break;
+ case dcp::VerificationNote::MISSING_ASSET:
+ text = _("An asset is missing.");
+ break;
+ case dcp::VerificationNote::MISMATCHED_STANDARD:
+ text = _("Parts of the DCP are written according to the Interop standard and parts according to SMPTE.");
+ break;
+ case dcp::VerificationNote::XML_VALIDATION_ERROR:
+ if (i.line()) {
+ text = wxString::Format(
+ _("The XML in %s is malformed on line %" PRIu64 "."),
+ std_to_wx(i.file()->filename().string()),
+ i.line().get()
+ );
+ } else {
+ text = wxString::Format(
+ _("The XML in %s is malformed."),
+ std_to_wx(i.file()->filename().string())
+ );
+ }
+ break;
+ case dcp::VerificationNote::MISSING_ASSETMAP:
+ text = _("No ASSETMAP or ASSETMAP.xml file was found.");
+ break;
+ case dcp::VerificationNote::INTRINSIC_DURATION_TOO_SMALL:
+ text = _("An asset has an instrinsic duration of less than 1 second, which is invalid.");
+ break;
+ case dcp::VerificationNote::DURATION_TOO_SMALL:
+ text = _("An asset has a duration of less than 1 second, which is invalid.");
+ break;
+ case dcp::VerificationNote::PICTURE_FRAME_TOO_LARGE:
+ text = _("At least one frame of the video data is over the limit of 250Mbit/s.");
+ break;
+ case dcp::VerificationNote::PICTURE_FRAME_NEARLY_TOO_LARGE:
+ text = _("At least one frame of the video data is close to the limit of 250MBit/s.");
+ break;
+ case dcp::VerificationNote::EXTERNAL_ASSET:
+ text = _("This DCP refers to at least one asset in another DCP, so it is a \"version file\" (VF)");
+ break;
+ }
+
+ _text->WriteText (text);
_text->Newline ();
switch (i.type()) {
- case dcp::VerificationNote::VERIFY_NOTE:
case dcp::VerificationNote::VERIFY_WARNING:
_text->EndStandardBullet ();
break;
projects / dcpomatic.git / blobdiff