Fixed file format check, add help button to server list popup

zenden2k · zenden2k · commit 4614276df4ed · 2025-08-10T14:10:04.000+03:00
* Fixed missing GUID argument in call to InstallIcon in FloatingWindow
* Add  "emf" and "wmf" to supported image format list
diff --git a/Docs/Pics/server_list.png b/Docs/Pics/server_list.png
diff --git a/Docs/index.html b/Docs/index.html
@@ -1,6 +1,6 @@
 <html>
 <head>
-<meta http-equiv="refresh" content="0; url=ru_RU/index.html">
+<meta http-equiv="refresh" content="0; url=ru/index.html">
 </head>
 <body>
 </body>
diff --git a/Docs/ru/advanced.html b/Docs/ru/advanced.html
diff --git a/Docs/ru/index.html b/Docs/ru/index.html
@@ -34,7 +34,7 @@ <h1>Uptooda</h1> <div id="version"></div>
 <li><strong>Загрузка изображений и файлов в интернет</strong> <br />Программа позволяет загружать изображения и файлы на множество сайтов &nbsp;(<a href="https://svistunov.dev/ru/imageuploader_servers" target="_blank">список</a>). 
     Среди них такие сайты, как fastpic.ru, flickr.com, Google Drive, Yandex.Disk. Также можно загружать на FTP, SFTP или WebDav сервер.<br /><br />
     При этом программа автоматически генерирует миниатюры желаемого размера, масштабирует изображения, по желанию может добавить надпись или логотип на изображение. 
-    Поддерживаются форматы изображений JPG, PNG, BMP, GIF, TIFF, WEBP.<br />После загрузки программа выдает список ссылок или коды HTML, BBCode и Markdown (для форумов, блогов и т.п.). 
+    Поддерживаются форматы изображений JPG, PNG, GIF, WEBP, а также только на чтение: BMP, TIFF, HEIF, HEIC, AVIF, WMF, EMF. <br />После загрузки программа выдает список ссылок или коды HTML, BBCode и Markdown (для форумов, блогов и т.п.). 
     Ссылки могут быть автоматически скопированы в буфер обмена.
 </li>
 <li><strong>Извлечение кадров из видео-файлов</strong><br /> Программа умеет извлекать заданное количество кадров из видеофайлов большинства современных форматов. Uptooda может использовать как кодеки, установленные в вашей системе, так и кодеки, распространяемые вместе с программой.
