X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fconfig.cc;h=4aa4788bf1493001945ee06ef57fcbb4cc4e94f0;hb=ccacce39c39d16977ab6c1592fcb6e941b05ddff;hp=ce02b046e4e91635857ec0a2daee5df1535097dc;hpb=c7917ecfce3200d807091cb64241e54066822cbf;p=dcpomatic.git

diff --git a/src/lib/config.cc b/src/lib/config.cc
index ce02b046e..4aa4788bf 100644
--- a/src/lib/config.cc
+++ b/src/lib/config.cc
@@ -56,8 +56,6 @@ using std::list;
 using std::min;
 using std::max;
 using std::remove;
-using std::exception;
-using std::cerr;
 using std::shared_ptr;
 using std::make_shared;
 using boost::optional;
@@ -104,6 +102,7 @@ Config::set_defaults ()
 	_default_j2k_bandwidth = 150000000;
 	_default_audio_delay = 0;
 	_default_interop = false;
+	_default_metadata.clear ();
 	_upload_after_make_dcp = false;
 	_mail_server = "";
 	_mail_port = 25;
@@ -126,8 +125,8 @@ Config::set_defaults ()
 #ifdef DCPOMATIC_WINDOWS
 	_win32_console = false;
 #endif
-	_cinemas_file = path ("cinemas.xml");
-	_dkdm_recipients_file = path ("dkdm_recipients.xml");
+	_cinemas_file = read_path ("cinemas.xml");
+	_dkdm_recipients_file = read_path ("dkdm_recipients.xml");
 	_show_hints_before_make_dcp = true;
 	_confirm_kdm_email = true;
 	_kdm_container_name_format = dcp::NameFormat ("KDM %f %c");
@@ -176,6 +175,7 @@ Config::set_defaults ()
 	_player_kdm_directory = boost::none;
 	_audio_mapping = boost::none;
 	_custom_languages.clear ();
+	_add_files_path = boost::none;
 
 	_allowed_dcp_frame_rates.clear ();
 	_allowed_dcp_frame_rates.push_back (24);
@@ -216,13 +216,13 @@ Config::backup ()
 	/* Make a copy of the configuration */
 	try {
 		int n = 1;
-		while (n < 100 && boost::filesystem::exists(path(String::compose("config.xml.%1", n)))) {
+		while (n < 100 && boost::filesystem::exists(write_path(String::compose("config.xml.%1", n)))) {
 			++n;
 		}
 
-		boost::filesystem::copy_file(path("config.xml", false), path(String::compose("config.xml.%1", n), false));
-		boost::filesystem::copy_file(path("cinemas.xml", false), path(String::compose("cinemas.xml.%1", n), false));
-		boost::filesystem::copy_file(path("dkdm_recipients.xml", false), path(String::compose("dkdm_recipients.xml.%1", n), false));
+		boost::filesystem::copy_file(read_path("config.xml"), write_path(String::compose("config.xml.%1", n)));
+		boost::filesystem::copy_file(read_path("cinemas.xml"), write_path(String::compose("cinemas.xml.%1", n)));
+		boost::filesystem::copy_file(read_path("dkdm_recipients.xml"), write_path(String::compose("dkdm_recipients.xml.%1", n)));
 	} catch (...) {}
 }
 
