Add button to bulk import existing tags

d3ce9ea0d5d7238d0c9c2532d5319fddacf5c4d4

d-ber <davide.bertalero@studenti.unimi.it>

public/scripts/BulkEditOverlay.js+16 -1 Viewed

             <div id="bulk_tag_popup">
                 <div id="bulk_tag_popup_holder">
                     <h3 class="marginBot5">Modify tags of ${this.characterIds.length} characters</h3>
                     <small class="bulk_tags_desc m-b-1">Add or remove the mutual tags of all selected characters. Import all or existing tags for all selected characters.</small>
                     <div id="bulk_tags_avatars_block" class="avatars_inline avatars_inline_small tags tags_inline"></div>
                     <br>
                     <div id="bulk_tags_div" class="marginBot5" data-characters='${characterData}'>

                         <div id="bulk_tag_popup_import_all_tags" class="menu_button" title="Import all tags from selected characters" data-i18n="[title]Import all tags from selected characters">
                             Import All
                         </div>
 +                        <div id="bulk_tag_popup_import_existing_tags" class="menu_button" title="Import existing tags from selected characters" data-i18n="[title]Import existing tags from selected characters">
 +                            Import Existing
 +                        </div>
                         <div id="bulk_tag_popup_cancel" class="menu_button" data-i18n="Cancel">Close</div>
                     </div>
                 </div>

         document.querySelector('#bulk_tag_popup_remove_mutual').addEventListener('click', this.removeMutual.bind(this));
         document.querySelector('#bulk_tag_popup_cancel').addEventListener('click', this.hide.bind(this));
         document.querySelector('#bulk_tag_popup_import_all_tags').addEventListener('click', this.importAllTags.bind(this));
 +        document.querySelector('#bulk_tag_popup_import_existing_tags').addEventListener('click', this.importExistingTags.bind(this));
 +    }
++
 +    /**
 +     * Import existing tags for all selected characters
 +     */
 +    async importExistingTags() {
 +        for (const characterId of this.characterIds) {
 +            await importTags(characters[characterId], { importExisting: true });
 +        }
++
 +        $('#bulkTagList').empty();
+    }
     /**

public/scripts/tags.js+7 -4 Viewed

  * @param {Character} character - The character
  * @param {object} [options] - Options
  * @param {boolean} [options.importAll=false] - Whether to import all tags without dialog
 + * @param {boolean} [options.importExisting=false] - Whether to import existing tags without dialog
  * @param {boolean} [options.forceShow=false] - Whether to force showing the import dialog
  * @returns {Promise<boolean>} Boolean indicating whether any tag was imported
  */
 async function importTags(character, { importAll = false, importExisting = false, forceShow = false } = {}) {
     // Gather the tags to import based on the selected setting
     const tagNamesToImport = await handleTagImport(character, { importAll, importExisting, forceShow });
     if (!tagNamesToImport?.length) {
         console.debug('No tags to import');
         return;

  * @param {Character} character - The character
  * @param {object} [options] - Options
  * @param {boolean} [options.importAll=false] - Whether to import all tags without dialog
 + * @param {boolean} [options.importExisting=false] - Whether to import existing tags without dialog
  * @param {boolean} [options.forceShow=false] - Whether to force showing the import dialog
  * @returns {Promise<string[]>} Array of strings representing the tags to import
  */
 async function handleTagImport(character, { importAll = false, importExisting = false, forceShow = false } = {}) {
     /** @type {string[]} */
     const importTags = character.tags.map(t => t.trim()).filter(t => t)
         .filter(t => !IMPORT_EXLCUDED_TAGS.includes(t))

     // Choose the setting for this dialog. If from settings, verify the setting really exists, otherwise take "ASK".
     const setting = forceShow ? tag_import_setting.ASK
         : importAll ? tag_import_setting.ALL
 -            : Object.values(tag_import_setting).find(setting => setting === power_user.tag_import_setting) ?? tag_import_setting.ASK;
 +            : importExisting ? tag_import_setting.ONLY_EXISTING
 +                : Object.values(tag_import_setting).find(setting => setting === power_user.tag_import_setting) ?? tag_import_setting.ASK;
     switch (setting) {
         case tag_import_setting.ALL: