@esri/solution-common 1.3.10 → 1.3.13

package/dist/esm/featureServiceHelpers.js

@@ -33,8 +33,8 @@ import { isTrackingViewTemplate, templatizeTracker } from "./trackingHelpers";
  * @param dependencies Array of IDependency for name mapping
  * @param templatizeFieldReferences Templatize all field references within a layer
  * @param templateDictionary Hash mapping property names to replacement values
- * @return A promise that will resolve when template has been updated
- * @protected
+ * @returns A promise that will resolve when template has been updated
+ * @private
  */
 export function templatize(itemTemplate, dependencies, templatizeFieldReferences, templateDictionary) {
     templateDictionary = templateDictionary || {};
@@ -122,7 +122,7 @@ export function deleteViewProps(layer) {
  * @param layer The data layer instance with field name references within
  * @param fieldInfos the object that stores the cached field infos
  * @param isPortal Controls what properties should be removed.
- * @return An updated instance of the fieldInfos
+ * @returns An updated instance of the fieldInfos
  */
 export function cacheFieldInfos(layer, fieldInfos, isPortal) {
     // cache the source fields as they are in the original source
@@ -157,6 +157,7 @@ export function cacheFieldInfos(layer, fieldInfos, isPortal) {
  * @param layer the data layer being cloned
  * @param prop the property name used to cache
  * @param fieldInfos the object that will store the cached property
+ * @private
  */
 export function _cacheFieldInfo(layer, prop, fieldInfos, removeProp) {
     /* istanbul ignore else */
@@ -177,7 +178,7 @@ export function _cacheFieldInfo(layer, prop, fieldInfos, removeProp) {
  * Cache popup info that can contain field references
  *
  * @param data The items data property
- * @return An updated instance of the popupInfos
+ * @returns An updated instance of the popupInfos
  */
 export function cachePopupInfos(data) {
     // store any popupInfo so we can update after any potential name changes
@@ -200,6 +201,7 @@ export function cachePopupInfos(data) {
  * @param popupInfos object to store the cahced popupInfo
  * @param type is it a layer or table
  * @param _items list or either layers or tables
+ * @private
  */
 export function _cachePopupInfo(popupInfos, type, _items) {
     _items.forEach((item) => {
@@ -215,8 +217,8 @@ export function _cachePopupInfo(popupInfos, type, _items) {
  * @param itemTemplate Item to be created; n.b.: this item is modified
  * @param templateDictionary Hash mapping property names to replacement values
  * @param createResponse Response from create service
- * @return An updated instance of the template
- * @protected
+ * @returns An updated instance of the template
+ * @private
  */
 export function updateTemplate(itemTemplate, templateDictionary, createResponse) {
     // Update the item with any typeKeywords that were added on create
@@ -237,8 +239,8 @@ export function updateTemplate(itemTemplate, templateDictionary, createResponse)
  *
  * @param itemTemplate Item to be created; n.b.: this item is modified
  * @param createResponse Response from create service
- * @return An updated instance of the template
- * @protected
+ * @returns An updated instance of the template
+ * @private
  */
 export function _updateTypeKeywords(itemTemplate, createResponse) {
     // https://github.com/Esri/solution.js/issues/589
@@ -257,7 +259,7 @@ export function _updateTypeKeywords(itemTemplate, createResponse) {
  * Example... { layer0: { fields: { lowerCaseSourceFieldName: newFieldNameAfterDeployment } } }
  *
  * @param layerInfos The object that stores the cached layer properties and name mapping
- * @return The settings object that will be used to de-templatize the field references.
+ * @returns The settings object that will be used to de-templatize the field references.
  */
 export function getLayerSettings(layerInfos, url, itemId, enterpriseIDMapping) {
     const settings = {};
@@ -289,7 +291,7 @@ export function getLayerSettings(layerInfos, url, itemId, enterpriseIDMapping) {
  *
  * @param templates A collection of AGO item templates.
  * @param solutionItemId The item id for the deployed solution item.
- * @return An updated collection of AGO templates with unique feature service names.
+ * @returns An updated collection of AGO templates with unique feature service names.
  */
 export function setNamesAndTitles(templates, solutionItemId) {
     const names = [];
@@ -362,8 +364,8 @@ export function updateSettingsFieldInfos(itemTemplate, settings) {
  *
  * @param template Template for feature service item
  * @param authentication Credentials for the request
- * @return A promise that will resolve when template has been updated
- * @protected
+ * @returns A promise that will resolve when template has been updated
+ * @private
  */
 export function updateTemplateForInvalidDesignations(template, authentication) {
     return new Promise((resolve, reject) => {
@@ -400,7 +402,7 @@ export function updateTemplateForInvalidDesignations(template, authentication) {
  * @param popupInfos The object from the popupInfo property for the layer
  * @param adminLayerInfos The object from the adminLayerInfo property for the layer
  * @param settings The settings object that has all of the mappings for de-templatizing.
- * @return An object that contains updated instances of popupInfos, fieldInfos, and adminLayerInfos
+ * @returns An object that contains updated instances of popupInfos, fieldInfos, and adminLayerInfos
  */
 export function deTemplatizeFieldInfos(fieldInfos, popupInfos, adminLayerInfos, settings) {
     const fieldInfoKeys = Object.keys(fieldInfos);
@@ -430,7 +432,7 @@ export function deTemplatizeFieldInfos(fieldInfos, popupInfos, adminLayerInfos,
  * to the item settings for the view.
  *
  * @param itemTemplate The current itemTemplate being processed.
- * @return array of layers and tables
+ * @returns array of layers and tables
  */
 export function getLayersAndTables(itemTemplate) {
     const properties = itemTemplate.properties;
@@ -457,8 +459,8 @@ export function getLayersAndTables(itemTemplate) {
  * @param url Feature service endpoint
  * @param ids layer and table ids
  * @param authentication Credentials for the request
- * @return A promise that will resolve an array of promises with either a failure or the data
- * @protected
+ * @returns A promise that will resolve an array of promises with either a failure or the data
+ * @private
  */
 export function getExistingLayersAndTables(url, ids, authentication) {
     // eslint-disable-next-line @typescript-eslint/no-floating-promises
@@ -480,8 +482,8 @@ export function getExistingLayersAndTables(url, ids, authentication) {
  *            service)
  * @param popupInfos the cached popup info from the layers
  * @param authentication Credentials for the request
- * @return A promise that will resolve when all layers and tables have been added
- * @protected
+ * @returns A promise that will resolve when all layers and tables have been added
+ * @private
  */
 export function addFeatureServiceLayersAndTables(itemTemplate, templateDictionary, popupInfos, authentication) {
     return new Promise((resolve, reject) => {
@@ -542,8 +544,8 @@ export function addFeatureServiceLayersAndTables(itemTemplate, templateDictionar
  * @param adminLayerInfos Hash map of a layers adminLayerInfo
  * @param fieldInfos Hash map of properties that contain field references
  * @param itemTemplate
- * @return A promise that will resolve when the feature service has been updated
- * @protected
+ * @returns A promise that will resolve when the feature service has been updated
+ * @private
  */
 export function addFeatureServiceDefinition(serviceUrl, listToAdd, templateDictionary, authentication, key, adminLayerInfos, fieldInfos, itemTemplate) {
     return new Promise((resolve, reject) => {
@@ -647,8 +649,8 @@ export function addFeatureServiceDefinition(serviceUrl, listToAdd, templateDicti
  * @param layerChunks Groups of layers or tables to add to the service
  * @param authentication Credentials for the request
  *
- * @return Add to service definition options
- * @protected
+ * @returns Add to service definition options
+ * @private
  */
 export function _updateAddOptions(itemTemplate, item, options, layerChunks, authentication) {
     const isMsView = getProp(itemTemplate, "properties.service.isMultiServicesView") || false;
@@ -681,7 +683,7 @@ export function _updateAddOptions(itemTemplate, item, options, layerChunks, auth
  * Remove "multiScaleGeometryInfo" for issue #526 to prevent invalid enablement of layer optimization
  *
  * @param layer the layer to evaluate
- * @protected
+ * @private
  */
 export function removeLayerOptimization(layer) {
     /* istanbul ignore else */
@@ -696,8 +698,8 @@ export function removeLayerOptimization(layer) {
  * @param itemTemplate the item template
  * @param templateDictionary Hash mapping Solution source id to id of its clone
  *
- * @return the updated item
- * @protected
+ * @returns the updated item
+ * @private
  */
 export function _updateForPortal(item, itemTemplate, templateDictionary) {
     // When deploying to portal we need to adjust the uniquie ID field up front
@@ -761,8 +763,8 @@ export function _updateForPortal(item, itemTemplate, templateDictionary) {
  * @param itemTemplate the item template
  * @param templateDictionary Hash mapping Solution source id to id of its clone
  *
- * @return an array of the source layers fields
- * @protected
+ * @returns an array of the source layers fields
+ * @private
  */
 export function _getFieldNames(table, itemTemplate, templateDictionary) {
     let sourceLayerFields = [];
@@ -793,8 +795,8 @@ export function _getFieldNames(table, itemTemplate, templateDictionary) {
  * @param item Layer or table
  * @param templateDictionary Hash mapping Solution source id to id of its clone
  *
- * @return updated layer or table
- * @protected
+ * @returns updated layer or table
+ * @private
  */
 export function _updateItemFields(item, fieldNames) {
     /* istanbul ignore else */
@@ -815,8 +817,8 @@ export function _updateItemFields(item, fieldNames) {
  *
  * @param table the table instance to evaluate
  * @param sourceLayerFields array of fields from the source service
- * @return Updated instance of the table
- * @protected
+ * @returns Updated instance of the table
+ * @private
  */
 export function _updateSourceLayerFields(table, sourceLayerFields) {
     /* istanbul ignore else */
@@ -836,7 +838,7 @@ export function _updateSourceLayerFields(table, sourceLayerFields) {
  *
  * @param item the item details from the current template
  * @param templateDictionary Hash mapping property names to replacement values
- * @protected
+ * @private
  */
 export function _updateGeomFieldName(adminLayerInfo, templateDictionary) {
     // issue #471
@@ -864,7 +866,7 @@ export function _updateGeomFieldName(adminLayerInfo, templateDictionary) {
  *
  * @param itemTemplate
  * @param templateDictionary Hash mapping Solution source id to id of its clone (and name & URL for feature service)
- * @protected
+ * @private
  */
 export function _updateTemplateDictionaryFields(itemTemplate, templateDictionary, compareItemId = true) {
     const layers = itemTemplate.properties.layers;
@@ -894,7 +896,7 @@ export function _updateTemplateDictionaryFields(itemTemplate, templateDictionary
  * @param templateDictionary Hash mapping Solution source id to id of its clone (and name & URL for feature service)
  * @param itemId The source id for the item
  * @param spatialReference { wkid: 102100 } for example
- * @protected
+ * @private
  */
 export function setDefaultSpatialReference(templateDictionary, itemId, spatialReference) {
     /* istanbul ignore else */
@@ -910,7 +912,7 @@ export function setDefaultSpatialReference(templateDictionary, itemId, spatialRe
  * @param serviceInfo Basic service information
  * @param itemTemplate The current template to process
  * @param templateDictionary Hash mapping Solution source id to id of its clone (and name & URL for feature service)
- * @protected
+ * @private
  */
 export function validateSpatialReferenceAndExtent(serviceInfo, itemTemplate, templateDictionary) {
     /* istanbul ignore else */
@@ -955,8 +957,8 @@ export function validateSpatialReferenceAndExtent(serviceInfo, itemTemplate, tem
  * @param adminLayerInfos Hash map of a layers adminLayerInfo
  * @param templateDictionary Hash mapping Solution source id to id of its clone (and name & URL for feature service)
  * @param authentication Credentials for the request
- * @return A promise that will resolve when the feature service has been updated
- * @protected
+ * @returns A promise that will resolve when the feature service has been updated
+ * @private
  */
 export function updateLayerFieldReferences(itemTemplate, fieldInfos, popupInfos, adminLayerInfos, templateDictionary) {
     return new Promise((resolveFn, rejectFn) => {
@@ -982,8 +984,8 @@ export function updateLayerFieldReferences(itemTemplate, fieldInfos, popupInfos,
  * @param adminLayerInfos Hash map of a layers adminLayerInfo
  * @param templateDictionary
  * @param authentication Credentials for the request
- * @return An object with detemplatized field references
- * @protected
+ * @returns An object with detemplatized field references
+ * @private
  */
 export function postProcessFields(itemTemplate, layerInfos, popupInfos, adminLayerInfos, templateDictionary) {
     return new Promise((resolveFn, rejectFn) => {
@@ -1064,8 +1066,8 @@ export function postProcessFields(itemTemplate, layerInfos, popupInfos, adminLay
  *  we need to pass it via an updateDef call or it will be set as the typeIdField of the source service
  *
  * @param item current layer or table
- * @return name of field to set for typeIdField in the update call
- * @protected
+ * @returns name of field to set for typeIdField in the update call
+ * @private
  */
 export function _getTypeIdField(item) {
     const typeIdFields = item.fields.filter((f) => {
@@ -1083,8 +1085,8 @@ export function _getTypeIdField(item) {
  *  visible true when added with the layer definition
  *
  * @param fieldInfo current layers or tables fieldInfo
- * @return Array of fields that should not be visible in the view
- * @protected
+ * @returns Array of fields that should not be visible in the view
+ * @private
  */
 export function _getFieldVisibilityUpdates(fieldInfo) {
     const visibilityUpdates = [];
@@ -1112,8 +1114,8 @@ export function _getFieldVisibilityUpdates(fieldInfo) {
  *
  * @param fieldInfo current view layer or table fieldInfo
  * @param fieldUpdates any existing field updates
- * @return Array of fields to be updated
- * @protected
+ * @returns Array of fields to be updated
+ * @private
  */
 export function _validateDomains(fieldInfo, fieldUpdates) {
     const domainAliasInfos = _getDomainAndAliasInfos(fieldInfo);
@@ -1138,7 +1140,7 @@ export function _validateDomains(fieldInfo, fieldUpdates) {
  * @param fields the alias or domain fields
  * @param key the field key to evaluate
  * @param fieldUpdates any existing field updates
- * @protected
+ * @private
  */
 export function _getPortalViewFieldUpdates(field, names, fields, key, fieldUpdates) {
     if (field.hasOwnProperty(key) && field[key]) {
@@ -1167,8 +1169,8 @@ export function _getPortalViewFieldUpdates(field, names, fields, key, fieldUpdat
  *
  * @param fieldInfo current view layer or table fieldInfo
  * @param fieldUpdates any existing field updates
- * @return Array of fields to be updated
- * @protected
+ * @returns Array of fields to be updated
+ * @private
  */
 export function _validateViewDomainsAndAlias(fieldInfo, item) {
     const domainAliasInfos = _getDomainAndAliasInfos(fieldInfo);
@@ -1188,7 +1190,7 @@ export function _validateViewDomainsAndAlias(fieldInfo, item) {
  *  Get array of domain fields and names and alias fields and names
  *
  * @param fieldInfo current view layer or table fieldInfo
- *
+ * @private
  */
 export function _getDomainAndAliasInfos(fieldInfo) {
     const domainFields = [];
@@ -1231,7 +1233,7 @@ export function _getDomainAndAliasInfos(fieldInfo) {
  * @param names array of field names
  * @param fields array of fields
  * @param key the field key to compare
- * @protected
+ * @private
  */
 export function _isViewFieldOverride(field, names, fields, key) {
     /* istanbul ignore else */
@@ -1248,7 +1250,7 @@ export function _isViewFieldOverride(field, names, fields, key) {
  *
  * @param itemTemplate
  * @param popupInfos popup info to be added back to the layer
- * @protected
+ * @private
  */
 export function updatePopupInfo(itemTemplate, popupInfos) {
     ["layers", "tables"].forEach(type => {
@@ -1268,6 +1270,7 @@ export function updatePopupInfo(itemTemplate, popupInfos) {
  *
  * @param basePath path used to de-templatize while deploying
  * @param value to be converted to lower case for lookup while deploying
+ * @private
  */
 export function _templatize(basePath, value, suffix) {
     if (value.startsWith("{{")) {
@@ -1283,6 +1286,7 @@ export function _templatize(basePath, value, suffix) {
  * @param object the object with the property to templatize
  * @param property the property of the object to templatize
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeProperty(object, property, basePath, suffix) {
     if (object && object.hasOwnProperty(property) && object[property]) {
@@ -1297,8 +1301,8 @@ export function _templatizeProperty(object, property, basePath, suffix) {
  * @param itemTemplate Template for feature service item
  * @param dependencies Array of IDependency for name mapping
  * @param templatizeFieldReferences Templatize all field references within a layer
- * @return A promise that will resolve when template has been updated
- * @protected
+ * @returns A promise that will resolve when template has been updated
+ * @private
  */
 export function _templatizeLayer(dataItem, adminItem, itemTemplate, dependencies, templatizeFieldReferences, templateDictionary) {
     // check for and repair common field issues
@@ -1353,6 +1357,7 @@ export function _validateFields(adminItem) {
  *
  * @param adminItem layer or table from the service
  * @param fieldNames string list of fields names
+ * @private
  */
 export function _validateDisplayField(adminItem, fieldNames) {
     const displayField = adminItem.displayField || "";
@@ -1393,6 +1398,7 @@ export function _validateDisplayField(adminItem, fieldNames) {
  *
  * @param adminItem layer or table from the service
  * @param fieldNames string list of fields names
+ * @private
  */
 export function _validateIndexes(adminItem, fieldNames) {
     const indexes = adminItem.indexes;
@@ -1428,6 +1434,7 @@ export function _validateIndexes(adminItem, fieldNames) {
  *
  * @param adminItem layer or table from the service
  * @param fieldNames string list of fields names
+ * @private
  */
 export function _validateTemplatesFields(adminItem, fieldNames) {
     const templates = adminItem.templates;
@@ -1454,6 +1461,7 @@ export function _validateTemplatesFields(adminItem, fieldNames) {
  *
  * @param adminItem layer or table from the service
  * @param fieldNames string list of fields names
+ * @private
  */
 export function _validateTypesTemplates(adminItem, fieldNames) {
     const types = adminItem.types;
@@ -1470,6 +1478,7 @@ export function _validateTypesTemplates(adminItem, fieldNames) {
  *
  * @param adminItem layer or table from the service
  * @param fieldNames string list of fields names
+ * @private
  */
 export function _validateEditFieldsInfo(adminItem, fieldNames) {
     const editFieldsInfo = adminItem.editFieldsInfo;
@@ -1506,7 +1515,8 @@ export function _validateEditFieldsInfo(adminItem, fieldNames) {
  * @param itemID The id for the item that contains this layer.
  * @param layer JSON return from the layer being templatized.
  * @param dependencies
- * @return An updated instance of the layer
+ * @returns An updated instance of the layer
+ * @private
  */
 export function _templatizeLayerFieldReferences(dataItem, itemID, layer, dependencies) {
     // This is the value that will be used as the template for adlib replacement
@@ -1535,8 +1545,8 @@ export function _templatizeLayerFieldReferences(dataItem, itemID, layer, depende
  *
  * @param layer The layer to be modified
  * @param dependencies Array of service dependencies
- * @return A new copy of the modified adminLayerInfo for the given layer
- * @protected
+ * @returns A new copy of the modified adminLayerInfo for the given layer
+ * @private
  */
 export function _templatizeAdminLayerInfo(layer, dependencies, templateDictionary) {
     // Create new instance of adminLayerInfo to update for clone
@@ -1569,7 +1579,7 @@ export function _templatizeAdminLayerInfo(layer, dependencies, templateDictionar
  *
  * @param object The layer to be modified
  * @param dependencies Array of service dependencies
- * @protected
+ * @private
  */
 export function _processAdminObject(object, dependencies) {
     deleteProp(object, "sourceId");
@@ -1582,8 +1592,8 @@ export function _processAdminObject(object, dependencies) {
  *
  * @param lookupName The current name from the source service
  * @param dependencies Array of IDependency for name mapping
- * @return The templatized name || undefined when no matching dependency is found
- * @protected
+ * @returns The templatized name || undefined when no matching dependency is found
+ * @private
  */
 export function _templatizeSourceServiceName(lookupName, dependencies) {
     const deps = dependencies.filter(dependency => dependency.name === lookupName);
@@ -1595,6 +1605,7 @@ export function _templatizeSourceServiceName(lookupName, dependencies) {
  * @param layer the layer object with the adminLayerInfo property to templatize
  * @param basePath path used to de-templatize while deploying
  * @param itemID the id for the item that contains this layer
+ * @private
  */
 export function _templatizeAdminLayerInfoFields(layer, dependencies) {
     // templatize the source layer fields
@@ -1628,6 +1639,7 @@ export function _templatizeAdminLayerInfoFields(layer, dependencies) {
  *
  * @param lookupName name of dependency we want to find the id of
  * @param dependencies array of item dependencies
+ * @private
  */
 export function _getDependantItemId(lookupName, dependencies) {
     const deps = dependencies.filter(dependency => dependency.name === lookupName);
@@ -1638,6 +1650,7 @@ export function _getDependantItemId(lookupName, dependencies) {
  *
  * @param fields array of sourceLayerFields to templatize
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeAdminSourceLayerFields(fields, basePath) {
     fields.forEach(f => _templatizeProperty(f, "source", basePath, "name"));
@@ -1647,6 +1660,7 @@ export function _templatizeAdminSourceLayerFields(fields, basePath) {
  *
  * @param topFilter the topFilter object to templatize
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeTopFilter(topFilter, basePath) {
     /* istanbul ignore else */
@@ -1677,6 +1691,7 @@ export function _templatizeTopFilter(topFilter, basePath) {
  *
  * @param layer the layer that has the relationships to templatize
  * @param itemID the id of the item that contains the related table
+ * @private
  */
 export function _templatizeRelationshipFields(layer, itemID) {
     if (layer && layer.relationships) {
@@ -1698,6 +1713,7 @@ export function _templatizeRelationshipFields(layer, itemID) {
  * @param basePath path used to de-templatize while deploying
  * @param itemID the id for the item that contains this layer
  * @param fieldNames array of fieldNames
+ * @private
  */
 export function _templatizePopupInfo(layerDefinition, layer, basePath, itemID, fieldNames) {
     // the data layer does not have the fields...will need to get those
@@ -1724,6 +1740,7 @@ export function _templatizePopupInfo(layerDefinition, layer, basePath, itemID, f
  * @param property that could have a field name referenced
  * @param fieldNames array for field names for the layer
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeName(object, property, fieldNames, basePath) {
     if (object.hasOwnProperty(property)) {
@@ -1745,6 +1762,7 @@ export function _templatizeName(object, property, fieldNames, basePath) {
  * @param layer json of layer being cloned
  * @param itemID id of the item that contains the current layer
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizePopupInfoFieldInfos(fieldInfos, layer, itemID, basePath) {
     fieldInfos.forEach((f) => {
@@ -1759,6 +1777,7 @@ export function _templatizePopupInfoFieldInfos(fieldInfos, layer, itemID, basePa
  * @param layer json of layer being cloned
  * @param itemID id of the item that contains the current layer
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeFieldName(name, layer, itemID, basePath) {
     if (name.indexOf("relationships/") > -1) {
@@ -1794,6 +1813,7 @@ export function _templatizeFieldName(name, layer, itemID, basePath) {
  * @param expressionInfos the popups expressionInfos to check
  * @param fieldNames array of the layers field names
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeExpressionInfos(expressionInfos, fieldNames, basePath) {
     return expressionInfos.map((i) => {
@@ -1811,6 +1831,7 @@ export function _templatizeExpressionInfos(expressionInfos, fieldNames, basePath
  * @param layer json of layer being cloned
  * @param itemID id of the item that contains the current layer
  * @param fieldNames array of field names
+ * @private
  */
 export function _templatizePopupElements(popupElelments, basePath, layer, itemID, fieldNames) {
     popupElelments.forEach((pe) => {
@@ -1830,6 +1851,7 @@ export function _templatizePopupElements(popupElelments, basePath, layer, itemID
  * @param basePath path used to de-templatize while deploying
  * @param layer json of layer being cloned
  * @param itemID id of the item that contains the current layer
+ * @private
  */
 export function _templatizeMediaInfos(mediaInfos, fieldNames, basePath, layer, itemId) {
     // templatize various properties of mediaInfos
@@ -1857,6 +1879,7 @@ export function _templatizeMediaInfos(mediaInfos, fieldNames, basePath, layer, i
  * @param layer the layer with the definition editor
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames json of layer being cloned
+ * @private
  */
 export function _templatizeDefinitionEditor(layer, basePath, fieldNames) {
     if (layer) {
@@ -1886,6 +1909,7 @@ export function _templatizeDefinitionEditor(layer, basePath, fieldNames) {
  * @param layer the layer with the definition editor
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames array of field names
+ * @private
  */
 export function _templatizeDefinitionExpression(layer, basePath, fieldNames) {
     if (layer && layer.hasOwnProperty("definitionExpression")) {
@@ -1898,6 +1922,7 @@ export function _templatizeDefinitionExpression(layer, basePath, fieldNames) {
  * @param expression the expression to test for field name references
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames array of the layers field names
+ * @private
  */
 export function _templatizeSimpleName(expression, basePath, fieldNames, suffix) {
     fieldNames.forEach(name => {
@@ -1915,6 +1940,7 @@ export function _templatizeSimpleName(expression, basePath, fieldNames, suffix)
  * @param layer the data layer
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames array of the layers field names
+ * @private
  */
 export function _templatizeDrawingInfo(layer, basePath, fieldNames) {
     if (layer) {
@@ -1935,6 +1961,7 @@ export function _templatizeDrawingInfo(layer, basePath, fieldNames) {
  * @param renderer the layers renderer
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames array of the layers field names
+ * @private
  */
 export function _templatizeRenderer(renderer, basePath, fieldNames) {
     switch (renderer.type) {
@@ -1958,6 +1985,7 @@ export function _templatizeRenderer(renderer, basePath, fieldNames) {
  * @param renderer the renderer object to check for field references
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames array of field names that will be used to search expressions
+ * @private
  */
 export function _templatizeGenRenderer(renderer, basePath, fieldNames) {
     /* istanbul ignore else */
@@ -2007,6 +2035,7 @@ export function _templatizeGenRenderer(renderer, basePath, fieldNames) {
  * @param renderer the renderer object to check for field references
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames array of field names that will be used to search expressions
+ * @private
  */
 export function _templatizeTemporalRenderer(renderer, basePath, fieldNames) {
     const renderers = [
@@ -2024,6 +2053,7 @@ export function _templatizeTemporalRenderer(renderer, basePath, fieldNames) {
  * @param authoringInfo  object containing metadata about the authoring process
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames the name of fields from the layer
+ * @private
  */
 export function _templatizeAuthoringInfo(authoringInfo, basePath, fieldNames) {
     /* istanbul ignore else */
@@ -2055,6 +2085,7 @@ export function _templatizeAuthoringInfo(authoringInfo, basePath, fieldNames) {
  * @param text the text that contains the expression
  * @param fieldName name of the field to test for
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeArcadeExpressions(text, fieldName, basePath) {
     const t = _templatize(basePath, fieldName, "name");
@@ -2102,6 +2133,7 @@ export function _templatizeArcadeExpressions(text, fieldName, basePath) {
  * @param labelingInfo the object that contains the labelingInfo
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames array of the layers field names
+ * @private
  */
 export function _templatizeLabelingInfo(labelingInfo, basePath, fieldNames) {
     labelingInfo.forEach((li) => {
@@ -2143,6 +2175,7 @@ export function _templatizeLabelingInfo(labelingInfo, basePath, fieldNames) {
  *
  * @param layer the data layer being cloned
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeTemplates(layer, basePath) {
     const templates = layer.templates || [];
@@ -2160,6 +2193,7 @@ export function _templatizeTemplates(layer, basePath) {
  *
  * @param layer the data layer being cloned
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeTypeTemplates(layer, basePath) {
     const types = layer.types;
@@ -2191,6 +2225,7 @@ export function _templatizeTypeTemplates(layer, basePath) {
  * @param obj the object to templatize
  * @param basePath path used to de-templatize while deploying
  * @param suffix expected suffix for template variable
+ * @private
  */
 export function _templatizeKeys(obj, basePath, suffix) {
     let _obj;
@@ -2212,6 +2247,7 @@ export function _templatizeKeys(obj, basePath, suffix) {
  *
  * @param layer the data layer being cloned
  * @param basePath path used to de-templatize while deploying
+ * @private
  */
 export function _templatizeTimeInfo(layer, basePath) {
     if (layer.timeInfo) {
@@ -2237,6 +2273,7 @@ export function _templatizeTimeInfo(layer, basePath) {
  * @param layer the data layer being cloned
  * @param basePath path used to de-templatize while deploying
  * @param fieldNames array of the layers field names
+ * @private
  */
 export function _templatizeDefinitionQuery(layer, basePath, fieldNames) {
     // templatize view definition query
@@ -2253,6 +2290,7 @@ export function _templatizeDefinitionQuery(layer, basePath, fieldNames) {
  *
  * @param fieldInfos the object that stores the cached information
  * @param id the id for the current layer being processed
+ * @private
  */
 export function _getNameMapping(fieldInfos, id) {
     // create name mapping
@@ -2333,7 +2371,8 @@ export function _getNameMapping(fieldInfos, id) {
 /**
  * Helper function to ensure same chunk size value is used in multiple locations
  *
- * @return a number that represents how many layers should be included per addToDef call
+ * @returns a number that represents how many layers should be included per addToDef call
+ * @private
  */
 export function _getLayerChunkSize() {
     return 20;