OpenWith: add option to display file name in the menu title; update/improve help; refactor

OpenWith/configs/OpenWithEng.lng

@@ -2,12 +2,14 @@
 
 "OpenWith"
 "OpenWith: Choose application"
+"OpenWith for "
+" file(s)"
 
 "&OK"
 "&Cancel"
 "OpenWith Error"
 "Selected item must be a file accessible locally via a real path."
-"Failed to save configuration."
+"Failed to save modified settings; the changes will apply to the current session only."
 "No applications found. Detected MIME type:"
 "The application cannot be executed."
 "OpenWith plugin is not available on this platform."
@@ -18,6 +20,7 @@
 "Don't &wait for command completion"
 "C&lear selection if command executed"
 "Co&nfirm opening if file count exceeds:"
+"Displa&y filename in the menu title"
 
 "Use `&xdg-mime` tool"
 "Use `&file` tool"
@@ -38,9 +41,9 @@
 
 "OpenWith: Details"
 
-"&Filepath:"
-"files selected: "
-"&MIME profile:"
+"F&iles selected:"
+"&Filepaths:"
+"&MIME profiles:"
 "L&aunch command:"
 "&Close"
 "&Launch"

OpenWith/configs/OpenWithRus.lng

@@ -2,12 +2,14 @@
 
 "OpenWith"
 "OpenWith: Выберите приложение"
+"OpenWith для "
+" файл(ов)"
 
 "&ОК"
 "О&тмена"
 "Ошибка OpenWith"
 "Выбранный объект должен быть файлом, доступным локально по реальному пути."
-"Ошибка записи конфигурации."
+"Не удалось сохранить измененные настройки; они будут действовать только в текущем сеансе."
 "Приложения не найдены. Распознанный MIME-тип:"
 "Невозможно выполнить приложение."
 "Плагин OpenWith недоступен на данной платформе."
@@ -18,6 +20,7 @@
 "Не ждать &завершения команды"
 "&Снимать выделение, если выбрана команда"
 "По&дтверждать открытие, если файлов больше чем:"
+"Отображать им&я открываемого файла в заголовке меню"
 
 "Использовать утилиту `&xdg-mime`"
 "Использовать утилиту `&file`"
@@ -38,12 +41,12 @@
 
 "OpenWith: Подробности"
 
-"Адрес &файла:"
-"выбрано файлов: "
-"&MIME-профиль:"
+"В&ыбрано файлов:"
+"Адреса &файлов:"
+"&MIME-профили:"
 "&Команда запуска:"
 "&Закрыть"
-"З&апустить"
+"За&пустить"
 
 "&Desktop файл:"
 "&Источник:"

OpenWith/src/AppProvider.cpp

@@ -5,18 +5,18 @@
 
 std::unique_ptr<AppProvider> AppProvider::CreateAppProvider(TMsgGetter msg_getter)
 {
-	std::unique_ptr<AppProvider> provider = nullptr;
-#ifdef __linux__
-	provider = std::make_unique<XDGBasedAppProvider>(msg_getter);
+#if defined(__linux__) || defined(__FreeBSD__) || defined(__OpenBSD__) || defined(__NetBSD__) || defined(__DragonFly__)
+	return std::make_unique<XDGBasedAppProvider>(msg_getter);
 #elif defined(__APPLE__)
-	provider = std::make_unique<MacOSAppProvider>(msg_getter);
-#elif defined(__FreeBSD__) || defined(__OpenBSD__) || defined(__NetBSD__) || defined(__DragonFly__)
-	provider = std::make_unique<XDGBasedAppProvider>(msg_getter);
+	return std::make_unique<MacOSAppProvider>(msg_getter);
+#else
+	return nullptr;
 #endif
+}
 
-	if (provider) {
-		provider->LoadPlatformSettings();
-	}
 
-	return provider;
-}
+AppProvider* AppProvider::GetInstance(TMsgGetter msg_getter)
+{
+	static auto s_provider = CreateAppProvider(msg_getter);
+	return s_provider.get();
+}

OpenWith/src/AppProvider.hpp

@@ -10,13 +10,15 @@
 
 struct ProviderSetting
 {
-	std::wstring internal_key;
-	std::wstring display_name;
+	std::wstring internal_key;  // persistent INI key and internal identifier
+	std::wstring display_name;  // localized UI label
 	bool value;
-	bool disabled = false; // true if the setting should be grayed out in the UI
-	bool affects_candidates = true;  // true if changing this setting affects the contents or order of the candidate list
+	bool disabled;              // true if the setting should be grayed out in the UI
+	bool affects_candidates;    // true if changing this setting affects the contents or order of the candidate list
 };
 
+class KeyFileReadHelper;
+class KeyFileHelper;
 
 class AppProvider
 {
@@ -26,7 +28,7 @@ class AppProvider
 	explicit AppProvider(TMsgGetter msg_getter) : m_GetMsg(std::move(msg_getter)) {}
 	virtual ~AppProvider() = default;
 
-	static std::unique_ptr<AppProvider> CreateAppProvider(TMsgGetter msg_getter);
+	static AppProvider* GetInstance(TMsgGetter msg_getter = nullptr);
 
 	virtual std::vector<CandidateInfo> GetAppCandidates(const std::vector<std::wstring>& filepaths) = 0;
 	virtual std::vector<std::wstring> GetMimeTypes() = 0;
@@ -35,9 +37,13 @@ class AppProvider
 
 	virtual std::vector<ProviderSetting> GetPlatformSettings() { return {}; }
 	virtual void SetPlatformSettings(const std::vector<ProviderSetting>& settings) { }
-	virtual void LoadPlatformSettings() { }
-	virtual void SavePlatformSettings() { }
+	virtual void LoadPlatformSettings(const KeyFileReadHelper& key_reader) { }
+	virtual void SavePlatformSettings(KeyFileHelper& key_writer) { }
+
 
 protected:
 	TMsgGetter m_GetMsg;
+
+private:
+	static std::unique_ptr<AppProvider> CreateAppProvider(TMsgGetter msg_getter);
 };