Suggested style improvements.

Author: gkellogg

lib/compact.js

@@ -143,7 +143,7 @@ api.compact = ({
 
     // process element keys in order
     const keys = Object.keys(element).sort();
-    for(let expandedProperty of keys) {
+    for(const expandedProperty of keys) {
       const expandedValue = element[expandedProperty];
 
       // compact @id and @type(s)
@@ -160,7 +160,7 @@ api.compact = ({
         } else {
           // expanded value must be a @type array
           compactedValue = [];
-          for(let expandedIri of expandedValue) {
+          for(const expandedIri of expandedValue) {
             const compactedType = api.compactIri(
               {activeCtx, iri: expandedIri, relativeTo: {vocab: true}})
 
@@ -193,7 +193,7 @@ api.compact = ({
         });
 
         // handle double-reversed properties
-        for(let compactedProperty in compactedValue) {
+        for(const compactedProperty in compactedValue) {
           if(activeCtx.mappings[compactedProperty] &&
             activeCtx.mappings[compactedProperty].reverse) {
             const value = compactedValue[compactedProperty];
@@ -270,7 +270,7 @@ api.compact = ({
       }
 
       // recusively process array values
-      for(let expandedItem of expandedValue) {
+      for(const expandedItem of expandedValue) {
         // compact property and get container type
         const itemActiveProperty = api.compactIri({
           activeCtx,
@@ -844,7 +844,7 @@ api.removePreserve = (ctx, input, options) => {
     }
 
     // recurse through properties
-    for(let prop in input) {
+    for(const prop in input) {
       let result = api.removePreserve(ctx, input[prop], options);
       const container = _getContextValue(ctx, prop, '@container') || [];
       if(options.compactArrays && _isArray(result) && result.length === 1 &&
@@ -939,4 +939,4 @@ function _checkNestProperty(activeCtx, nestProperty) {
       'JSON-LD compact error; nested property must have an @nest value resolving to @nest.',
       'jsonld.SyntaxError', {code: 'invalid @nest value'});
   }
-}
+}

lib/context.js

@@ -172,7 +172,7 @@ api.process = ({activeCtx, localCtx, options}) => {
     }
 
     // process all other keys
-    for(let key in ctx) {
+    for(const key in ctx) {
       api.createTermDefinition(rval, ctx, key, defined);
     }
 
@@ -266,7 +266,7 @@ api.createTermDefinition = (activeCtx, localCtx, term, defined) => {
     validKeys.push('@context', '@nest', '@prefix');
   }
 
-  for(let kw in value) {
+  for(const kw in value) {
     if(!validKeys.includes(kw)) {
       throw new JsonLdError(
         'Invalid JSON-LD syntax; a term definition must not contain ' + kw,
@@ -730,7 +730,7 @@ api.getInitialContext = (options) => {
     }
 
     // build fast CURIE map
-    for(let key in fastCurieMap) {
+    for(const key in fastCurieMap) {
       _buildIriMap(fastCurieMap, key, 1);
     }
 
@@ -765,7 +765,7 @@ api.getInitialContext = (options) => {
       }
     }
 
-    for(let key in next) {
+    for(const key in next) {
       if(key === '') {
         continue;
       }
@@ -1042,7 +1042,7 @@ function _findContextUrls(input, urls, replace, base) {
   }
 
   // input is an object
-  for(let key in input) {
+  for(const key in input) {
     if(key !== '@context') {
       _findContextUrls(input[key], urls, replace, base);
       continue;
@@ -1077,7 +1077,7 @@ function _findContextUrls(input, urls, replace, base) {
           }
         } else {
           // look for scoped context
-          for(let key in _ctx) {
+          for(const key in _ctx) {
             if(_isObject(_ctx[key])) {
               _findContextUrls(_ctx[key], urls, replace, base);
             }
@@ -1098,7 +1098,7 @@ function _findContextUrls(input, urls, replace, base) {
       }
     } else {
       // look for scoped context
-      for(let key in ctx) {
+      for(const key in ctx) {
         if(_isObject(ctx[key])) {
           _findContextUrls(ctx[key], urls, replace, base);
         }

lib/expand.js

@@ -161,7 +161,6 @@ api.expand = ({
     }
   }
 
-
   // expand the active property
   const expandedActiveProperty = _expandIri(
     activeCtx, activeProperty, {vocab: true});
@@ -455,7 +454,7 @@ function _expandObject({
       });
       // properties double-reversed
       if('@reverse' in expandedValue) {
-        for(let property in expandedValue['@reverse']) {
+        for(const property in expandedValue['@reverse']) {
           _addValue(
             expandedParent, property, expandedValue['@reverse'][property],
             {propertyIsArray: true});
@@ -465,7 +464,7 @@ function _expandObject({
       // FIXME: can this be merged with code below to simplify?
       // merge in all reversed properties
       let reverseMap = expandedParent['@reverse'] || null;
-      for(let property in expandedValue) {
+      for(const property in expandedValue) {
         if(property === '@reverse') {
           continue;
         }
@@ -644,9 +643,9 @@ function _expandObject({
   }
 
   // expand each nested key
-  for(let key of nests) {
+  for(const key of nests) {
     const nestedValues = _isArray(element[key]) ? element[key] : [element[key]];
-    for(let nv of nestedValues) {
+    for(const nv of nestedValues) {
       if(!_isObject(nv) ||
         Object.keys(nv).some(k => _expandIri(activeCtx, k, {vocab: true}) === '@value')) {
           throw new JsonLdError(
@@ -740,13 +739,13 @@ function _expandValue({activeCtx, activeProperty, value}) {
 function _expandLanguageMap(activeCtx, languageMap) {
   const rval = [];
   const keys = Object.keys(languageMap).sort();
-  for(let key of keys) {
+  for(const key of keys) {
     const expandedKey = _expandIri(activeCtx, key, {vocab: true})
     let val = languageMap[key];
     if(!_isArray(val)) {
       val = [val];
     }
-    for(let item of val) {
+    for(const item of val) {
       if(item === null) {
         // null values are allowed (8.5) but ignored (3.1)
         continue;