Ojo: este sitio es un espejo (mirror). nulo.in es la fuente.

☚ Volver al inicio

Arreglando bugs ajenos

Último cambio:

2021/10/11

Estoy aprendiendo alemán y mi principal recurso es el "Aprender Alemán" de la Deutsche Welle (learngerman.dw.com). Sin embargo, en una de las recientes actualizaciones al sitio/app, rompieron varias cosas.

Les mandé un mail al equipo de desarrollo ¡pero no me contestaron! Así que lo arreglé yo mismo.

Pero ¿cómo? El código es privativo y no tengo acceso. El JavaScript que llega a mi navegador está minificado, es decir que se le elimina palabras y se comprime a un archivo para ahorrar ancho de banda. Pero otra de las razones por las que se minifica es para no dar acceso al código.

Sin embargo, el equipo de desarrollo cometió un error (¿intencionalmente?) en donde dejaron los "source maps". Los source maps son unos archivos que le permiten a las herramientas de desarrolladorx saber que parte del código minificado es cuál línea de código. ¡Por eso se llaman mapas! Para lograr esto, contienen casi todo el código del sitio.

Pero hay un último 5% del código que me hace falta para poder aplicar cambios: la configuración de las herramientas que minifican y compilan el código. Para mi suerte, este sitio utiliza una tecnología con la que ya tengo experiencia (React) así que puedo reconstruir esta configuración por mi cuenta.

Sabiendo esto, vayamos a lo práctico.

Extrayendo el código

Empiezo por abrir la ventana de herramientas de desarrolladorx de mi navegador en el sitio y extraigo los enlaces a los archivos que terminan con .map, que son los source maps. Los guardo en un archivo con un enlace por línea y los descargo utilizando un script:

for f in $(cat files); do
	wget $f
done

Ok, ahora necesito extraer los archivos de código de cada uno. Por suerte, ya existen herramientas que lo hagan por mi. Probé con este script pero mi mejor experiencia fue con este.

for f in *.map; do
	node source-from-sourcemaps.js $f
done

Levantando el código localmente

Después configuré una herramienta que compila código de React y hace otras magias por mi llamada Vite. Podría haber usado la que lxs desarrolladorxs del sitio usaron originalmente, webpack, pero en mi experiencia es muy tedioso de configurar.

Tuve varios problemas al intentar levantar un entorno de desarrollo local con el código. Acá están algunos junto a sus soluciones en ningún orden particular:

Es muy probable que haya tenido que hacer más cosas y me haya olvidado. Fue mucho prueba y error hasta que el sitio finalmente cargó.

Arreglando bug 1: problema de accesibilidad

Por un tiempo, tuve una de mis muñecas inaccesibles por unas semanas. Esto me obligo a usar exclusivamente la otra, complicando el uso del mouse y utilizando el teclado lo más posible.

Recientemente, en el sitio agregaron una solapa de "transcripción" mostrando todo el dialogo que había en el video del ejercicio. Sin embargo, cuando esta solapa estaba cerrada, se podían seguir seleccionando los enlaces dentro de la solapa con el teclado, haciendo la navegación por teclado tediosa.

La solucion es simplemente usar el elemento de solapa que ya viene con el navegador: <details>. (La página de <details> en MDN esta desactualizada al momento de escribir este artículo.)

Estos fueron los cambios que hice:

diff --git a/./components/ContentContainer/AccordionContainer.jsx.orig b/./components/ContentContainer/AccordionContainer.jsx
index ed71204..d0ea66e 100644
--- a/./components/ContentContainer/AccordionContainer.jsx.orig
+++ b/./components/ContentContainer/AccordionContainer.jsx
@@ -6,43 +6,26 @@ import { colors } from '../../utils/css';
 import { useTranslation } from '../../hooks/useTranslation';

 export const AccordionContainer = ({ title, children, className }) => {
-  const element = useRef(null);
-  const [isOpen, { toggleOnClick }] = useToggle();
-  const height = element.current ? element.current.scrollHeight : '0';
   const titleTranslation = useTranslation(title);

   return (
-    <div className={className}>
-      <div className="row noVMargins">
+    <details className={className}>
+      <summary className="row noVMargins">
         <div className="col-sm-offset-1 col-sm-10 col-lg-offset-2 col-lg-8">
-          <button tabIndex={0} onClick={toggleOnClick}>
             <h4>
               {titleTranslation}
-              <ToggleableArrow
-                className="toggleable-arrow"
-                fill={colors.LG_BLACK}
-                {...{
-                  isUp: isOpen,
-                }}
-              />
             </h4>
-          </button>
         </div>
-      </div>
+      </summary>
       <AccordionContainerContent
-        height={height}
-        isOpen={isOpen}
-        ref={element}
-        aria-expanded={isOpen}
       >
         {children}
       </AccordionContainerContent>
-    </div>
+    </details>
   );
 };

 export const AccordionContainerContent = styled.div`
-  max-height: ${({ isOpen, height }) => (isOpen ? height : '1')}px;
   overflow: hidden;
   background-color: ${colors.LG_WHITE};
   transition: max-height 0.7s;

Arreglando bug 2: no cargan algunos ejercicios

Esto tardó mucho más tiempo de arreglar y sinceramente no vale la pena explicarlo. El cambio es bastante simple, lo explico en inglés:

diff --git a/components/Lesson/LessonExercise/LessonExerciseItem/LessonExerciseItem.jsx.orig b/components/Lesson/LessonExercise/LessonExerciseItem/LessonExerciseItem.jsx
index 6d35539..66b5005 100644
--- a/components/Lesson/LessonExercise/LessonExerciseItem/LessonExerciseItem.jsx.orig
+++ b/components/Lesson/LessonExercise/LessonExerciseItem/LessonExerciseItem.jsx
@@ -91,12 +91,20 @@ export const getExerciseMediaInputComponentByType = data => {
           <MediaInputAudio data={data} />
         </div>
       );
+    // XXX: Here is the problem. These exercises return inputType == 'VIDEO'
+    // even though they don't have any videos (videos == []) causing some
+    // code that assumes that there is a video inside videos to fail
+    // (the MediaInputVideo component.)
     case 'VIDEO':
-      return (
-        <div className="input-header-video">
-          <MediaInputVideo data={data} />
-        </div>
-      );
+      // This "if" is our patch. If it isn't true, it fallsthrough to the default
+      // behaviour (returning null.)
+      if (data.content.videos.length > 0) {
+        return (
+          <div className="input-header-video">
+            <MediaInputVideo data={data} />
+          </div>
+        );
+      }
     default:
       return null;
   }

Conclusión

Más allá del desarrollo técnico que hice, creo que es más importante el político: el código desarrollado con dinero público debería ser público. Para el caso... todo el código debería ser público, pero eso es un debate para otro día.

Por otro lado, se me vienen a la mente la necesidad de testear el código para que no pasen cosas como bug 2, y también el testeo de herramientas de accesibilidad para bug 1.

Gute Nacht!