bigcommerce · jairo-bc · Feb 28, 2025 · Feb 26, 2025
diff --git a/helpers/3p/object.js b/helpers/3p/object.js
@@ -36,7 +36,7 @@ helpers.extend = function(/*objects*/) {
     var obj = args[i];
     if (utils.isObject(obj)) {
       for (var key in obj) {
-        if (obj.hasOwnProperty(key)) {
+        if (Object.hasOwnProperty.call(obj, key)) {
           context[key] = obj[key];
         }
       }
@@ -93,7 +93,7 @@ helpers.forOwn = function(obj, options) {
   var result = '';
 
   for (var key in obj) {
-    if (obj.hasOwnProperty(key)) {
+    if (Object.hasOwnProperty.call(obj, key)) {
       data.key = key;
       result += options.fn(obj[key], {data: data});
     }

diff --git a/helpers/3p/utils/index.js b/helpers/3p/utils/index.js
@@ -182,7 +182,7 @@ utils.isUndefined = function (val) {
  */
 
 utils.isOptions = function (val) {
-  return utils.isObject(val) && val.hasOwnProperty('hash');
+  return utils.isObject(val) && Object.hasOwnProperty.call(val, 'hash');
 };
 
 /**

diff --git a/helpers/3p/utils/lib/deepMatches.js b/helpers/3p/utils/lib/deepMatches.js
@@ -41,7 +41,7 @@ function matchArray(arr, value) {
 
 function matchObject(obj, value) {
   for (var key in value) {
-    if (value.hasOwnProperty(key)) {
+    if (Object.hasOwnProperty.call(value, key)) {
       if (deepMatches(obj[key], value[key]) === false) {
         return false;
       }

diff --git a/helpers/3p/utils/lib/isPlainObject.js b/helpers/3p/utils/lib/isPlainObject.js
@@ -16,7 +16,7 @@ function isPlainObject(o) {
     if (isObject(prot) === false) {return false;}
 
     // If constructor does not have an Object-specific method
-    if (prot.hasOwnProperty('isPrototypeOf') === false) {
+    if (Object.hasOwnProperty.call(prot, 'isPrototypeOf') === false) {
         return false;
     }
 

diff --git a/helpers/3p/utils/lib/markdown.js b/helpers/3p/utils/lib/markdown.js
@@ -28,7 +28,7 @@ module.exports = function markdown(config) {
 
         options = merge({html: true, breaks: true}, config, options);
         options = merge({}, options, options.markdown, options.hash);
-        if (options.hasOwnProperty('lang')) {
+        if (Object.hasOwnProperty.call(options, 'lang')) {
             options.langPrefix = options.lang;
         }
 

diff --git a/helpers/decrementVar.js b/helpers/decrementVar.js
@@ -15,7 +15,7 @@ const factory = globals => {
             globals.storage.variables = {};
         }
 
-        if (globals.storage.variables.hasOwnProperty(key) && Number.isInteger(globals.storage.variables[key])) {
+        if (Object.hasOwnProperty.call(globals.storage.variables, key) && Number.isInteger(globals.storage.variables[key])) {
             // Decrement value if it already exists
             globals.storage.variables[key] -= 1;
         } else {
@@ -28,7 +28,7 @@ const factory = globals => {
         }
 
         // Return current value
-        return globals.storage.variables.hasOwnProperty(key) ? globals.storage.variables[key] : undefined;
+        return Object.hasOwnProperty.call(globals.storage.variables, key) ? globals.storage.variables[key] : undefined;
     };
 };
 

diff --git a/helpers/deprecated/pick.js b/helpers/deprecated/pick.js
@@ -12,7 +12,7 @@ const factory = () => {
         const paths = args[0];
         if (paths && Array.isArray(paths)) {
             paths.forEach((key) => {
-                if (target.hasOwnProperty(key)) {
+                if (Object.hasOwnProperty.call(target, key)) {
                     toReturn[key] = target[key];
                 }
             })

diff --git a/helpers/dynamicComponent.js b/helpers/dynamicComponent.js
@@ -19,7 +19,7 @@ const factory = globals => {
 
         path = Path.join(path, this['partial']);
 
-        if (globals.handlebars.partials[path] && globals.handlebars.partials.hasOwnProperty(path)) {
+        if (globals.handlebars.partials[path] && Object.hasOwnProperty.call(globals.handlebars.partials, path)) {
             return globals.handlebars.partials[path](this);
         }
     };

diff --git a/helpers/getVar.js b/helpers/getVar.js
@@ -9,7 +9,7 @@ const factory = globals => {
             throw new ValidationError("getVar helper key must be a string");
         }
 
-        return globals.storage.variables && globals.storage.variables.hasOwnProperty(key) ? globals.storage.variables[key] : undefined
+        return globals.storage.variables && Object.hasOwnProperty.call(globals.storage.variables, key) ? globals.storage.variables[key] : undefined
     };
 };
 

diff --git a/helpers/incrementVar.js b/helpers/incrementVar.js
@@ -15,7 +15,7 @@ const factory = globals => {
             globals.storage.variables = {};
         }
 
-        if (globals.storage.variables.hasOwnProperty(key) && Number.isInteger(globals.storage.variables[key])) {
+        if (Object.hasOwnProperty.call(globals.storage.variables, key) && Number.isInteger(globals.storage.variables[key])) {
             // Increment value if it already exists
             globals.storage.variables[key] += 1;
         } else {
@@ -28,7 +28,7 @@ const factory = globals => {
         }
 
         // Return current value
-        return globals.storage.variables.hasOwnProperty(key) ? globals.storage.variables[key] : undefined;
+        return Object.hasOwnProperty.call(globals.storage.variables, key) ? globals.storage.variables[key] : undefined;
     };
 };
 

diff --git a/helpers/lib/cdnify.js b/helpers/lib/cdnify.js
@@ -61,7 +61,7 @@ module.exports = globals => {
 
             if (cdnSettings) {
                 const endpointKey = protocol.slice(0, protocol.length - 1);
-                if (cdnSettings.hasOwnProperty(endpointKey)) {
+                if (Object.hasOwnProperty.call(cdnSettings, endpointKey)) {
                     if (cdnUrl) {
                         return [cdnSettings[endpointKey], path].join('/');
                     }

diff --git a/helpers/pluck.js b/helpers/pluck.js
@@ -3,7 +3,7 @@
 const factory = () => {
     return function(collection, path) {
         if (collection && Array.isArray(collection)) {
-            return collection.map(item => item.hasOwnProperty(path) ? item[path] : undefined);
+            return collection.map(item => Object.hasOwnProperty.call(item, path) ? item[path] : undefined);
         }
         return [];
     };

diff --git a/package.json b/package.json
@@ -15,7 +15,7 @@
   "scripts": {
     "lint": "eslint .",
     "lint-and-fix": "eslint . --fix",
-    "test": "lab -v -t 97.4 --ignore FinalizationRegistry,WeakRef,WebAssembly,__propKey,__esDecorate,__runInitializers,__setFunctionName,_time,format,SharedArrayBuffer,Atomics,BigUint64Array,BigInt64Array,BigInt,URL,URLSearchParams,TextEncoder,TextDecoder,queueMicrotask,__extends,__assign,__rest,__decorate,__param,__metadata,__awaiter,__generator,__exportStar,__createBinding,__values,__read,__spread,__spreadArrays,__spreadArray,__await,__asyncGenerator,__asyncDelegator,__asyncValues,__makeTemplateObject,__importStar,__importDefault,__classPrivateFieldGet,__classPrivateFieldSet,__classPrivateFieldIn,AggregateError,BroadcastChannel,structuredClone,DOMException,AbortController,AbortSignal,EventTarget,Event,MessageChannel,MessagePort,MessageEvent,atob,btoa,Blob,Performance,performance,ReadableStream,ReadableStreamDefaultReader,ReadableStreamBYOBReader,ReadableStreamBYOBRequest,ReadableByteStreamController,ReadableStreamDefaultController,TransformStream,TransformStreamDefaultController,WritableStream,WritableStreamDefaultWriter,WritableStreamDefaultController,ByteLengthQueuingStrategy,CountQueuingStrategy,TextEncoderStream,TextDecoderStream,CompressionStream,DecompressionStream,fetch,FormData,Headers,Request,Response,__addDisposableResource,__disposeResources,File,PerformanceEntry,PerformanceMark,PerformanceMeasure,PerformanceObserver,PerformanceObserverEntryList,PerformanceResourceTiming,crypto,Crypto,CryptoKey,SubtleCrypto,CustomEvent,WebSocket,Iterator,Navigator,navigator spec",
+    "test": "lab -v -t 97.4 --ignore FinalizationRegistry,WeakRef,WebAssembly,__propKey,__esDecorate,__runInitializers,__setFunctionName,_time,format,SharedArrayBuffer,Atomics,BigUint64Array,BigInt64Array,BigInt,URL,URLSearchParams,TextEncoder,TextDecoder,queueMicrotask,__extends,__assign,__rest,__decorate,__param,__metadata,__awaiter,__generator,__exportStar,__createBinding,__values,__read,__spread,__spreadArrays,__spreadArray,__await,__asyncGenerator,__asyncDelegator,__asyncValues,__makeTemplateObject,__importStar,__importDefault,__classPrivateFieldGet,__classPrivateFieldSet,__classPrivateFieldIn,AggregateError,BroadcastChannel,structuredClone,DOMException,AbortController,AbortSignal,EventTarget,Event,MessageChannel,MessagePort,MessageEvent,atob,btoa,Blob,Performance,performance,ReadableStream,ReadableStreamDefaultReader,ReadableStreamBYOBReader,ReadableStreamBYOBRequest,ReadableByteStreamController,ReadableStreamDefaultController,TransformStream,TransformStreamDefaultController,WritableStream,WritableStreamDefaultWriter,WritableStreamDefaultController,ByteLengthQueuingStrategy,CountQueuingStrategy,TextEncoderStream,TextDecoderStream,CompressionStream,DecompressionStream,fetch,FormData,Headers,Request,Response,__addDisposableResource,__disposeResources,File,PerformanceEntry,PerformanceMark,PerformanceMeasure,PerformanceObserver,PerformanceObserverEntryList,PerformanceResourceTiming,crypto,Crypto,CryptoKey,SubtleCrypto,CustomEvent,WebSocket,Iterator,Navigator,navigator,__rewriteRelativeImportExtension spec",
     "coverage": "lab -c -r console -o stdout -r html -o coverage.html spec",
     "release": "semantic-release"
   },
-Original file line number
+Diff line change
@@ Expand Up / @@ -182,7 +182,7 @@ utils.isUndefined = function (val) { @@
      */
     utils.isOptions = function (val) {
-      return utils.isObject(val) && val.hasOwnProperty('hash');
+      return utils.isObject(val) && Object.hasOwnProperty.call(val, 'hash');
     };
     /**
@@ Expand Down @@