Crowdin API v2, allow editing of any files as XLIFF

src/catalog.cpp

@@ -600,6 +600,24 @@ void Catalog::SetFileName(const wxString& fn)
     m_fileName = f.GetFullPath();
 }
 
+bool Catalog::IsFromCrowdin() const
+{
+    if (m_crowdinFileId > 0 && m_crowdinProjectId > 0)
+        return true;
+
+    if (m_header.HasHeader("X-Crowdin-Project") && m_header.HasHeader("X-Crowdin-File"))
+        return true;
+
+    auto name = wxFileName(m_fileName).GetName();
+    if(name.StartsWith("CrowdinSync_"))
+    {
+        name = name.AfterFirst('_');
+        name.ToLong(&m_crowdinProjectId);
+        name.AfterFirst('_').ToLong(&m_crowdinFileId);
+    }
+
+    return m_crowdinFileId > 0 && m_crowdinProjectId > 0;
+}
 
 namespace
 {

src/catalog.h

@@ -552,9 +552,11 @@ class Catalog
         virtual void SetLanguage(Language lang);
 
         /// Is the PO file from Crowdin, i.e. sync-able?
-        bool IsFromCrowdin() const
-            { return m_header.HasHeader("X-Crowdin-Project") && m_header.HasHeader("X-Crowdin-File"); }
-
+        bool IsFromCrowdin() const;
+        long GetCrowdinFileId() const { return m_crowdinFileId; }
+        long GetCrowdinProjectId() const { return m_crowdinProjectId; }
+        void SetCrowdinProjectAndFileId(int projId, int fileId) { m_crowdinProjectId  = projId; m_crowdinFileId = fileId; }
+
         /// Returns true if the catalog contains obsolete entries (~.*)
         virtual bool HasDeletedItems() const = 0;
 
@@ -594,6 +596,7 @@ class Catalog
         bool m_isOk;
         Type m_fileType;
         wxString m_fileName;
+        mutable long m_crowdinFileId = -1, m_crowdinProjectId = -1;
         HeaderData m_header;
         Language m_sourceLanguage;
 

src/catalog_po.cpp

@@ -861,7 +861,7 @@ bool POCatalog::Load(const wxString& po_file, int flags)
 
     Clear();
     m_isOk = false;
-    m_fileName = po_file;
+    SetFileName(po_file);
     m_header.BasePath = wxEmptyString;
 
     wxString ext;

src/catalog_xliff.cpp

@@ -29,6 +29,7 @@
 #include "configuration.h"
 #include "str_helpers.h"
 #include "utility.h"
+#include "crowdin_gui.h"
 
 #include <wx/intl.h>
 #include <wx/log.h>
@@ -378,7 +379,6 @@ bool XLIFFCatalog::CanLoadFile(const wxString& extension)
     return extension == "xlf" || extension == "xliff";
 }
 
-
 std::shared_ptr<XLIFFCatalog> XLIFFCatalog::Open(const wxString& filename)
 {
     constexpr auto parse_flags = parse_full | parse_ws_pcdata | parse_fragment;
@@ -403,6 +403,7 @@ std::shared_ptr<XLIFFCatalog> XLIFFCatalog::Open(const wxString& filename)
     else
         throw XLIFFReadException(filename, wxString::Format(_("unsupported XLIFF version (%s)"), xliff_version));
 
+    cat->SetFileName(filename);
     cat->Parse(xliff_root);
 
     return cat;
@@ -595,14 +596,24 @@ class XLIFF10CatalogItem : public XLIFF12CatalogItem
     }
 };
 
-
 void XLIFF1Catalog::Parse(pugi::xml_node root)
 {
     int id = 0;
+
     for (auto file: root.children("file"))
     {
-        m_sourceLanguage = Language::TryParse(file.attribute("source-language").value());
-        m_language = Language::TryParse(file.attribute("target-language").value());
+        // TODO: Only first `file` node atthributes are used for now
+        //       what works well in case of either all next are same within
+        //       same `xliff` or if the only single `file` node is there
+        //       (what is always the only case for downloaded from Crowdin)
+        //       But should be taken into account and improved for probable
+        //       more wide varietty of cases in the future.
+        if(id == 0)
+        {
+            m_sourceLanguage = Language::TryParse(file.attribute("source-language").value());
+            m_language = Language::TryParse(file.attribute("target-language").value());
+        }
+
         for (auto unit: file.select_nodes(".//trans-unit"))
         {
             auto node = unit.node();

src/cloud_sync.h

@@ -30,6 +30,8 @@
 #include "concurrency.h"
 
 #include <wx/string.h>
+#include <wx/stdpaths.h>
+#include <wx/utils.h>
 
 #if wxUSE_GUI
     #include "customcontrols.h"
@@ -62,6 +64,7 @@ class CloudSyncDestination
 
     /// Asynchronously uploads the file. Returned future throws on error.
     virtual dispatch::future<void> Upload(CatalogPtr file) = 0;
+    virtual bool Auth(wxWindow* parent) = 0;
 
 
     /// Convenicence for creating a destination from a lambda.
@@ -83,6 +86,27 @@ class CloudSyncDestination
 
         return std::make_shared<Dest>(name, std::move(func));
     }
+
+    static const wxString& GetCacheDir()
+    {
+        static wxString localBaseDir;
+
+        if(localBaseDir.empty())
+        {
+
+            #if defined(__WXOSX__)
+                localBaseDir = wxGetHomeDir() + "/Library/Caches/net.poedit.Poedit";
+            #elif defined(__UNIX__)
+                if (!wxGetEnv("XDG_CACHE_HOME", &localBaseDir))
+                    localBaseDir = wxGetHomeDir() + "/.cache";
+                localBaseDir += "/poedit";
+            #else
+                localBaseDir = wxStandardPaths::Get().GetUserDataDir() + wxFILE_SEP_PATH + "Cache";
+            #endif
+        }
+
+        return localBaseDir;
+    }
 };
 
 
@@ -116,6 +140,11 @@ class CloudSyncProgressWindow : public wxDialog
     /// Show the window while performing background sync action. Show error if 
     static void RunSync(wxWindow *parent, std::shared_ptr<CloudSyncDestination> dest, CatalogPtr file)
     {
+        if(!dest->Auth(parent))
+        {
+            return;
+        }
+
         wxWindowPtr<CloudSyncProgressWindow> progress(new CloudSyncProgressWindow(parent, dest));
 #ifdef __WXOSX__
         progress->ShowWindowModal();