Latest Contributions by TracySchloss

I have been using this function on a map click event to run multiple identify tasks using deferred and deferredList.  I'm now hoping to get this rewritten to use dojo/promise/all.  I'm getting lost on what should be changed in this code. Here is the original, and it works. function runIdentifies(evt) { idPoint = evt.mapPoint;     var layers = dojo.map(map.layerIds, function(layerId) {         return map.getLayer(layerId);     }); //Create an array of all layers in the map     layers = dojo.filter(layers, function(layer) {         if (layer.visibleLayers[0] !== -1){         return layer.getImageUrl && layer.visible         }     }); //Only dynamic layers have the getImageUrl function. Filter so you only query visible dynamic layers     var tasks = dojo.map(layers, function(layer) {         return new esri.tasks.IdentifyTask(layer.url);     }); //map each visible dynamic layer to a new identify task, using the layer url     var defTasks = dojo.map(tasks, function (task) {         return new dojo.Deferred();     }); //map each identify task to a new dojo.Deferred     var params = createIdentifyParams(layers,evt);     var dlTasks = new dojo.DeferredList(defTasks); //And use all of these Deferreds in a DeferredList     dlTasks.then(showIdentifyResults); //chain showIdentifyResults onto your DeferredList     for (i=0;i<tasks.length;i++) { //Use 'for' instead of 'for...in' so you can sync tasks with defTasks         try {             tasks.execute(params, defTasks.callback, defTasks.errback); //Execute each task         } catch (e) {             console.log("Error caught");             console.log(e);             defTasks.errback(e); //If you get an error for any task, execute the errback         }     } } //needed to limit identify to only currently visible layers function createIdentifyParams(layers,evt){ identifyParamsList.length = 0; dojo.forEach(layers, function (layer) {     var idParams = new esri.tasks.IdentifyParameters();     idParams.width = map.width;     idParams.height = map.height;     idParams.geometry = evt.mapPoint;     idParams.mapExtent = map.extent;     idParams.layerOption = esri.tasks.IdentifyParameters.LAYER_OPTION_VISIBLE;     var visLayers = layer.visibleLayers;     if (visLayers !== -1) {     idParams.layerIds = layer.visibleLayers;     }else {     idParams.layerIds = [];     }     idParams.tolerance = 3;     idParams.returnGeometry = true;     idParams.spatialReference = spatialReference;     identifyParamsList.push(idParams);     });     return identifyParamsList; }  function showIdentifyResults(r) {     var results = [];     r = dojo.filter(r, function (result) {         return r[0];     }); //filter out any failed tasks     for (i=0;i<r.length;i++) {         results = results.concat(r[1]);     }      results = dojo.map(results, function(result) {         var feature = result.feature;          var layerName = result.layerName;         feature.attributes.layerName = result.layerName;          feature.setInfoTemplate(generateInfoTemplate);         var resultGeometry = feature.geometry;         var resultType = resultGeometry.type;             if (layerName == "House" || layerName == "Congressional" ) {             feature.attributes.nameTitle = "Representative ";             feature.setInfoTemplate(legisInfoTemplate);         }                              if (layerName == "Senate") {             feature.attributes.nameTitle = "Senator ";             feature.setInfoTemplate(legisInfoTemplate);         }           return feature;              });             if(results.length === 0) {                 map.infoWindow.clearFeatures();             } else {                 map.infoWindow.setFeatures(results);             }             map.infoWindow.show(idPoint);             return results; }   Here's what I have so far, trying to substitute with Deferred and all.  Or should it be Promise and all?  From my reading, all and deferredList are very similar, but not so for Deferred vs. Promise function createIdentifyParams(layers,evt){ identifyParamsList.length = 0; arrayUtils.forEach(layers, function (layer) {     var idParams = new IdentifyParameters();     idParams.width = app.map.width;     idParams.height = app.map.height;     idParams.geometry = evt.mapPoint;     idParams.mapExtent = app.map.extent;     idParams.layerOption = IdentifyParameters.LAYER_OPTION_VISIBLE;     var visLayers = layer.visibleLayers;     if (visLayers !== -1) {     idParams.layerIds = layer.visibleLayers;     }else {     idParams.layerIds = [];     }     idParams.tolerance = 3;     idParams.returnGeometry = true;     idParams.spatialReference = spatialReference;     identifyParamsList.push(idParams);     });     return identifyParamsList; }  function runIdentifies(evt) { idPoint = evt.mapPoint; //Create an array of all layers in the map     var layers = arrayUtils.map(app.map.layerIds, function(layerId) {         return app.map.getLayer(layerId);     });  //Only dynamic layers have the getImageUrl function. Filter so you only query visible dynamic layers     layers = arrayUtils.filter(layers, function(layer) {         if (layer.visibleLayers[0] !== -1){         return layer.getImageUrl && layer.visible         }     }); //map each visible dynamic layer to a new identify task, using the layer url      var tasks = arrayUtils.map(layers, function(layer) {         return new IdentifyTask(layer.url);     });      var defTasks = arrayUtils.map(tasks, function (task) {       //  return new dojo.Deferred();  // original code       return new Deferred();     }); //map each identify task to a new dojo.Deferred //function to create an array of parameters, limits identify to only visible layers     var params = createIdentifyParams(layers,evt);     var dlTasks = new all(defTasks);     // var dlTasks = new dojo.DeferredList(defTasks); //And use all of these Deferreds in a DeferredList, original code     dlTasks.then(showIdentifyResults);      for (i=0;i<tasks.length;i++) {          try {             tasks.execute(params, defTasks.callback, defTasks.errback);          } catch (e) {             console.log("Error caught");             console.log(e);             defTasks.errback(e);          }     } }  function showIdentifyResults(r) {     var results = [];     r = dojo.filter(r, function (result) {         return r[0];     }); //filter out any failed tasks     for (i=0;i<r.length;i++) {         results = results.concat(r[1]);     }      results = arrayUtils.map(results, function(result) {         var feature = result.feature;          var layerName = result.layerName;         feature.attributes.layerName = result.layerName;          feature.setInfoTemplate(generateInfoTemplate);         var resultGeometry = feature.geometry;         var resultType = resultGeometry.type;             if (layerName == "House" || layerName == "Congressional" ) {             feature.attributes.nameTitle = "Representative ";             feature.setInfoTemplate(legisInfoTemplate);         }                              if (layerName == "Senate") {             feature.attributes.nameTitle = "Senator ";             feature.setInfoTemplate(legisInfoTemplate);         }           return feature;              });             if(results.length === 0) {                 map.infoWindow.clearFeatures();             } else {                 map.infoWindow.setFeatures(results);             }             map.infoWindow.show(idPoint);             return results; }   ... View more

I have several projects that have the basemap defined as part of the map constructor.  I have a project I'm reworking in the AMD style, so I'm going through, changing the code as best I can.  Maybe I've hit a gotcha on this one that I'm not aware of.  The definition for the map is   highlightMarkerSymbol = new SimpleMarkerSymbol(SimpleMarkerSymbol.STYLE_CIRCLE, 22,    new SimpleLineSymbol(SimpleLineSymbol.STYLE_SOLID,    new Color([255,255,0]), 2),    new Color([255,255,0,0.5]));    var popup = new Popup({      markerSymbol: highlightMarkerSymbol,     fillSymbol: new SimpleFillSymbol(SimpleFillSymbol.STYLE_SOLID,      new SimpleLineSymbol(SimpleLineSymbol.STYLE_SOLID,      new Color([255,200,0]), 2), new Color([255,255,0,0.50]))     }, dojo.create("div"));           map = new Map("mapDiv", {        infoWindow: popup,       basemap: "topo",       center: [-92.593, 38.5],       zoom: 7     });   After this I load a few different layers.  I'm also defining a basemapGallery to let the users change the background.  All my layers load fine and I'm not getting any errors.  When my page first loads, there is no basemap displayed, it's just blank.  If I zoom in a couple of times, then it draws.  If I use my tool to change the basemap, it also works fine.  I tried a few different basemaps, not just topo, with the same results.  Can anyone provide some insight as to why the basemap might not draw at first?  I have this same syntax in other projects and it works just fine in them! ... View more