-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fix project loading with bad text-number connections #2175
Fix project loading with bad text-number connections #2175
Conversation
This bug has come up a number of times on the forum. Here's the most recent example: https://community.appinventor.mit.edu/t/the-blocks-area-did-not-load-properly-changes-to-the-blocks-for-screen-5690035441500160-scrhoy-will-not-be-saved/7729?u=ewpatton |
Change-Id: I78c1666fd34346e46e68e112d02603e4de197fa5
b6dda5d
to
7e6aff6
Compare
Change-Id: I52e646cbfab94cf4a317c9149146bd01fd0da49a
I like the tests :D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The block is now marked bad upon load and is not kicked out. The blocks error count is correct as well.
@@ -471,13 +471,15 @@ Blockly.WorkspaceSvg.prototype.loadBlocksFile = function(formJson, blocksContent | |||
if (blocksContent.length != 0) { | |||
try { | |||
Blockly.Events.disable(); | |||
this.isLoading = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you help to explain why this is needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Never mind. I got it
Fixes #1213
Change-Id: I78c1666fd34346e46e68e112d02603e4de197fa5