Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug al llamar a updateDesc sin los dos parámetros #5

Closed
paravaariar opened this issue Mar 8, 2023 · 3 comments
Closed

Bug al llamar a updateDesc sin los dos parámetros #5

paravaariar opened this issue Mar 8, 2023 · 3 comments

Comments

@paravaariar
Copy link
Collaborator

Este bug está relacionado con #3 pero por lo menos no era tan bloqueante.
Si llamamos a updateDesc con un solo argumento, por ejemplo un preLook() como dice la Wiki resulta que loc tiene el valor de un String... y ahí empiezan los problemas que hace que luego falle todo. El chequeo de argumentos que hay ahora no lo soluciona.

       function updateDesc(loc, desc)
        {
            ...
            if ( arguments.length < 1
              || loc == null )
            {
                loc = ctrl.places.getCurrentLoc();
            }

            if ( arguments.length < 2
              || desc == null )
            {
                desc = loc.desc;
            }
@Baltasarq
Copy link
Owner

Baltasarq commented Mar 9, 2023

¡Gracias! No se debería llamar a updateDesc() con solo un texto, eso es un error de la wiki, no del código. De hecho, me estoy temiendo que el texto que citas de la wiki explica cómo funciona, no cómo se debe usar. Quedo pendiente de que me indiques la sección de la wiki para cambiarlo.

@paravaariar
Copy link
Collaborator Author

En la wiki está en Miscelánea, en el tercer punto. El trozo de código pone ctrl.lugares.updateDesc( ctrl.lugares.getCurrentLoc().preLook() ); y así falla.

@Baltasarq
Copy link
Owner

Corregido. ¡Muchas gracias!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants