GitLab

Change-Id: If360e42e1fc497b7cdd5ffdc0756513ade51af14

Change-Id: Ifd5cd24c006bac1acc996fcb7d3763c58c54dd84

Change-Id: Ie39c3a9e0e35c58a97a9307db8cd31a086e602b5

Change-Id: Ic1a9060e2664a9709267e6dc3a36116f79ecca49

Add support for using WebRTC to communicate between the browser and the
Companion. While using WebRTC assets (images, sounds, etc.) are
downloaded directly to the Companion from the MIT App Inventor server
without being redirected through the user’s browser.

By supporting WebRTC we pave the way for offering MIT App Inventor over
https. The current HTTP based communications system used today (without
WebRTC) prohibits us from using https for MIT App Inventor itself.

This version of code supports both the new WebRTC approach as well as
the pre-WebRTC approach both within the MIT App Inventor
client (browser) and the Companion.

The new Companion will use WebRTC by default, but can be told to use the
HTTP based system (called a “Legacy Connection”) by checking a box on
the Companion home screen.

We will have to remove HTTP (Legacy) support prior to serving MIT App
Inventor over https.

Change-Id: Ib7b2db4261f2c852a7593b7a7b32b7356ff8cb6c

Set target api to 26 which corresponds to Android verison
8.0 (Oreo). Numerous changes where required in order to target api 26
while still having minSdk set to 7. As a trade-off, some “theme”
selections will not be available on older devices, but the apps packaged
with this code will operate on API 7 (Android 2.1).

Author: Evan W. Patton <ewpatton@mit.edu>
Author: Jeffrey I. Schiller <jis@mit.edu>

Change-Id: Ia307c080072c76d4699550892c99a07b65d3465e

Projects with many components may cause a large number calls to
OdeLog.log, which appends messages by converting the debugger panel's
content to HTML and back. This change enables logging only if the user
is an Admin, since only Admins have access to the debugger panel to
begin with. This reduces the number of cycles spent on logging for the
vast majority of App Inventor users. We also optimize how the log is
constructed so that users with Admin rights have reasonable
performance on complicated projects.

Change-Id: I62cc2af0e421d3f5fda668f72de62c1895414092

Fixes #1318 

Change-Id: Ie2ae31c90304d6278541f22513adfcf731f999a0

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

* Reset min size of button with images

If someone sets an image on a button in a non-Classic theme, the
minimum width/height settings in the theme will force the button to
sometimes resize the image arbitrarily to fit the minimum constraints
rather than allowing the button to assume the shape of the image like
it would in the Classic theme. This fix sets the minimum size of a
button to 0x0 so that it can take on the image size regardless of what
the platform's default dimensions for a button.

Change-Id: I590eb1bec860cda221e821c663c03c5cd81a65fe

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

The advantage to tracking the documentation in the same repo as the code itself is that changes to docco can be tightly coupled to the code changes it describes.

blocks.html has been changed to index.html. Images are also added to the repository with related links updated. All other links in the html are now external links appinventor.mit.edu/explore

HELPURLs are now relative links to the new location of the documentation.

Links are now set only in the English message file, since we only have English docco for the forseeable future, and all links in the localization files point to the same place anyway.

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

Setting the WebViewString from blocks will also trigger a
WebViewStringChange event. This forces users to have to work around
the extra change event when they are typically expecting the change
from within JavaScript code only. This change sets the internal value
directly to bypass firing the event.

Change-Id: Ic8f38bfb34faad1f67ee0bcc29289417ef780eb3

Fixes #1321

Change-Id: I1bb7050e0754bc9c027aad475e15aae2db677de0

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