Merge branch 'features/i18n-spatial-y-add-dataset' into development

Reactiva los overrides de i18n que pybabel había marcado obsoletos: el
modal de búsqueda espacial de ckanext-spatial vuelve a verse en español y
el botón de /dataset dice "Agregar dataset" en vez de "Agregar conjunto
de datos".

Author: AlenCarbajal

babel.cfg

@@ -1,2 +1,5 @@
 [python: **.py]
 [ckan: **.html]
+# Los patrones se evalúan relativos al input dir (gobar_theme/...), no a la
+# raíz del repo. Solo assets/js: public/vendor/ tiene el Poncho minificado.
+[javascript: **/assets/js/*.js]

ckanext/gobar_theme/assets/js/i18n-overrides.js

@@ -0,0 +1,22 @@
+/**
+ * Anclas de extracción de i18n para strings que emite el JavaScript de OTRAS
+ * extensiones. Este archivo no se ejecuta ni entra en ningún bundle de
+ * webassets: existe solo para `pybabel extract`.
+ *
+ * CKAN arma el catálogo de traducciones de JS (ckan.lib.i18n.
+ * build_js_translations) tomando de cada .pot únicamente las entradas cuya
+ * ocurrencia termina en ".js". Como estos msgid viven en el código de
+ * ckanext-spatial y no en el nuestro, sin este archivo nunca llegarían al
+ * .pot y su traducción no se serviría al navegador: por eso el modal del
+ * mapa se veía en inglés.
+ *
+ * En _build_js_translation los catálogos de los plugins se procesan después
+ * del de core, así que lo que se ponga acá pisa la traducción de core.
+ */
+(function () {
+  // ckanext-spatial — ckanext/spatial/assets/js/spatial_query.js
+  _('Please draw query extent in the map:');
+  _('Apply');
+  _('Cancel');
+  _('Draw an extent');
+})();

ckanext/gobar_theme/i18n/README_i18n.md

@@ -32,3 +32,35 @@ los mismos `msgid`. Sirve para sobrescribir, por ejemplo, "Datasets"
 
 > Sin `.mo` compilado, el catálogo no se activa (no rompe nada): se sigue
 > mostrando el texto de CKAN core.
+
+## Strings que emite el JavaScript de otras extensiones
+
+CKAN sirve las traducciones de JS por `/api/i18n/<lang>`, y arma ese catálogo
+en `ckan.lib.i18n.build_js_translations()` con una regla propia: de cada
+`.pot` toma **solo las entradas cuya ocurrencia (`#:`) termina en `.js`**, y
+descarta las obsoletas. Los catálogos de los plugins se procesan después del
+de core, así que lo nuestro pisa lo de core.
+
+Consecuencia: para sobrescribir un string que emite el JS de otra extensión
+(por ejemplo el modal de búsqueda espacial de `ckanext-spatial`) **no alcanza
+con ponerlo en la sección de overrides manuales del `.po`** — esas entradas
+no tienen `#:` y nunca llegarían al catálogo JS.
+
+Por eso existe `ckanext/gobar_theme/assets/js/i18n-overrides.js`: un archivo
+que no se ejecuta ni entra en ningún bundle, y cuya única función es darle a
+esos msgid un call-site `.js` real para que `pybabel extract` los deje en el
+`.pot`. Como efecto secundario resuelve el HAZARD documentado en los `.po`:
+al tener call-site, `pybabel update` ya no los re-obsoleta.
+
+Para agregar uno nuevo:
+
+1. Sumar `_('El string exacto en inglés');` en `assets/js/i18n-overrides.js`.
+2. `python setup.py extract_messages` para regenerar el `.pot`.
+3. Agregar el `msgid`/`msgstr` en cada `<locale>/LC_MESSAGES/*.po`, con el
+   comentario `#:` que quedó en el `.pot`.
+4. Compilar (paso 2 de arriba) y reiniciar.
+
+> `babel.cfg` tiene el mapping `[javascript: **/assets/js/*.js]`. Los patrones
+> se evalúan **relativos al input dir** (`gobar_theme/...`), no a la raíz del
+> repo — un patrón que arranque con `ckanext/` no matchea nada. Está acotado a
+> `assets/js` a propósito: `public/vendor/` tiene el Poncho minificado.