Move upload-DCP-to-TMS button to preferences.

5 files changed, 16 insertions, 29 deletions

diff --git a/src/lib/config.cc b/src/lib/config.cc
index a138cc879..b1f2a4f18 100644
--- a/src/lib/config.cc
+++ b/src/lib/config.cc
@@ -104,7 +104,7 @@ Config::set_defaults ()
 	_default_j2k_bandwidth = 150000000;
 	_default_audio_delay = 0;
 	_default_interop = true;
-	_default_upload_after_make_dcp = false;
+	_upload_after_make_dcp = false;
 	_mail_server = "";
 	_mail_port = 25;
 	_mail_protocol = EMAIL_PROTOCOL_AUTO;
@@ -317,7 +317,11 @@ try
 		_dcp_issuer = f.string_child ("DCPIssuer");
 	}
 
-	_default_upload_after_make_dcp = f.optional_bool_child("DefaultUploadAfterMakeDCP").get_value_or (false);
+	optional<bool> up = f.optional_bool_child("UploadAfterMakeDCP");
+	if (!up) {
+		up = f.optional_bool_child("DefaultUploadAfterMakeDCP");
+	}
+	_upload_after_make_dcp = up.get_value_or (false);
 	_dcp_creator = f.optional_string_child ("DCPCreator").get_value_or ("");
 
 	if (version && version.get() >= 2) {
@@ -725,8 +729,8 @@ Config::write_config () const
 	root->add_child("DCPIssuer")->add_child_text (_dcp_issuer);
 	/* [XML] DCPIssuer Creator text to write into CPL files. */
 	root->add_child("DCPCreator")->add_child_text (_dcp_creator);
-	/* [XML] DefaultUploadAfterMakeDCP 1 to default to uploading to a TMS after making a DCP, 0 to default to no upload. */
-	root->add_child("DefaultUploadAfterMakeDCP")->add_child_text (_default_upload_after_make_dcp ? "1" : "0");
+	/* [XML] UploadAfterMakeDCP 1 to upload to a TMS after making a DCP, 0 for no upload. */
+	root->add_child("UploadAfterMakeDCP")->add_child_text (_upload_after_make_dcp ? "1" : "0");
 
 	/* [XML] ISDCFMetadata Default ISDCF metadata to use for new films; child tags are <code>&lt;ContentVersion&gt;</code>,
 	   <code>&lt;AudioLanguage&gt;</code>, <code>&lt;SubtitleLanguage&gt;</code>, <code>&lt;Territory&gt;</code>,
diff --git a/src/lib/config.h b/src/lib/config.h
index 23f7ce0b3..4b7488d15 100644
--- a/src/lib/config.h
+++ b/src/lib/config.h
@@ -214,8 +214,8 @@ public:
 		return _default_interop;
 	}
 
-	bool default_upload_after_make_dcp () {
-		return _default_upload_after_make_dcp;
+	bool upload_after_make_dcp () {
+		return _upload_after_make_dcp;
 	}
 
 	void set_default_kdm_directory (boost::filesystem::path d) {
@@ -710,8 +710,8 @@ public:
 		maybe_set (_default_interop, i);
 	}
 
-	void set_default_upload_after_make_dcp (bool u) {
-		maybe_set (_default_upload_after_make_dcp, u);
+	void set_upload_after_make_dcp (bool u) {
+		maybe_set (_upload_after_make_dcp, u);
 	}
 
 	void set_mail_server (std::string s) {
@@ -1236,7 +1236,7 @@ private:
 	    the home directory will be offered.
 	*/
 	boost::optional<boost::filesystem::path> _default_kdm_directory;
-	bool _default_upload_after_make_dcp;
+	bool _upload_after_make_dcp;
 	std::list<boost::shared_ptr<Cinema> > _cinemas;
 	std::list<boost::shared_ptr<DKDMRecipient> > _dkdm_recipients;
 	std::string _mail_server;
diff --git a/src/lib/film.cc b/src/lib/film.cc
index 93a22d186..c04f2996f 100644
--- a/src/lib/film.cc
+++ b/src/lib/film.cc
@@ -160,7 +160,6 @@ Film::Film (optional<boost::filesystem::path> dir)
 	, _audio_processor (0)
 	, _reel_type (REELTYPE_SINGLE)
 	, _reel_length (2000000000)
-	, _upload_after_make_dcp (Config::instance()->default_upload_after_make_dcp())
 	, _reencode_j2k (false)
 	, _user_explicit_video_frame_rate (false)
 	, _user_explicit_container (false)
@@ -454,7 +453,6 @@ Film::metadata (bool with_content_paths) const
 	}
 	root->add_child("ReelType")->add_child_text (raw_convert<string> (static_cast<int> (_reel_type)));
 	root->add_child("ReelLength")->add_child_text (raw_convert<string> (_reel_length));
