Modules feature

.gitignore

@@ -4,6 +4,7 @@ build-debug.log
 .DS_Store
 .settings
 .project
+.idea
 *.gz
 *.pyc
 *.komodoproject

README.md

@@ -1,3 +1,60 @@
+# Varwin Blockly
+
+Это форк Google Blockly с доработками для [Varwin](https://varwin.com/).  
+Основная фича Varwin Blockly: "Модули" (`/core/module.*`), которая сильно затрагивает core blockly.
+
+# Алгоритм обновления репозитория
+
+1. Подтягиваем изменения из `google/blockly`. (Github сам предложит это сделать, если обнаружит новые коммиты в master `google/blockly`)
+2. Если есть конфликты, будет создан PR для мерджа
+3. Переходим на ветку слияния и решаем все конфликты
+4. Выполняем 
+```bash
+npm run build
+```
+6. Запускаем локальный playground, которые лежит в `tests/playground.html`
+7. Если обнаруживаются ошибки, то переходим к пункту 3 и решаем возникшие проблемы. Обратите внимание на раздел с частыми проблемами ниже
+8. Выполняем
+```bash
+npm run prepare
+```
+10. Переходим в директорию `dist`
+11. Выполняем npm publish
+
+# Как удобно тестировать Varwin Blockly сразу в RMS
+
+Можно вызывать `gulp package` с аргументом `--output "path"` где `path` это путь до папки `blockly` в `node_modules` RMS.  
+Учитывайте, что путь должен быть относительно корня репозитория Varwin Blockly.
+
+Т.е.
+1. Делаем правки в Varwin Blockly
+2. Вызываем 
+```bash
+  npm run build
+```
+3. Вызываем
+```bash
+  gulp package --output "~/node_modules/blockly/"
+```
+6. Запускаем сборку RMS (или hot reload сам подтянет node_modules)
+
+# Частые проблемы
+
+### Playground не может найти файлы
+
+Идем в файл `/blockly_uncompresed.js` и исправляем пути до файлов.
+
+### Build падает с ошибкой "No supported platform for closure-compiler found"
+
+Можно попробовать установить не поддерживаемый пакет `google-closure-compiler-js`.  
+Затем залезть в `node_modules/google-closure-compiler/lib/utils.js` и в методе `getFirstSupportedPlatform` всегда возвращать  
+`"google-closure-compiler-js"`.
+
+Так пакет будет собираться средствами node.js, но это медленнее, зато работает. Пока что.
+
+
+=======
+
 # Blockly [![Build Status]( https://travis-ci.org/google/blockly.svg?branch=master)](https://travis-ci.org/google/blockly)
 
 Google's Blockly is a library that adds a visual code editor to web and mobile apps. The Blockly editor uses interlocking, graphical blocks to represent code concepts like variables, logical expressions, loops, and more. It allows users to apply programming principles without having to worry about syntax or the intimidation of a blinking cursor on the command line.  All code is free and open source.

blocks/procedures.js

@@ -940,6 +940,10 @@ Blockly.Blocks['procedures_callnoreturn'] = {
     option.callback = function() {
       var def = Blockly.Procedures.getDefinition(name, workspace);
       if (def) {
+        if (!def.InActiveModule()) {
+          workspace.getModuleManager().activateModule(workspace.getModuleManager().getModuleById(def.getModuleId()));
+          def = workspace.getBlockById(def.id);
+        }
         workspace.centerOnBlock(def.id);
         def.select();
       }

core/block.js

@@ -56,12 +56,13 @@ goog.requireType('Blockly.VariableModel');
  *     type-specific functions for this block.
  * @param {string=} opt_id Optional ID.  Use this ID if provided, otherwise
  *     create a new ID.
+ * @param {string=} moduleId Optional module ID.  Use this ID if provided, otherwise use active module.
  * @constructor
  * @implements {Blockly.IASTNodeLocation}
  * @implements {Blockly.IDeletable}
  * @throws When the prototypeName is not valid or not allowed.
  */
-Blockly.Block = function(workspace, prototypeName, opt_id) {
+Blockly.Block = function(workspace, prototypeName, opt_id, moduleId) {
   if (Blockly.Generator &&
       typeof Blockly.Generator.prototype[prototypeName] != 'undefined') {
     // Occluding Generator class members is not allowed.
@@ -83,6 +84,21 @@ Blockly.Block = function(workspace, prototypeName, opt_id) {
   this.inputList = [];
   /** @type {boolean|undefined} */
   this.inputsInline = undefined;
+  /**
+   * @type {string}
+   * @private
+   * */
+  this.moduleId_ = moduleId ? moduleId : workspace.getModuleManager().getActiveModule().getId();
+  /**
+   * @type {boolean}
+   * @private
+   */
+  this.obsolete = false;
+  /**
+   * @type {boolean}
+   * @private
+   */
+  this.removed = false;
   /**
    * @type {boolean}
    * @private
@@ -548,6 +564,43 @@ Blockly.Block.prototype.unplugFromStack_ = function(opt_healStack) {
   }
 };
 
+
+/**
+ * Returns module id for this block.
+ * @return string
+ * @package
+ */
+Blockly.Block.prototype.getModuleId = function() {
+  return this.moduleId_;
+};
+
+/**
+ * Returns module order for this block.
+ * @return int
+ * @package
+ */
+Blockly.Block.prototype.getModuleOrder = function() {
+  return this.workspace.getModuleManager().getModuleOrder(this.getModuleId());
+};
+
+/**
+ * Returns is this block in active module.
+ * @return string
+ * @package
+ */
+Blockly.Block.prototype.InActiveModule = function() {
+  return this.moduleId_ === this.workspace.getModuleManager().getActiveModule().getId();
+};
+
+/**
+ * Set module module id for this block.
+ * @param {string} moduleId module id.
+ * @package
+ */
+Blockly.Block.prototype.setModuleId = function(moduleId) {
+  return this.moduleId_ = moduleId;
+};
+
 /**
  * Returns all connections originating from this block.
  * @param {boolean} _all If true, return all connections even hidden ones.
@@ -1307,6 +1360,48 @@ Blockly.Block.prototype.setEnabled = function(enabled) {
   }
 };
 
+/**
+ * Get whether this block is obsolete or not.
+ * @return {boolean} True if obsolete.
+ */
+Blockly.Block.prototype.isObsolete = function() {
+  return this.obsolete;
+};
+
+/**
+ * Set whether the block is obsolete or not.
+ * @param {boolean} obsolete True if obsolete.
+ */
+Blockly.Block.prototype.setObsolete = function(obsolete) {
+  this.obsolete = obsolete;
+  if (obsolete) {
+    this.setWarningText(Blockly.Msg["OBSOLETE_WARNING"]);
+  } else {
+    this.setWarningText(null);
+  }
+};
+
+/**
+ * Get whether this block is removed or not.
+ * @return {boolean} True if removed.
+ */
+Blockly.Block.prototype.isRemoved = function() {
+  return this.removed;
+};
+
+/**
+ * Set whether the block is removed or not.
+ * @param {boolean} removed True if removed.
+ */
+Blockly.Block.prototype.setRemoved = function(removed) {
+  this.removed = removed;
+  if (removed) {
+    this.setWarningText(Blockly.Msg["REMOVED_WARNING"]);
+  } else {
+    this.setWarningText(null);
+  }
+};
+
 /**
  * Get whether the block is disabled or not due to parents.
  * The block's own disabled property is not considered.
@@ -1540,6 +1635,12 @@ Blockly.Block.prototype.jsonInit = function(json) {
   if (json['nextStatement'] !== undefined) {
     this.setNextStatement(true, json['nextStatement']);
   }
+  if (json['obsolete'] === true) {
+    this.setObsolete(true);
+  }
+  if (json['removed'] === true) {
+    this.setRemoved(true);
+  }
   if (json['tooltip'] !== undefined) {
     var rawValue = json['tooltip'];
     var localizedText = Blockly.utils.replaceMessageReferences(rawValue);