diff --git a/Docs/ru/server.html b/Docs/ru/server.html
diff --git a/Docs/ru/usage.html b/Docs/ru/usage.html
@@ -87,6 +87,19 @@ <h3>Выбор области экрана с помощью мыши (стар
 Чтобы поменять цвет линии, нажмите среднюю клавишу мыши. Чтобы уменьшить или увеличить ширину линии, нажимайте клавиши "[" и "]".
 </div>
 
+<h2 id="serverlist">Окно выбора сервера</h2>
+<div class="image"><img src="../Pics/server_list.png" title="Окно со списком серверов" alt="Окно со списком серверов"/></div>
+<p>
+В нём отображается таблица со списком серверов (хостингов) с возможностью фильтрации по типу и по названию. 
+В таблице также отображается максимально допустимый размер файла, время хранения файла в днях, список поддерживаемых форматов и поддерживается ли учетная запись на данном сервере.
+Значения, перечисленные через слэш "/", относятся к разным видам учетных записей. Первое значение относится к анонимному пользователю,
+второе к зарегистрированному. Третье значение может относиться, например, к premium-аккаунту (это зависит от условий самого хостинга). 
+</p>
+<b>Время хранения</b>
+<p>
+Знак "∞" означает неограниченное время хранения. Число дней, рядом с которым стоит знак "⤓", означает, какое количество дней должно пройти после последнего скачивания (просмотра) файла, 
+чтобы файл был удалён с сервера.
+</p>
 </div>
 <div id="footer">
 
diff --git a/Source/Core/FileTypeCheckTask.cpp b/Source/Core/FileTypeCheckTask.cpp
@@ -10,12 +10,11 @@
 
 constexpr auto MAX_BAD_ITEMS = std::numeric_limits<size_t>::max();
 
-FileTypeCheckTask::FileTypeCheckTask(IFileList* fileList, const ServerProfileGroup& sessionImageServer, const ServerProfileGroup& sessionFileServer)
-    :
-    fileList_(fileList),
-    sessionImageServer_(sessionImageServer),
-    sessionFileServer_(sessionFileServer)
-{
+FileTypeCheckTask::FileTypeCheckTask(IFileList* fileList, const ServerProfileGroup& sessionImageServer, const ServerProfileGroup& sessionFileServer, ImageUploadParams defaultImageUploadParams) :
+    fileList_(fileList)
+    , sessionImageServer_(sessionImageServer)
+    , sessionFileServer_(sessionFileServer)
+    , defaultImageUploadParams_(std::move(defaultImageUploadParams)) {
 
 }
 
@@ -51,7 +50,7 @@ BackgroundTaskResult FileTypeCheckTask::doJob()
             
             
             if (item->isImage()) {
-                ImageConverterFilter::supposedOutputFormat(sf, serverProfile);
+                ImageConverterFilter::supposedOutputFormat(sf, serverProfile, defaultImageUploadParams_);
             }
             if (sf.fileSize < 0) {
                 sf.fileSize = size;
diff --git a/Source/Core/FileTypeCheckTask.h b/Source/Core/FileTypeCheckTask.h
@@ -15,7 +15,7 @@ struct BadFileFormat {
 
 class FileTypeCheckTask: public BackgroundTask {
 public:
-    explicit FileTypeCheckTask(IFileList* fileList, const ServerProfileGroup& sessionImageServer, const ServerProfileGroup& sessionFileServer);
+    explicit FileTypeCheckTask(IFileList* fileList, const ServerProfileGroup& sessionImageServer, const ServerProfileGroup& sessionFileServer, ImageUploadParams defaultImageUploadParams);
 	
 	BackgroundTaskResult doJob() override;
 
@@ -30,4 +30,5 @@ class FileTypeCheckTask: public BackgroundTask {
         ServerProfileGroup sessionFileServer_;
         std::string message_;
         std::vector<BadFileFormat> errors_;
+        ImageUploadParams defaultImageUploadParams_;
 };
diff --git a/Source/Core/Settings/CommonGuiSettings.cpp b/Source/Core/Settings/CommonGuiSettings.cpp
@@ -24,12 +24,11 @@ limitations under the License.
 #include "Func/WinUtils.h"
 #endif
 
-CommonGuiSettings::CommonGuiSettings()
-    : BasicSettings()
-    , imageServer(false) {
+CommonGuiSettings::CommonGuiSettings() : BasicSettings() {
     // Default values of settings
     MaxThreads = 3;
     DeveloperMode = false;
+
 #ifndef IU_QT
     HistorySettings.EnableDownloading = true;
     HistorySettings.HistoryConverted = false;
@@ -188,6 +187,7 @@ void CommonGuiSettings::BindToManager() {
     urlShorteningServer.bind(upload["UrlShorteningServer"]);
     temporaryServer.bind(upload["TemporaryServer"]);
     imageSearchServer.bind(upload["ImageSearchServer"]);
+    DefaultImageUploadParams.bind(upload["DefaultImageUploadParams"]);
 
     SettingsNode& video = mgr_["VideoGrabber"];
     video.nm_bind(VideoSettings, NumOfFrames);
diff --git a/Source/Core/Settings/CommonGuiSettings.h b/Source/Core/Settings/CommonGuiSettings.h
@@ -101,8 +101,8 @@ struct ScreenRecordingStruct {
 
 #ifndef IU_QT
 struct MediaInfoSettingsStruct {
-    int InfoType; // 0 - short summary, 1 - full info
-    bool EnableLocalization;
+    int InfoType = 0; // 0 - short summary, 1 - full info
+    bool EnableLocalization = true;
 };
 
 struct HistorySettingsStruct {
@@ -123,6 +123,7 @@ class CommonGuiSettings : public BasicSettings {
         ServerProfilesMap ServerProfiles;
         VideoSettingsStruct VideoSettings;
         ScreenRecordingStruct ScreenRecordingSettings;
+        ImageUploadParams DefaultImageUploadParams;
 #ifndef IU_QT
         CString Language;
         CString DataFolder;
@@ -136,8 +137,8 @@ class CommonGuiSettings : public BasicSettings {
         MediaInfoSettingsStruct MediaInfoSettings;
         HistorySettingsStruct HistorySettings;
 
-        int CodeLang;
-        int CodeType;
+        int CodeLang = 0;
+        int CodeType = 0;
 
         bool IsPortable = true;
         CString VideoFolder, ImagesFolder;
diff --git a/Source/Core/Settings/WtlGuiSettings.cpp b/Source/Core/Settings/WtlGuiSettings.cpp
@@ -467,7 +467,6 @@ WtlGuiSettings::WtlGuiSettings() :
     UseDirectLinks = true;
     TrayResult = trJustURL;
     DropVideoFilesToTheList = false;
-    CodeLang = 0;
     ConfirmOnExit = 1;
     EnableToastNotifications = true;
 
diff --git a/Source/Core/Upload/Filters/ImageConverterFilter.cpp b/Source/Core/Upload/Filters/ImageConverterFilter.cpp
@@ -90,10 +90,10 @@ bool ImageConverterFilter::PostUpload(UploadTask* task)
     return true;
 }
 
-bool ImageConverterFilter::supposedOutputFormat(SupposedFormat& fileFormat, ServerProfile serverProfile) {
+bool ImageConverterFilter::supposedOutputFormat(SupposedFormat& fileFormat, ServerProfile serverProfile, const ImageUploadParams& defaultImageUploadParams) {
     auto* settings = ServiceLocator::instance()->settings<WtlGuiSettings>();
 
-    ImageUploadParams imageUploadParams = serverProfile.getImageUploadParams();
+    ImageUploadParams imageUploadParams = serverProfile.UseDefaultSettings ? defaultImageUploadParams : serverProfile.getImageUploadParams();
     if (!imageUploadParams.ProcessImages) {
         return false;
     }
diff --git a/Source/Core/Upload/Filters/ImageConverterFilter.h b/Source/Core/Upload/Filters/ImageConverterFilter.h
@@ -11,6 +11,6 @@ class ImageConverterFilter : public UploadFilter
     bool PreUpload(UploadTask* task) override;
     bool PostUpload(UploadTask* task) override;
 
-    static bool supposedOutputFormat(SupposedFormat& fileFormat, ServerProfile serverProfile);
+    static bool supposedOutputFormat(SupposedFormat& fileFormat, ServerProfile serverProfile, const ImageUploadParams& defaultImageUploadParams);
 };
 #endif
diff --git a/Source/Core/Upload/ServerProfile.cpp b/Source/Core/Upload/ServerProfile.cpp
@@ -7,8 +7,7 @@
 #include "Core/Settings/CommonGuiSettings.h"
 #endif
 
-ServerProfile::ServerProfile(bool useDefaultSettings) {
-    UseDefaultSettings = useDefaultSettings;
+ServerProfile::ServerProfile() {
     shortenLinks_ = false;
 }
 
@@ -128,9 +127,9 @@ void ServerProfile::bind(SettingsNode& serverNode)
 ImageUploadParams ServerProfile::getImageUploadParams()
 {
 #ifdef _WIN32
-    CommonGuiSettings* Settings = ServiceLocator::instance()->settings<CommonGuiSettings>();
-    if (UseDefaultSettings && Settings && !Settings->imageServer.isEmpty() && &Settings->imageServer.getByIndex(0) != this) {
-        return Settings->imageServer.getByIndex(0).imageUploadParams;
+    auto* settings = ServiceLocator::instance()->settings<CommonGuiSettings>();
+    if (UseDefaultSettings && settings) {
+        return settings->DefaultImageUploadParams;
     }
 #endif
     return imageUploadParams;
diff --git a/Source/Core/Upload/ServerProfile.h b/Source/Core/Upload/ServerProfile.h
@@ -46,7 +46,7 @@ struct ImageUploadParams {
 class ServerProfile {
 
 public:
-    explicit ServerProfile(bool useDefaultSettings = true);
+    explicit ServerProfile();
     explicit ServerProfile(const std::string& serverName);
 
     CUploadEngineData* uploadEngineData() const;
diff --git a/Source/Core/Upload/ServerProfileGroup.cpp b/Source/Core/Upload/ServerProfileGroup.cpp
@@ -1,25 +1,15 @@
 #include "ServerProfileGroup.h"
 
 ServerProfileGroup::ServerProfileGroup() {
-    canFirstUseDefaultSettings_ = true;
 }
 
 ServerProfileGroup::ServerProfileGroup(const ServerProfile& profile) {
-    canFirstUseDefaultSettings_ = true;
-    addItem(profile);
-}
-
 
-ServerProfileGroup::ServerProfileGroup(bool canFirstUseDefaultSettings) {
-    canFirstUseDefaultSettings_ = canFirstUseDefaultSettings;
+    addItem(profile);
 }
 
 void ServerProfileGroup::addItem(const ServerProfile& profile) {
-    ServerProfile p = profile;
-    if (items_.empty() && !canFirstUseDefaultSettings_) {
-        p.UseDefaultSettings = false;
-    }
-    items_.push_back(p);
+    items_.push_back(profile);
 }
 
 std::vector<ServerProfile>& ServerProfileGroup::getItems() {
@@ -36,7 +26,7 @@ bool ServerProfileGroup::isEmpty() const {
 
 ServerProfile& ServerProfileGroup::getByIndex(size_t index) {
     if (items_.empty()) {
-        addItem(ServerProfile(canFirstUseDefaultSettings_));
+        addItem(ServerProfile {});
     }
     return items_.at(index);
 }
diff --git a/Source/Core/Upload/ServerProfileGroup.h b/Source/Core/Upload/ServerProfileGroup.h
@@ -8,7 +8,6 @@
 class ServerProfileGroup {
 public:
     ServerProfileGroup();
-    explicit ServerProfileGroup(bool canFirstUseDefaultSettings);
     ServerProfileGroup(const ServerProfile& profile);
     void addItem(const ServerProfile& profile);
 
@@ -24,6 +23,5 @@ class ServerProfileGroup {
     }*/
 private:
     std::vector<ServerProfile> items_;
-    bool canFirstUseDefaultSettings_;
 };
 #endif
diff --git a/Source/Func/Common.cpp b/Source/Func/Common.cpp
@@ -282,3 +282,35 @@ CString HotkeyToString(CString funcName, CString menuItemText) {
     }
     return menuItemText + _T("\t") + hotkeyStr;
 }
+
+void OpenDocumentation(HWND parent, const CString& file, const CString& id /*= {}*/) {
+    CString docsFolder = WinUtils::GetAppFolder() + "Docs\\";
+    CString actualFile;
+
+    if (!file.IsEmpty()) {
+        CString locale = U2W(ServiceLocator::instance()->translator()->getCurrentLocale());
+        CString tryFile = docsFolder + locale + _T("\\") + file + _T(".html");
+        CString tryFile2 = docsFolder + _T("en_US\\") + file + _T(".html");
+
+        if (WinUtils::FileExists(tryFile)) {
+            actualFile = tryFile;
+        } else if (WinUtils::FileExists(tryFile2)) {
+            actualFile = tryFile2;
+        } 
+
+        /*if (!actualFile.IsEmpty()) {
+            actualFile += _T("#");
+            actualFile += id;
+        }*/
+   }
+
+    if (file.IsEmpty()) {
+        actualFile = docsFolder + _T("index.html");
+    }
+
+    try {
+        WinUtils::ShellOpenFileOrUrl(actualFile, parent, {}, true);
+    } catch (const Win32Exception& ex) {
+        GuiTools::LocalizedMessageBox(parent, TR("Cannot open documentation: ") + ex.getMessage(), TR("Error"), MB_ICONERROR); 
+    }
+}
diff --git a/Source/Func/Common.h b/Source/Func/Common.h
@@ -69,5 +69,6 @@ void DecodeString(LPCTSTR szSource, CString &Result, LPSTR code = "{DAb[]=_T('')
 void EncodeString(LPCTSTR szSource, CString &Result, LPSTR code = "{DAb[]=_T('')+b/16;H3N SHJ");
 CString HotkeyToString(CString funcName, CString menuItemText);
 
+void OpenDocumentation(HWND parent, const CString& file, const CString& id = {});
 typedef CAtlArray<CString> CStringList;
 #endif
diff --git a/Source/Func/IuCommonFunctions.cpp b/Source/Func/IuCommonFunctions.cpp
@@ -209,7 +209,7 @@ CString GenerateFileName(const CString& templateStr, int index, const CPoint& si
     return result;
 }
 
-const std::unordered_set<std::string> supportedImageExtensions = { "jpg", "jpeg", "jpe", "jif", "jfif", "png", "bmp", "gif","tif", "tiff", "webp", "heic", "heif", "avif"};
+const std::unordered_set<std::string> supportedImageExtensions = { "jpg", "jpeg", "jpe", "jif", "jfif", "png", "bmp", "gif", "tif", "tiff", "webp", "heic", "heif", "avif", "emf", "wmf"};
 
 bool IsImage(LPCTSTR szFileName)
 {
diff --git a/Source/GUI.rc b/Source/GUI.rc
@@ -163,6 +163,7 @@ BEGIN
     CONTROL         "",IDC_SERVERLISTCONTROL,"SysListView32",LVS_REPORT | LVS_SHOWSELALWAYS | LVS_ALIGNLEFT | LVS_OWNERDATA | WS_BORDER | WS_TABSTOP,6,15,461,206
     EDITTEXT        IDC_SEARCHQUERYEDIT,6,224,129,14,ES_AUTOHSCROLL
     PUSHBUTTON      "#add",IDC_ADDBUTTON,363,224,96,14
+    PUSHBUTTON      "?",IDC_HELPBUTTON,451,-1,17,14
 END
 
 IDD_SERVERSELECTORCONTROL DIALOGEX 0, 0, 260, 46
diff --git a/Source/Gui/Controls/ServerListView.cpp b/Source/Gui/Controls/ServerListView.cpp
@@ -123,8 +123,8 @@ void CServerListView::setColumnWidths() {
     int dpi = DPIHelper::GetDpiForDialog(m_hWnd);
     SetColumnWidth(ServerListModel::tcServerName, MulDiv(140, dpi, USER_DEFAULT_SCREEN_DPI));
     SetColumnWidth(ServerListModel::tcMaxFileSize, MulDiv(115, dpi, USER_DEFAULT_SCREEN_DPI));
-    SetColumnWidth(ServerListModel::tcStorageTime, MulDiv(100, dpi, USER_DEFAULT_SCREEN_DPI));
-    SetColumnWidth(ServerListModel::tcAccount, MulDiv(50, dpi, USER_DEFAULT_SCREEN_DPI));
+    SetColumnWidth(ServerListModel::tcStorageTime, MulDiv(110, dpi, USER_DEFAULT_SCREEN_DPI));
+    SetColumnWidth(ServerListModel::tcAccount, MulDiv(60, dpi, USER_DEFAULT_SCREEN_DPI));
     SetColumnWidth(ServerListModel::tcFileFormats, MulDiv(200, dpi, USER_DEFAULT_SCREEN_DPI));
 }
 
diff --git a/Source/Gui/Dialogs/FloatingWindow.cpp b/Source/Gui/Dialogs/FloatingWindow.cpp
@@ -144,19 +144,19 @@ LRESULT CFloatingWindow::OnCreate(LPCREATESTRUCT lpCreateStruct)
 
     RegisterHotkeys();
 
-    auto trayIconGUID = WinUtils::GenerateFakeUUIDv4(MainTrayIconBaseGUID);
+    trayIconGuid_ = WinUtils::GenerateFakeUUIDv4(MainTrayIconBaseGUID);
     //LOG(ERROR) << WinUtils::GUIDToString(*trayIconGUID);
-    if (!InstallIcon(APP_NAME, m_hIconSmall, NULL, trayIconGUID ? &trayIconGUID.value() : nullptr)) {
+    if (!InstallIcon(APP_NAME, m_hIconSmall, NULL, trayIconGuid_ ? &trayIconGuid_.value() : nullptr)) {
         LOG(WARNING) << "Failed to create tray icon!";
     }
     NOTIFYICONDATA nid;
     ZeroMemory(&nid, sizeof(nid));
     nid.cbSize = NOTIFYICONDATA_V2_SIZE;
     nid.hWnd = m_hWnd;
     nid.uVersion = NOTIFYICON_VERSION;
-    if (trayIconGUID) {
+    if (trayIconGuid_) {
         nid.uFlags = NIF_GUID;
-        nid.guidItem = *trayIconGUID;
+        nid.guidItem = *trayIconGuid_;
     }
     Shell_NotifyIcon(NIM_SETVERSION, &nid);
 
@@ -808,7 +808,7 @@ LRESULT CFloatingWindow::OnMediaInfo(WORD wNotifyCode, WORD wID, HWND hWndCtl)
 
 LRESULT CFloatingWindow::OnTaskbarCreated(UINT uMsg, WPARAM wParam, LPARAM lParam)
 {
-    InstallIcon(APP_NAME, m_hIconSmall, 0);
+    InstallIcon(APP_NAME, m_hIconSmall, 0, trayIconGuid_ ? &trayIconGuid_.value() : nullptr);
     return 0;
 }
 
diff --git a/Source/Gui/Dialogs/FloatingWindow.h b/Source/Gui/Dialogs/FloatingWindow.h
@@ -259,6 +259,7 @@ class CFloatingWindow :
     protected:
     std::unique_ptr<IconBitmapUtils> iconBitmapUtils_;
     CBitmap addFilesBitmap_, screenshotBitmap_, regionScreenshotBitmap_, screenRecordingBitmap_, startRecordingBitmap_, settingsBitmap_;
+    std::optional<GUID> trayIconGuid_;
 };
 
 #endif
diff --git a/Source/Gui/Dialogs/FolderAddDlg.cpp b/Source/Gui/Dialogs/FolderAddDlg.cpp
@@ -119,7 +119,7 @@ DWORD CFolderAdd::Run()
         GuiTools::LocalizedMessageBox(m_pWizardDlg->m_hWnd, m_bImagesOnly ? TR("No pictures were found.") : TR("No files were found."), APP_NAME, MB_ICONINFORMATION);
     else {
         if (m_pWizardDlg->getQuickUploadMarker()) {
-            if (settings->CheckFileTypesBeforeUpload && !m_pWizardDlg->checkFileFormats(m_pWizardDlg->getSessionImageServer(), m_pWizardDlg->getSessionFileServer())) {
+            if (settings->CheckFileTypesBeforeUpload && !m_pWizardDlg->checkFileFormats(m_pWizardDlg->getSessionImageServer(), m_pWizardDlg->getSessionFileServer(), settings->DefaultImageUploadParams)) {
                 m_pWizardDlg->ShowPage(CWizardDlg::wpUploadSettingsPage);
             } else {
                 m_pWizardDlg->ShowPage(CWizardDlg::wpUploadPage);
diff --git a/Source/Gui/Dialogs/ServerListPopup.cpp b/Source/Gui/Dialogs/ServerListPopup.cpp
diff --git a/Source/Gui/Dialogs/ServerListPopup.h b/Source/Gui/Dialogs/ServerListPopup.h
diff --git a/Source/Gui/Dialogs/UploadParamsDlg.cpp b/Source/Gui/Dialogs/UploadParamsDlg.cpp
diff --git a/Source/Gui/Dialogs/UploadSettings.cpp b/Source/Gui/Dialogs/UploadSettings.cpp
diff --git a/Source/Gui/Dialogs/WizardDlg.cpp b/Source/Gui/Dialogs/WizardDlg.cpp
diff --git a/Source/Gui/Dialogs/WizardDlg.h b/Source/Gui/Dialogs/WizardDlg.h

-Original file line number
+Diff line change
@@ @@ -1,6 +1,6 @@ @@
 <html>
 <head>
 -<meta http-equiv="refresh" content="0; url=ru_RU/index.html">
 +<meta http-equiv="refresh" content="0; url=ru/index.html">
 </head>
 <body>
 </body>