Skip to content

Commit

Permalink
Merge pull request #484 from cmv/fix/replace-indexOf-with-array.indexOf
Browse files Browse the repository at this point in the history
replace indexOf with array.indexOf
  • Loading branch information
DavidSpriggs committed Nov 23, 2015
2 parents a176c14 + 557e67d commit 8c7fe83
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions viewer/js/gis/dijit/Identify.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ define([
// Identify (Feature Service), create an
// infoTemplate for the graphic features. Create
// it only if one does not already exist.
if (layer.capabilities && layer.capabilities.toLowerCase().indexOf('data') < 0) {
if (layer.capabilities && array.indexOf(layer.capabilities.toLowerCase(), 'data') < 0) {
if (!layer.infoTemplate) {
var infoTemplate = this.getInfoTemplate(layer, layer.layerId);
if (infoTemplate) {
Expand Down Expand Up @@ -200,7 +200,7 @@ define([
// handle feature layers that come from a feature service
// and may already have an info template
var layer = evt.graphic._layer;
if (layer.infoTemplate || (layer.capabilities && layer.capabilities.toLowerCase().indexOf('data') < 0)) {
if (layer.infoTemplate || (layer.capabilities && array.indexOf(layer.capabilities.toLowerCase(), 'data') < 0)) {
return false;
}

Expand Down Expand Up @@ -262,7 +262,7 @@ define([
} else if ((ref.declaredClass === 'esri.layers.FeatureLayer') && !isNaN(ref.layerId)) { // feature layer
// do not allow feature layer that does not support
// Identify (Feature Service)
if (ref.capabilities && ref.capabilities.toLowerCase().indexOf('data') > 0) {
if (ref.capabilities && array.indexOf(ref.capabilities.toLowerCase(), 'data') >= 0) {
layerIds = [ref.layerId];
}
} else if (ref.layerInfos) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ define([
postCreate: function () {
this.inherited(arguments);
// Should the control be visible or hidden?
if (this.control.controlOptions.layerIds && this.control.controlOptions.layerIds.indexOf(this.sublayerInfo.id) < 0) {
if (this.control.controlOptions.layerIds && array.indexOf(this.control.controlOptions.layerIds, this.sublayerInfo.id) < 0) {
domClass.add(this.domNode, 'layerControlHidden');
}
var checkNode = this.checkNode;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ define([
postCreate: function () {
this.inherited(arguments);
// Should the control be visible or hidden?
if (this.control.controlOptions.layerIds && this.control.controlOptions.layerIds.indexOf(this.sublayerInfo.id) < 0) {
if (this.control.controlOptions.layerIds && array.indexOf(this.control.controlOptions.layerIds, this.sublayerInfo.id) < 0) {
domClass.add(this.domNode, 'layerControlHidden');
}
var checkNode = this.checkNode;
Expand Down
2 changes: 1 addition & 1 deletion viewer/js/gis/dijit/StreetView.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ define([

'dijit/form/Button',
'xstyle/css!./StreetView/css/StreetView.css',
'gis/plugins/async!//maps.google.com/maps/api/js?v=3&sensor=false'
'gis/plugins/async!//maps.google.com/maps/api/js?v=3'
], function (declare, _WidgetBase, _TemplatedMixin, _WidgetsInTemplateMixin, lang, aspect, topic, GraphicsLayer, Graphic, SimpleRenderer, template, PictureMarkerSymbol, domStyle, Point, SpatialReference, MenuItem, proj4, i18n) {

return declare([_WidgetBase, _TemplatedMixin, _WidgetsInTemplateMixin], {
Expand Down

0 comments on commit 8c7fe83

Please sign in to comment.