Don't offer to make KDMs for CPLs with no encrypted assets (#1490).

2 files changed, 35 insertions, 6 deletions

diff --git a/src/wx/kdm_cpl_panel.cc b/src/wx/kdm_cpl_panel.cc
index db05db27c..341b5f764 100644
--- a/src/wx/kdm_cpl_panel.cc
+++ b/src/wx/kdm_cpl_panel.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2015-2018 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2015-2019 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -23,6 +23,7 @@
 #include "static_text.h"
 #include "dcpomatic_button.h"
 #include <libcxml/cxml.h>
+#include <boost/foreach.hpp>
 
 using std::vector;
 
@@ -67,8 +68,8 @@ KDMCPLPanel::update_cpl_choice ()
 {
 	_cpl->Clear ();
 
-	for (vector<CPLSummary>::const_iterator i = _cpls.begin(); i != _cpls.end(); ++i) {
-		_cpl->Append (std_to_wx (i->cpl_id));
+	BOOST_FOREACH (CPLSummary const & i, _cpls) {
+		_cpl->Append (std_to_wx(i.cpl_id));
 
 		if (_cpls.size() > 0) {
 			_cpl->SetSelection (0);
@@ -109,13 +110,33 @@ KDMCPLPanel::cpl_browse_clicked ()
 	cxml::Document cpl_document ("CompositionPlaylist");
 	cpl_document.read_file (cpl_file);
 
+	bool encrypted = false;
+	BOOST_FOREACH (cxml::ConstNodePtr i, cpl_document.node_children("ReelList")) {
+		BOOST_FOREACH (cxml::ConstNodePtr j, i->node_children("Reel")) {
+			BOOST_FOREACH (cxml::ConstNodePtr k, j->node_children("AssetList")) {
+				BOOST_FOREACH (cxml::ConstNodePtr l, k->node_children()) {
+					if (!l->node_children("KeyId").empty()) {
+						encrypted = true;
+					}
+				}
+			}
+		}
+	}
+
+	if (!encrypted) {
+		/* XXX: this needs to be marked translatable after 2.14.0 */
+		error_dialog (this, wxT("This CPL contains no encrypted assets."));
+		return;
+	}
+
 	try {
 		_cpls.push_back (
 			CPLSummary (
 				dcp_dir.filename().string(),
 				cpl_document.string_child("Id").substr (9),
-				cpl_document.string_child ("ContentTitleText"),
-				cpl_file
+				cpl_document.string_child("ContentTitleText"),
+				cpl_file,
+				encrypted
 				)
 			);
 	} catch (cxml::Error) {
diff --git a/src/wx/kdm_dialog.cc b/src/wx/kdm_dialog.cc
index c31e6f643..920f54a3e 100644
--- a/src/wx/kdm_dialog.cc
+++ b/src/wx/kdm_dialog.cc
@@ -87,7 +87,15 @@ KDMDialog::KDMDialog (wxWindow* parent, shared_ptr<const Film> film)
 	h = new StaticText (this, _("CPL"));
 	h->SetFont (subheading_font);
 	right->Add (h, 0, wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_Y_GAP * 2);
-	_cpl = new KDMCPLPanel (this, film->cpls ());
+
+	vector<CPLSummary> cpls;
+	BOOST_FOREACH (CPLSummary const & i, film->cpls()) {
+		if (i.encrypted) {
+			cpls.push_back (i);
+		}
+	}
+
+	_cpl = new KDMCPLPanel (this, cpls);
 	right->Add (_cpl, 0, wxEXPAND);
 
 	/* Sub-heading: Output */