Solucionar bug al cambiar el retraso con la ejecución pausada

[kishtarn555]

FIX #120

Cree funciones llamadas stopAutoStep y startAutoStep. que controlan el intervalo de la ejecución de Karel.

El motivo de este problema era que en la línea 857, se limpiaba el intervalo, pero este no era establecido a null.

Esto ocasiona que cuando cambiaras el retraso se active un nuevo intervalo, por ejemplo desde la línea 939.

Ahora, cuando le das el botón de reproducir otra vez, se crea otro intervalo y el intervalo creado por el cambio del retraso queda flotando sin ninguna referencia, ahora es imposible detenerlo.

Para arreglar esto, bastaba agregar en la abajo de la línea 939 intervalo = null pero agregue las funciones para:

• Si se cambia el comportamiento del autoplay, hay varios lugares que editar.
• startAutoDelay se asegura de siempre quitar cualquier intervalo previo.
• evitar este bug en el futuro.

[lhchavez]

muchas gracias!

[kishtarn555]

Pregunta, cuando se hacen estos cambios publicos? (cuando se liberan en www.omegaup.com/karel.js/# ? (Creo que es necesario sacar una release)

[lhchavez]

ah hay que actualizar la versión del submódulo en https://github.com/omegaup/omegaup

[kishtarn555]

No pude, lo intente. Pero al montar la VM, me sale un error.

Segun yo lo que hay que hacer es nadamas 1) clonar el repo. 2) Correr git submodule update frontend/www/karel.js 3) Correr los tests 4) PR.

Pero me trabe en el paso 1. Segui los pasos de https://github.com/omegaup/omegaup/wiki/Instalaci%C3%B3n-de-m%C3%A1quina-virtual