bSecureCheckout / bsecure-payments-js

utils/index.js

'use strict';

function isEmpty(x) {
    return (
        typeof x === 'undefined' ||
        x === null ||
        x === 'null' ||
        x === 'undefined' ||
        x === false ||
        x.length === 0 ||
        x === ''||
        (x === "object" && Object.key(x).length === 0) ||
        (x && Object.keys(x).length === 0
        && Object.getPrototypeOf(x) === Object.prototype)
    );
}

function isJson(str) {
    try {
      str=JSON.parse(str);

There is no return statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding return undefined; explicitly.

    } catch (e) {
        return str;
    }
}

function checkEmptyHtml(id, checkChildNode = false) {
    const elem = document.querySelector(`${id}`);
    if (elem === null || elem === undefined) {
        return true;
    } else if (elem && elem.childNodes.length > 0 && checkChildNode) {
        return true;
    }
    return false;
}

function prepareFrame(checkout_url) {
    const ifrm = document.createElement("iframe");
    let url = new URL(checkout_url);

The variable URL seems to be never declared. If this is a global, consider adding a /** global: URL */ comment.

    ifrm.setAttribute("src", url.href);
    ifrm.setAttribute("id", 'bSecurePaymentPluginEmbeddedIframe');
    ifrm.setAttribute("name", 'bSecurePaymentPluginEmbeddedIframe');
    ifrm.style.position = "absolute";
    ifrm.style.border = "none";
    ifrm.style.top = "0";
    ifrm.style.left = "0";
    ifrm.style.width = "100%";
    ifrm.style.height = "100%";
    document.getElementById("bSecurePaymentPluginContainer").appendChild(ifrm);
}


function resetFrame() {
    document.getElementById("bSecurePaymentPluginContainer").remove();
}

module.exports = {
    isEmpty: isEmpty,
    isJson: isJson,
    checkEmptyHtml: checkEmptyHtml,
    prepareFrame: prepareFrame,
    resetFrame: resetFrame,
};