@@ -231,7 +231,7 @@ Config::read ()
 try
 {
 	cxml::Document f ("Config");
-	f.read_file (config_file ());
+	f.read_file (config_read_file());
 
 	auto version = f.optional_number_child<int> ("Version");
 	if (version && *version < _current_version) {
@@ -312,6 +312,11 @@ try
 	_default_j2k_bandwidth = f.optional_number_child<int>("DefaultJ2KBandwidth").get_value_or (200000000);
 	_default_audio_delay = f.optional_number_child<int>("DefaultAudioDelay").get_value_or (0);
 	_default_interop = f.optional_bool_child("DefaultInterop").get_value_or (false);
+
+	for (auto const& i: f.node_children("DefaultMetadata")) {
+		_default_metadata[i->string_attribute("key")] = i->content();
+	}
+
 	_default_kdm_directory = f.optional_string_child("DefaultKDMDirectory");
 
 	/* Read any cinemas that are still lying around in the config file
@@ -462,8 +467,8 @@ try
 			_dkdms->add (DKDMBase::read (i));
 		}
 	}
-	_cinemas_file = f.optional_string_child("CinemasFile").get_value_or (path ("cinemas.xml").string ());
-	_dkdm_recipients_file = f.optional_string_child("DKDMRecipientsFile").get_value_or (path("dkdm_recipients.xml").string());
+	_cinemas_file = f.optional_string_child("CinemasFile").get_value_or(read_path("cinemas.xml").string());
+	_dkdm_recipients_file = f.optional_string_child("DKDMRecipientsFile").get_value_or(read_path("dkdm_recipients.xml").string());
 	_show_hints_before_make_dcp = f.optional_bool_child("ShowHintsBeforeMakeDCP").get_value_or (true);
 	_confirm_kdm_email = f.optional_bool_child("ConfirmKDMEmail").get_value_or (true);
 	_kdm_container_name_format = dcp::NameFormat (f.optional_string_child("KDMContainerNameFormat").get_value_or ("KDM %f %c"));
@@ -553,6 +558,8 @@ try
 		} catch (std::runtime_error& e) {}
 	}
 
+	_add_files_path = f.optional_string_child("AddFilesPath");
+
 	if (boost::filesystem::exists (_cinemas_file)) {
 		cxml::Document f ("Cinemas");
 		f.read_file (_cinemas_file);
@@ -687,6 +694,11 @@ Config::write_config () const
 	root->add_child("DefaultAudioDelay")->add_child_text (raw_convert<string> (_default_audio_delay));
 	/* [XML] DefaultInterop 1 to default new films to Interop, 0 for SMPTE. */
 	root->add_child("DefaultInterop")->add_child_text (_default_interop ? "1" : "0");
+	for (auto const& i: _default_metadata) {
+		auto c = root->add_child("DefaultMetadata");
+		c->set_attribute("key", i.first);
+		c->add_child_text(i.second);
+	}
 	if (_default_kdm_directory) {
 		/* [XML:opt] DefaultKDMDirectory Default directory to write KDMs to. */
 		root->add_child("DefaultKDMDirectory")->add_child_text (_default_kdm_directory->string ());
@@ -976,22 +988,28 @@ Config::write_config () const
 	for (auto const& i: _custom_languages) {
 		root->add_child("CustomLanguage")->add_child_text(i.to_string());
 	}
+	if (_add_files_path) {
+		/* [XML] The default path that will be offered in the picker when adding files to a film */
+		root->add_child("AddFilesPath")->add_child_text(_add_files_path->string());
+	}
+
+	auto target = config_write_file();
 
 	try {
 		auto const s = doc.write_to_string_formatted ();
-		boost::filesystem::path tmp (string(config_file().string()).append(".tmp"));
+		boost::filesystem::path tmp (string(target.string()).append(".tmp"));
 		auto f = fopen_boost (tmp, "w");
 		if (!f) {
 			throw FileError (_("Could not open file for writing"), tmp);
 		}
 		checked_fwrite (s.c_str(), s.bytes(), f, tmp);
 		fclose (f);
-		boost::filesystem::remove (config_file());
-		boost::filesystem::rename (tmp, config_file());
+		boost::filesystem::remove (target);
+		boost::filesystem::rename (tmp, target);
 	} catch (xmlpp::exception& e) {
 		string s = e.what ();
 		trim (s);
-		throw FileError (s, config_file());
+		throw FileError (s, target);
 	}
 }
 
@@ -1185,12 +1203,14 @@ Config::clean_history_internal (vector<boost::filesystem::path>& h)
 	}
 }
 
+
 bool
 Config::have_existing (string file)
 {
-	return boost::filesystem::exists (path (file, false));
+	return boost::filesystem::exists (read_path(file));
 }
 
