Bug en updateDesc #3
Comments
paravaariar
added a commit
to paravaariar/fi-js
that referenced
this issue
Mar 8, 2023
Esto fixes Baltasarq#3 siempre y cuando se le pasen los dos argumentos a updateDesc. Hay algo todavía mal si alguien no le pasa alguno de los dos argumentos.
This was referenced Mar 8, 2023
Merged
Baltasarq
added a commit
that referenced
this issue
Mar 9, 2023
fixes #3 second parameter to updateDesc(loc, txt) is ignored with the present code.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No estoy seguro si hay algo que estoy haciendo mal o si hay un bug...
Tengo una acción que pone algo en el área de resultado pero quiero también que modifique ligeramente el código de la descripción de la localidad. Uso el preLook para tener un if que me da bien el texto nuevo. Primero usando lo que comenta la wiki
ctrl.lugares.updateDesc( ctrl.lugares.getCurrentLoc().preLook() );me daba error cuando llegaba a
if ( loc.audio.src != null ) {y ya no se puede jugar.Pero usando
ctrl.lugares.updateDesc( ctrl.lugares.getCurrentLoc(), ctrl.lugares.getCurrentLoc().preLook() );ya no me daba el error. Sin embargo, el updateDesc llamadesc = ctrl.buildDescOfEntity( loc );que directamente hace que se reescriba el parámetrodescque le he pasado a updateDesc... total, que vuelve a poner el mismo texto que tenía al principio, y no el nuevo calculado con el preLook...The text was updated successfully, but these errors were encountered: