GitLab

Fixes #1379

Change-Id: I2682c4b12806e13a78524fce2d47293d46123f93

When the 2017 Blockly update was done, it used isRenderingOn to
determine when to position and render the blocks. This was to optimize
the interaction between the load/upgrade process and workspace
resizes. However, in rare instances it seems to result in blocks not
being placed correctly.

This change removes the special handling of upgrading. Instead, it
disables and enables the Blockly event system during a load/upgrade
operation. At the end of the upgrade process (if needed), it will also
force a save of the blocks workspace.

Fixes #1071

Change-Id: I6bec41dd67bc6371e794e93850ea1455b9acf8c7

Some aar files do not have an explicit directory entry for every
object. Make sure we create the needed directories.

Change-Id: Ic79d8852b6cc48f2399794c4087507e8f9c3bbaf

While GeoJSON files should not have a BOM per the spec, sometimes
systems will include the UTF byte order marker (BOM). This breaks the
JSON parsing on both in the web client and app. This change will check
for and strip out the BOM before passing the JSON string along for
further processing.

Change-Id: I9452871e19b3985f9beea48b30cf49b60e4835cb

Change-Id: Icf74d804689dcaa323bac3b4d1f0400631b74a5e

If there were multiple features of the same type in a GeoJSON file and
the file did not include names for the components, one would receive
multiple components with the same name (e.g., LineString1). LineString
processing was also broken, resulting in a coordinate string without
values. Setting the Source to None... would remove the components but
not clear the features off the map.

This commit addresses three changes:

1. Refactors how naming of map features is handled during the loading
   of GeoJSON into a MockFeatureCollection so that each component has a
   unique name.
2. It corrects handling of LineString coordinates when the line string
   is not part of a MultiLineString construct.
3. It cleans up how features are removed when a feature collection is
   cleared.

Change-Id: Iccbcab65989169ab730239b8915b62ca6b6f636c

We properly close the HttpURLConnection between App Engine (the App
Inventor server) and the buildserver infrastructure. If we don’t App
Engine sends RST packets which causes issues for some buildserver
infrastructures.

Also limit project size to 10MB up-front (rather then after an
IOException is thrown). Add reporting to the buildserver status page to
report the number of Java threads.

Change-Id: I934df537dfe9419f5f8effafe80f0a2460091cdd

In Firefox, Blockly seems to handle mouse events differently than in
Chrome. This generally isn't a problem, but it affects the warning
indicator toggle button. Blockly starts a drag at the mousedown but
then we never will see a mouseup or click event.

This change registers a mouse down listener on the toggle to stop
propagation to Blockly so that the warning indicator can receive the
click event. We also update to include support for touch events on the
toggle button to support App Inventor on tablets.

Change-Id: Iba4e1e274dd3eac86939bb122a1c7f15cc7d41f9

The base64 utility on macOS writes a carriage return at the end of the
base64 encoded string. When App Inventor attempts to read a project
template with this extra whitespace, it causes an exception to be
thrown since the server's decode() implementation expects a string
with a length multiple of 4. This change switches to using
decodeLines(), which ignores whitespace.

Change-Id: I34f6c7b1ef1fcd12b82eab67c98ffb422bbf5894

Fixes #1390

Set HELPURL correctly for several multi-operation math and logic blocks.

Added HELPURL definition for min and max.

Old HELPURL was to the Do-Return procedure.

Change-Id: I0fd9caafd4aeae00d451acc0699e833730415ace

This commit adds two new items to the context menu of the blocks editor. The first one is "Enable All Blocks" which enables all blocks on the workspace. The second option is "Disable All Blocks" which disables all of the blocks on the workspace.

Resolves #1091

Issue #1336 - Projects with reserved words do no compile correctly.
YAIL reserved words are also a bad idea, but they were only
checked for component names.

Now component and project names check for both YAIL and java words

Closes #1336

This commit makes sure that the custom colour picker autohides when the user interacts outside of the picker.

Resolves #979

Google Search reported this to us. If you go to “/b” you get an
ArrayIndexOutOfBounds exception. This is not a valid URL, normally in
practice a code follows the /b/ which points to the APK to download. So
given that /b is not valid, we turn the error into a 404, which makes
more sense.

Change-Id: Id3ac26a35eb2cb51038a74974850ee401b9fb91c

Provide for a second buildserver. If provided then the user is presented
a choice of two different build servers. This feature was introduced to
support building against two different versions of the Android SDK. As
of 8/1/2018 MIT is using this feature with the first buildserver
building Apps with a minSdk of 7 (but no targetSdk defined) to support
older devices. The second buildserver is configured with a minSdk of 14
and a targetSdk of 26 (along with the necessary component code
changes). This is required for new Apps submitted to the Google Play
Store. Having two buildservers provides us the flexibility to support
older devices while also ensuring that people can submit MIT App
Inventor created Apps to the Google Play Store.

Change-Id: Iae58f9ff4d112f8bc5d3203675f37cec7bf76920

Change-Id: I292edeb088aefbb6edeb171bc84ef1218b9d79d9

Change-Id: I6f82d9f363a95e8fb2f72e0bad022ad2977d7162

Change-Id: Ib6cffd0a01efd3e06c67e049768f11d382f4e599

Make the DataChanged event consistent with the GotValue event as to how
it handles images.

Change-Id: I08bdbd829cd423fe435081845b863ec4200eb7c4

Closes #1036

Change-Id: I616d845134a42b9f93186d0403d4683c1200af35

When we update App Inventor to a later Android API, the requirement that
webview interface have the annotation will be enforced. This one was
missing

Change-Id: I61dca27b000fb7b9166fe75bdd34ba00f2077841

This commit adds two new properties to the FusionTables component. The
first property is "LoadingDialogMessage". This allows the user to
customise the default "Please wait loading..." text which is shown in
the dialog after calling the .SendQuery function.  The second property
is "ShowLoadingDialog" this allows the user to choose whether or not
to show the dialog.

Change-Id: Ia034d66dcab4960ad3ef972e17c0d89336aca75d

If a designer property is included in a component without a
corresponding block entry, then the YAIL code generator will throw an
exception attempting to look up the type of the property when
generating set-and-coerce-property! code for the designer. This change
enforces that every @DesignerProperty have a corresponding
@SimpleProperty to ensure that components and extensions include the
necessary information.

Change-Id: I85cf53b1b657254658fd0cb0f05af3dde6b32740

Change-Id: Ibae0a6e0b9b4a06e809f4cf8265a1f10f86382aa

Fixes #1144 

Reference #!null directly in the while macro instead of via
the *the-null-value* which apparently isn’t always bound.

Change-Id: I2e178f4c6a0317d4014d115912c16b9689bf1dd5

Modified the “while” macro to avoid a bug in the Kawa in-line optimizer.

Change-Id: I4247adab9fbf80e2e3396f67c4aac5f4a21fa752

Due to an optimization that reused the component references in blocks
to determine the names of components, components with event handlers
would have their init-thunks called before components without
events. This would result in different initialization paths between
the REPL and compiled apps. This change makes the ordering of
init-thunks consistent by always using the order created by walking
the Form's JSON description.

Fixes #1279

Change-Id: I5ea1070df17a1dd96923cff2f0adfd6889a987b2

Change-Id: I1cc72fa09f8e5b51c22b78e774a6c5b5d119f9f7

The components reference page was missing a link to the Experimental component category.

This results in compilation building a single large Javascript download
instead of one for each browser/language combination. In production,
minification reduces the size of this file to something
reasonable. However if you build directly from master, this one file
will be pretty large. However it will work just fine.

This changes significantly speeds up the build process.

Change-Id: I330faf96fd95e538e3c19130c85c3ae8dbb077df

Fixes #1230

Change-Id: I1eb4579f4873d2e60b357acd463dbbd77261c936

Change-Id: Id0995d95294c4e604b17eebf495f3ee04b87c36d

Change-Id: I86ad3e3f63315d7b78765f59d1b5763b4f63104c