-	root->add_child("UploadAfterMakeDCP")->add_child_text (_upload_after_make_dcp ? "1" : "0");
 	root->add_child("ReencodeJ2K")->add_child_text (_reencode_j2k ? "1" : "0");
 	root->add_child("UserExplicitVideoFrameRate")->add_child_text(_user_explicit_video_frame_rate ? "1" : "0");
 	for (map<dcp::Marker, DCPTime>::const_iterator i = _markers.begin(); i != _markers.end(); ++i) {
@@ -602,7 +600,6 @@ Film::read_metadata (optional<boost::filesystem::path> path)
 
 	_reel_type = static_cast<ReelType> (f.optional_number_child<int>("ReelType").get_value_or (static_cast<int>(REELTYPE_SINGLE)));
 	_reel_length = f.optional_number_child<int64_t>("ReelLength").get_value_or (2000000000);
-	_upload_after_make_dcp = f.optional_bool_child("UploadAfterMakeDCP").get_value_or (false);
 	_reencode_j2k = f.optional_bool_child("ReencodeJ2K").get_value_or(false);
 	_user_explicit_video_frame_rate = f.optional_bool_child("UserExplicitVideoFrameRate").get_value_or(false);
 
@@ -1085,13 +1082,6 @@ Film::set_reel_length (int64_t r)
 }
 
 void
-Film::set_upload_after_make_dcp (bool u)
-{
-	ChangeSignaller<Film> ch (this, UPLOAD_AFTER_MAKE_DCP);
-	_upload_after_make_dcp = u;
-}
-
-void
 Film::set_reencode_j2k (bool r)
 {
 	ChangeSignaller<Film> ch (this, REENCODE_J2K);
@@ -1748,7 +1738,6 @@ Film::use_template (string name)
 	_audio_processor = _template_film->_audio_processor;
 	_reel_type = _template_film->_reel_type;
 	_reel_length = _template_film->_reel_length;
-	_upload_after_make_dcp = _template_film->_upload_after_make_dcp;
 	_isdcf_metadata = _template_film->_isdcf_metadata;
 }
 
diff --git a/src/lib/film.h b/src/lib/film.h
index 7f7590210..64bb159c7 100644
--- a/src/lib/film.h
+++ b/src/lib/film.h
@@ -224,7 +224,6 @@ public:
 		AUDIO_PROCESSOR,
 		REEL_TYPE,
 		REEL_LENGTH,
-		UPLOAD_AFTER_MAKE_DCP,
 		REENCODE_J2K,
 		MARKERS,
 		RATINGS,
@@ -307,10 +306,6 @@ public:
 		return _reel_length;
 	}
 
-	bool upload_after_make_dcp () const {
-		return _upload_after_make_dcp;
-	}
-
 	std::string context_id () const {
 		return _context_id;
 	}
@@ -358,7 +353,6 @@ public:
 	void set_audio_processor (AudioProcessor const * processor);
 	void set_reel_type (ReelType);
 	void set_reel_length (int64_t);
-	void set_upload_after_make_dcp (bool);
 	void set_reencode_j2k (bool);
 	void set_marker (dcp::Marker type, dcpomatic::DCPTime time);
 	void unset_marker (dcp::Marker type);
@@ -450,7 +444,6 @@ private:
 	ReelType _reel_type;
 	/** Desired reel length in bytes, if _reel_type == REELTYPE_BY_LENGTH */
 	int64_t _reel_length;
-	bool _upload_after_make_dcp;
 	bool _reencode_j2k;
 	/** true if the user has ever explicitly set the video frame rate of this film */
 	bool _user_explicit_video_frame_rate;
diff --git a/src/lib/transcode_job.cc b/src/lib/transcode_job.cc
index fb586b082..6d73a3673 100644
--- a/src/lib/transcode_job.cc
+++ b/src/lib/transcode_job.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2019 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2020 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -22,6 +22,7 @@
  *  @brief A job which transcodes from one format to another.
  */
 
+#include "config.h"
 #include "transcode_job.h"
 #include "dcp_encoder.h"
 #include "upload_job.h"
@@ -101,7 +102,7 @@ TranscodeJob::run ()
 		}
 
 		/* XXX: this shouldn't be here */
-		if (_film->upload_after_make_dcp() && dynamic_pointer_cast<DCPEncoder>(_encoder)) {
+		if (Config::instance()->upload_after_make_dcp() && dynamic_pointer_cast<DCPEncoder>(_encoder)) {
 			shared_ptr<Job> job (new UploadJob (_film));
 			JobManager::instance()->add (job);
 		}