Remove use of Array.prototype.includes which is not implemented in IE or Edge < 14. Fixes google/blockly#876.

This commit is contained in:
Tim Dawborn
2017-01-27 13:50:49 +11:00
parent 10108cc697
commit 14ee069543

View File

@@ -107,7 +107,7 @@ Blockly.Extensions.buildTooltipForDropdown = function(dropdownName, lookupTable)
var extensionFn = function() {
var thisBlock = this;
if (this.type && !blockTypesChecked.includes(this.type)) {
if (this.type && blockTypesChecked.indexOf(this.type) === -1) {
Blockly.Extensions.checkDropdownOptionsInTable_(
this, dropdownName, lookupTable);
blockTypesChecked.push(this.type);
@@ -117,7 +117,7 @@ Blockly.Extensions.buildTooltipForDropdown = function(dropdownName, lookupTable)
var value = thisBlock.getFieldValue(dropdownName);
var tooltip = lookupTable[value];
if (tooltip == null) {
if (!blockTypesChecked.includes(thisBlock.type)) {
if (blockTypesChecked.indexOf(thisBlock.type) === -1) {
// Warn for missing values on generated tooltips
var warning = 'No tooltip mapping for value ' + value +
' of field ' + dropdownName;