-
-
Notifications
You must be signed in to change notification settings - Fork 155
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
Some chars in prelude tags are not escaped properly, and these can cause exported games to fail compilation #932
Comments
I suggest you supply an explicit repro case including a sample (tiny) game. |
it's ok, there's already a repro in the linked issue; i'm just lazy |
I thought I might have a look at it, but since I can't build the standalone right now I guess there isn't much point. |
I've had a look, I can fix it. But I don't see any easy way to test it. My motto is: if you can't test it don't write it. Might have to make an exception for this one. |
Illegal characters in file name or script are replaced by "!", and a warning emitted.
Illegal characters in file name or script are replaced by "!", and a warning emitted.
…warning emitted. Fixes increpare#932
Replaced by single purpose PR |
https://www.puzzlescript.net/play.html?p=1564d25b0c0eac9b02c8c77ea4f1bfc3 the homepage url is displayed incorrectly in the compiled version, but correctly in the uncompiled version, it seems, lol src/standalone_inlined.txt bin/standalone_inlined.txt
lol.... |
e.g. if you give a game the title of a HTML tag, for example
<script>
, then you will be allowed to share export the game, but for the export, the generated file is not valid HTML5 and it will cause the HTML5 code to run into errors and the game to fail to load. It seems that some characters (the<
and>
brackets in this example, possibly more) are not always escaped properly.Note that this only affects exported games and not share links.
The text was updated successfully, but these errors were encountered: