-
Notifications
You must be signed in to change notification settings - Fork 4
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
Sanitización nombres #91
Comments
…miento, que se impriman al código Gobstones, y que además sanitize todos los demás caracteres #91
Implementé esto: Escupe:
También agregué ese chequeo a los parámetros de las funciones y procedimientos, sucede lo mismo. Y también a la asignación, y de yapa te hice la modificación de los parámetros, @fidel-ml : Escupe:
Cerrame este issue si te parece que con esto estamos para diciembre, respecto de la sanitización.
|
Me faltó testear las variables. @rodri042 esto se integra en la 0.5.11 @fidel-ml se puede probar en https://program-ar.github.io/gs-element-blockly/ |
Un detalle que vi: las variables, parámetros y funciones deben empezar con
minúscula, como toda expresión no literal.
Por el resto, tengo que sentarme con tranquilidad a verlo...
Gracias, capo!
|
¿También en el nombre del bloque? |
Los nombres de procedimientos en Gobstones son siempre con mayúsculas...
FF
…On Nov 6, 2017 23:37, "asanzo" ***@***.***> wrote:
¿También en el nombre del bloque?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#91 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIaqtg90O9yOLCe8Cm97FClDeVPrQUsUks5sz8JLgaJpZM4QRW0w>
.[image: Web Bug from
https://github.com/notifications/beacon/AIaqthmx1b4cDlWXm0tligO_DvHGdPzdks5sz8JLgaJpZM4QRW0w.gif]
{"api_version":"1.0","publisher":{"api_key":"
05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":
{"external_key":"github/Program-AR/gs-element-blockly"
,"title":"Program-AR/gs-element-blockly","subtitle":"GitHub
repository","main_image_url":"https://cloud.githubusercontent.com/assets/
143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","
avatar_image_url":"https://cloud.githubusercontent.com/
assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png
","action":{"name":"Open in GitHub","url":"https://github.
com/Program-AR/gs-element-blockly"}},"updates":{"
***@***.*** in #91: ¿También en el
nombre del bloque?"}],"action":{"name":"View Issue","url":"https://github.
com/Program-AR/gs-element-blockly/issues/91#issuecomment-342357144"}}}
--
Este mensaje ha sido analizado por el servidor antispam1.unq.edu.ar de la
Universidad Nacional de Quilmes
en busca de virus y otros contenidos peligrosos, y se considera que está
limpio.
|
Ah, perfecto.
Mientras resulte Gobstones compatible. :)
Después veremos los mensajes de error.
FF
…On Nov 7, 2017 13:28, "asanzo" ***@***.***> wrote:
Arreglado en 0.5.12
[image: seleccion_012]
<https://user-images.githubusercontent.com/5421992/32504558-e1e7449a-c3be-11e7-9ef1-6d097185f897.png>
escupe:
program {
pasaACamel := pasaACamel + 1
}
A lo que me refería, Fidel, es a que si le tengo que obligar a escribir
con minúscula a quien escrible el nombre del bloque. Como se ve arriba,
decidí que no.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#91 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIaqtv_Q6k6SrFXJJBQ4cWMWkU2Pu_Cmks5s0IUEgaJpZM4QRW0w>
.[image: Web Bug from
https://github.com/notifications/beacon/AIaqtitEAp123agi84fB88cVKU97_8SZks5s0IUEgaJpZM4QRW0w.gif]
{"api_version":"1.0","publisher":{"api_key":"
05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":
{"external_key":"github/Program-AR/gs-element-blockly"
,"title":"Program-AR/gs-element-blockly","subtitle":"GitHub
repository","main_image_url":"https://cloud.githubusercontent.com/assets/
143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","
avatar_image_url":"https://cloud.githubusercontent.com/
assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png
","action":{"name":"Open in GitHub","url":"https://github.
com/Program-AR/gs-element-blockly"}},"updates":{"
***@***.*** in #91: Arreglado en
0.5.12\r\n\r\n![seleccion_012](https://user-images.
githubusercontent.com/5421992/32504558-e1e7449a-c3be-11e7-
9ef1-6d097185f897.png)\r\n\r\nescupe: \r\n\r\n```Gobstones\r\nprogram
{\r\n pasaACamel := pasaACamel + 1\r\n}\r\n```\r\n\r\nA lo que me refería,
Fidel, es a que si le tengo que obligar a escribir con minúscula a quien
escrible el nombre del bloque. Como se ve arriba, decidí que
no."}],"action":{"name":"View Issue","url":"https://github.
com/Program-AR/gs-element-blockly/issues/91#issuecomment-342538750"}}}
--
Este mensaje ha sido analizado por el servidor antispam2.unq.edu.ar de la
Universidad Nacional de Quilmes
en busca de virus y otros contenidos peligrosos, y se considera que está
limpio.
|
No description provided.
The text was updated successfully, but these errors were encountered: