nystudio107 / craft-seomatic

src/assetbundles/seomatic/src/js/seomatic-tokens.js

/**
 * SEOmatic plugin for Craft CMS 3.x
 *
 * A turnkey SEO implementation for Craft CMS that is comprehensive, powerful,
 * and flexible
 *
 * @link      https://nystudio107.com
 * @copyright Copyright (c) 2017 nystudio107
 */

/**
 * @author    nystudio107
 * @package   SEOmatic
 * @since     3.0.0
 */

// JavaScript
import Tokenfield from 'tokenfield';

// Tokenize any seomatic-keywords fields
for (const el of document.querySelectorAll('.seomatic-keywords')) {
    let keywords = undefined;

The assignment of undefined is not necessary as keywords is implicitly marked as undefined by the declaration.

    if (el.value) {
        keywords = el.value.split(',').map((value, index) => {
            if (value !== '') {

There is no return statement if value !== "" is false. Are you sure this is correct? If so, consider adding return; explicitly.

                return {id: index, name: value};
            }
        });
    }
    let options = {
        el: el,
        addItemOnBlur: true,
        addItemsOnPaste: true,
        delimiters: [','],
    };
    if (keywords !== undefined && keywords[0] !== undefined) {
        options.setItems = keywords;
    }
    let tf = new Tokenfield(options);
    let resetting = false;
    tf.on('change', (tokenField) => {
        if (!resetting && el.disabled) {
            resetting = true;
            tokenField._onReset();
            resetting = false;
            return;
        }

There should be no empty lines in a multi-line function call.

        let values = tokenField._vars.setItems.map((value) => {
            return value.name;
        });
        

There should be no empty lines in a multi-line function call.

        tokenField.el.value = values.join(',');
    });
}