Move translations source path to config

Author: goanpeca

.gitignore

@@ -146,5 +146,6 @@ doc/source/savefig/
 # Web & Translations #
 ##############################
 web/translations/
-web/pandas/pt/
 web/pandas/es/
+web/pandas/pt/
+web/pandas/fr/

web/pandas/config.yml

@@ -207,6 +207,7 @@ roadmap:
 translations:
   url: https://github.com/Scientific-Python-Translations/pandas-translations/archive/refs/heads/main.tar.gz
   folder: translations
+  source_path: pandas-translations-main/web/pandas/
   default_language: 'en'
   ignore:
     - docs/

web/pandas/static/js/language_switcher.js

@@ -1,9 +1,11 @@
 window.addEventListener("DOMContentLoaded", function() {
-  var BASE_URL = location.protocol + "//" + location.hostname + ":" + location.port
-  var CURRENT_LANGUAGE = document.documentElement.lang;
-  var PATHNAME = location.pathname.replace('/' + CURRENT_LANGUAGE + '/', '')
+  var baseUrl = location.protocol + "//" + location.hostname
+  if (location.port) {
+     baseUrl = baseUrl + ":" + location.port
+  }
+  var currentLanguage = document.documentElement.lang;
   var languages = JSON.parse(document.getElementById("languages").getAttribute('data-lang').replace(/'/g, '"'));
-  const language_names = {
+  const languageNames = {
         'en': 'English',
         'es': 'Español',
         'fr': 'Français',
@@ -24,24 +26,23 @@ window.addEventListener("DOMContentLoaded", function() {
     link.setAttribute("role", "button");
     link.setAttribute("aria-haspopup", "true");
     link.setAttribute("aria-expanded", "false");
-    link.textContent = language_names[CURRENT_LANGUAGE];
+    link.textContent = languageNames[currentLanguage];
 
     var dropdownMenu = document.createElement("div");
     dropdownMenu.classList.add("dropdown-menu");
 
     options.forEach(function(i) {
       var dropdownItem = document.createElement("a");
       dropdownItem.classList.add("dropdown-item");
-      dropdownItem.textContent = language_names[i] || i.toUpperCase();
+      dropdownItem.textContent = languageNames[i] || i.toUpperCase();
       dropdownItem.setAttribute("href", "#");
       dropdownItem.addEventListener("click", function() {
-        if (i == 'en') {
-          URL_LANGUAGE = '';
-        } else {
-          URL_LANGUAGE = '/' + i;
+        var urlLanguage = '';
+        if (i !== 'en') {
+          urlLanguage = '/' + i;
         }
-        var PATHNAME = location.pathname.replace('/' + CURRENT_LANGUAGE + '/', '/')
-        var newUrl = BASE_URL + URL_LANGUAGE + PATHNAME
+        var pathName = location.pathname.replace('/' + currentLanguage + '/', '/')
+        var newUrl = baseUrl + urlLanguage + pathName
         window.location.href = newUrl;
       });
       dropdownMenu.appendChild(dropdownItem);

web/pandas_web.py

@@ -467,6 +467,7 @@ def download_and_extract_translations(url: str, dir_name: str):
     """
     Download the translations from the GitHub repository.
     """
+    shutil.rmtree(dir_name, ignore_errors=True)
     response = requests.get(url)
     if response.status_code == 200:
         doc = io.BytesIO(response.content)
@@ -480,30 +481,29 @@ def get_languages(source_path: str):
     """
     Get the list of languages available in the translations directory.
     """
-    languages_path = f"{source_path}/pandas-translations-main/web/pandas/"
-    en_path = f"{languages_path}/en/"
+    en_path = f"{source_path}/en/"
     if os.path.exists(en_path):
         shutil.rmtree(en_path)
 
-    paths = os.listdir(languages_path)
-    return [path for path in paths if os.path.isdir(f"{languages_path}/{path}")]
+    paths = os.listdir(source_path)
+    return [path for path in paths if os.path.isdir(f"{source_path}/{path}")]
 
 
 def copy_translations(source_path: str, target_path: str, languages: list[str]):
     """
     Copy the translations to the appropriate directory.
     """
-    languages_path = f"{source_path}/pandas-translations-main/web/pandas/"
     for lang in languages:
         cmds = [
             "rsync",
             "-av",
             "--delete",
-            f"{languages_path}/{lang}/",
+            f"{source_path}/{lang}/",
             f"{target_path}/{lang}/",
         ]
         p = Popen(cmds, stdout=PIPE, stderr=PIPE)
         stdout, stderr = p.communicate()
+        sys.stderr.write(f"\nCopying: {lang}...\n\n")
         sys.stderr.write(stdout.decode())
         sys.stderr.write(stderr.decode())
 
@@ -526,15 +526,22 @@ def main(
     config = get_config(base_config_fname)
     translations_path = os.path.join(base_folder, f"{config['translations']['folder']}")
 
-    sys.stderr.write("Downloading and extracting translations...\n")
-    download_and_extract_translations(config["translations"]["url"], translations_path)
+    sys.stderr.write("\nDownloading and extracting translations...\n\n")
+    translations_extract_path = translations_path
+    translations_source_path = os.path.join(
+        translations_path, config["translations"]["source_path"]
+    )
+
+    download_and_extract_translations(
+        config["translations"]["url"], translations_extract_path
+    )
 
-    translated_languages = get_languages(translations_path)
+    translated_languages = get_languages(translations_source_path)
     default_language = config["translations"]["default_language"]
     languages = [default_language] + translated_languages
 
-    sys.stderr.write("Copying translations...\n")
-    copy_translations(translations_path, source_path, translated_languages)
+    sys.stderr.write("\nCopying translations...\n")
+    copy_translations(translations_source_path, source_path, translated_languages)
 
     for language in languages:
         sys.stderr.write(f"\nProcessing language: {language}...\n\n")
@@ -596,7 +603,6 @@ def main(
                 shutil.copy(
                     os.path.join(source_path, fname), os.path.join(target_path, dirname)
                 )
-    return 0
 
 
 if __name__ == "__main__":