+
 void
 Config::read_cinemas (cxml::Document const & f)
 {
@@ -1257,18 +1277,19 @@ Config::set_dkdm_recipients_file (boost::filesystem::path file)
 void
 Config::save_template (shared_ptr<const Film> film, string name) const
 {
-	film->write_template (template_path (name));
+	film->write_template (template_write_path(name));
 }
 
+
 list<string>
 Config::templates () const
 {
-	if (!boost::filesystem::exists (path ("templates"))) {
+	if (!boost::filesystem::exists(read_path("templates"))) {
 		return {};
 	}
 
 	list<string> n;
-	for (auto const& i: boost::filesystem::directory_iterator(path("templates"))) {
+	for (auto const& i: boost::filesystem::directory_iterator(read_path("templates"))) {
 		n.push_back (i.path().filename().string());
 	}
 	return n;
@@ -1277,33 +1298,41 @@ Config::templates () const
 bool
 Config::existing_template (string name) const
 {
-	return boost::filesystem::exists (template_path (name));
+	return boost::filesystem::exists (template_read_path(name));
+}
+
+
+boost::filesystem::path
+Config::template_read_path (string name) const
+{
+	return read_path("templates") / tidy_for_filename (name);
 }
 
+
 boost::filesystem::path
-Config::template_path (string name) const
+Config::template_write_path (string name) const
 {
-	return path("templates") / tidy_for_filename (name);
+	return write_path("templates") / tidy_for_filename (name);
 }
 
+
 void
 Config::rename_template (string old_name, string new_name) const
 {
-	boost::filesystem::rename (template_path (old_name), template_path (new_name));
+	boost::filesystem::rename (template_read_path(old_name), template_write_path(new_name));
 }
 
 void
 Config::delete_template (string name) const
 {
-	boost::filesystem::remove (template_path (name));
+	boost::filesystem::remove (template_write_path(name));
 }
 
 /** @return Path to the config.xml containing the actual settings, following a link if required */
 boost::filesystem::path
-Config::config_file ()
+config_file (boost::filesystem::path main)
 {
 	cxml::Document f ("Config");
-	auto main = path("config.xml", false);
 	if (!boost::filesystem::exists (main)) {
 		/* It doesn't exist, so there can't be any links; just return it */
 		return main;
@@ -1325,6 +1354,21 @@ Config::config_file ()
 	return main;
 }
 
+
+boost::filesystem::path
+Config::config_read_file ()
+{
+	return config_file (read_path("config.xml"));
+}
+
+
+boost::filesystem::path
+Config::config_write_file ()
+{
+	return config_file (write_path("config.xml"));
+}
+
+
 void
 Config::reset_cover_sheet ()
 {
@@ -1338,11 +1382,11 @@ Config::link (boost::filesystem::path new_file) const
 	xmlpp::Document doc;
 	doc.create_root_node("Config")->add_child("Link")->add_child_text(new_file.string());
 	try {
-		doc.write_to_file_formatted(path("config.xml", true).string());
+		doc.write_to_file_formatted(write_path("config.xml").string());
 	} catch (xmlpp::exception& e) {
 		string s = e.what ();
 		trim (s);
-		throw FileError (s, path("config.xml"));
+		throw FileError (s, write_path("config.xml"));
 	}
 }
 
@@ -1350,14 +1394,14 @@ void
 Config::copy_and_link (boost::filesystem::path new_file) const
 {
 	write ();
-	boost::filesystem::copy_file (config_file(), new_file, boost::filesystem::copy_option::overwrite_if_exists);
+	boost::filesystem::copy_file (config_read_file(), new_file, boost::filesystem::copy_option::overwrite_if_exists);
 	link (new_file);
 }
 
 bool
 Config::have_write_permission () const
 {
-	auto f = fopen_boost (config_file(), "r+");
+	auto f = fopen_boost (config_write_file(), "r+");
 	if (!f) {
 		return false;
 	}