strip(n) {
		n = n.replace(/[.\u2026,!?:;'\u2019"\-_\u2010\s+]/g, '').toLowerCase();
		return this.removeDiacritics(n);
	}

Not applying fuzzy logic when matching last.fm names to library track names during writing the popularity tag was a deliberate decision. Track names sometimes have very subtle differences and so fuzzy matching can lack specificity. Same applies to library track matching during playlist generation by auto DJ or find etc. but then it's not as critical as it's not altering the music files.

Anyway in the next version the standard matching will ignore diacritics, e.g. a = á, so your use case would be solved more specifically that way.

If you want to try it before then, you can change the strip function in helpers. You should be able to switch partial matching off.

Code: [Select]

	strip(n) {
		n = n.replace(/[.\u2026,!?:;'\u2019"\-_\u2010\s+]/g, '').toLowerCase();
		return this.removeDiacritics(n);
	}

It's trivial to add the fuzzy logic to the tagger if you still think its warranted.

The current UI allows multiple selection, so maybe it can be processed on entire lists too

Quote

The current UI allows multiple selection, so maybe it can be processed on entire lists too

Well that can be done already. Select the items in the last.fm top tracks list to be tagged using middle click. They'll be added to the playlist. Then select the items in the playlist & tag.

It's also possible to use the + (top right), then click local. That will load all the matching library tracks into a playlist which can be tagged. Best if track source on drop down is set to library, so cached youtube tracks don't load,  as I think foo_youtube has issues with adding multi-value tags.

So no need to copy/paste and tag one by one.

[20:39:57] foo_spider_monkey_panel:
Error: Spider Monkey Panel v1.6.1 (Find & Play v1.1.1 by WilB)
Index is out of bounds

File: web.js
Line: 1985, Column: 7
Stack trace:
  runAddLoc/</<@web.js:1985:7

Quote

The current UI allows multiple selection, so maybe it can be processed on entire lists too

Well that can be done already. Select the items in the last.fm top tracks list to be tagged using middle click. They'll be added to the playlist. Then select the items in the playlist & tag.

It's also possible to use the + (top right), then click local. That will load all the matching library tracks into a playlist which can be tagged. Best if track source on drop down is set to library, so cached youtube tracks don't load,  as I think foo_youtube has issues with adding multi-value tags.

So no need to copy/paste and tag one by one.

	runAddLoc(p_loc, p_alb_id, p_pn, p_alb_set, type) {
		if (type == 'outstanding') $.sort(p_loc, 'id');
		if (p_alb_set) { // should only be one item so can clear here
			if (p_alb_set !== 2) pl.clear(pl.selection());
		}
		p_loc.forEach(v => {
			if (type == 'stnd') {
				if (v.id == (!p_alb_set ? panel.add_loc.ix : p_alb_id)) {
					v.id = 'x';
					panel.add_loc.ix++;
				} else return;
			}
			if (type == 'outstanding') {
				if (v.id != 'x') v.id = 'x';
				else return;
			}
			if (v.path) {
				this.start[v.path.slice(-11)] = Date.now();
				panel.addLoc(v.path, p_pn, true, p_alb_set, p_alb_set, true);
				if (p_alb_set) panel.addLoc(v.path, pl.selection(), true, p_alb_set, p_alb_set);
				if (p_alb_id == 'playTracks') panel.addLoc(v.path, pl.cache(), true, p_alb_set, p_alb_set, true);
			} else if (v.handle) {
				let timeout = this.loadTime.length ? Math.min($.average(this.loadTime), 500) : 25;
				this.hl.Add(v.handle);
				setTimeout(() => {
					const pn = pl.selection()
					if (p_alb_set) plman.ClearPlaylistSelection(pn);
					panel.add_loc.timestamp = Date.now();
					if (!p_alb_set || p_alb_set === 2) plman.UndoBackup(pn);
					if (p_alb_set) plman.InsertPlaylistItems(pn, plman.PlaylistItemCount(pn), this.hl, p_alb_set);
					else plman.InsertPlaylistItems(p_pn, plman.PlaylistItemCount(p_pn), this.hl, p_alb_set);
					if (p_alb_set) {
						plman.EnsurePlaylistItemVisible(pn, plman.PlaylistItemCount(pn) - 1);
						plman.SetPlaylistFocusItemByHandle(pn, this.hl[0]);
					}
					this.hl = new FbMetadbHandleList();
				}, timeout);
			}
		});
	}

That's the error you reported before? It's should be fixed in the shortly to be released version.

New Version: Find & Play v1.2.1

album IS MusicBrainzName

album HAS MusicBrainzName

It looks like the ones that aren't matching are EPs. As there appears to be no agreed specific tag field for EPs, it just checks the album field.

case v.releaseType == 'Single':// && ppt.mb != 1:

if (v.releaseType != 'Album' && v.releaseType != 'Compilation' && !v.title.endsWith(' [' + v.releaseType + ']')) {

if (v.releaseType != 'Album' && v.releaseType != 'Compilation' && v.releaseType != 'EP' && !v.title.endsWith(' [' + v.releaseType + ']')) {

If you don't want to wait until then:

Your code change suggestions worked

I couldn't see any problems, so I'll need an exact reproducible example of the 'all' issue if you want me to look into it.

It looks like the ones that aren't matching are EPs. As there appears to be no agreed specific tag field for EPs, it just checks the album field.

Default uses the following type of query:

Code: [Select]

album IS MusicBrainzName

If partial match (search tab) is enabled the following type of query is used:

Code: [Select]

album HAS MusicBrainzName

Latter might be better if, e.g. 'Ballad of the Starchild' is actually named 'Ballad of the Starchild EP'?

If there are still unexpected issues, please advise foobar2000 version and post a screenshot of the properties of an item that fails so I can see all the metadata names and values.

For artist photos, go to nowplaying panel & click it until artist photos show, then toggle back. If artist photos don't show, check the path on the image tab.

I've made the necessary changes for the next version.

M:\Music\_Remote\%artist%\%album%\