0016631: Add option to trim leading/ending space character from metadata like ignore THE option - MantisBT

ID	Project	Category	View Status	Date Submitted	Last Update

0016631	MMW 5	Playlists (Auto) / Search / Filters	public	2020-05-20 12:40	2022-09-14 14:39

Reporter	peke	Assigned To
Priority	high	Severity	tweak	Reproducibility	N/A
Status	assigned	Resolution	duplicate
Product Version	5.0
Target Version	5.2

Summary	0016631: Add option to trim leading/ending space character from metadata like ignore THE option
Description	In case that SPACE characters is in front of playlist names we should add context option on playlist root Tree node to trim leading/ending space character from playlist names. It should not clutter Context menu. Maybe even adding it in "Manage Library" so that it can be cleaned for other Tree nodes and values? Or in options for playlist ( eg. "Trim Leading space in playlist names") as some users still want to leading space to influence sorting.
Additional Information	http://www.mediamonkey.com/forum/viewtopic.php?f=30&t=96584 IPV-293-83801
Tags	No tags attached.

Fixed in build

rusty 2020-05-22 17:42 administrator ~0058188	I don't think we should add a config entry for such an obscure usecase. We could always truncate spaces that appear at the beginning of an entry, but that might get in the way of users that enter a space on purpose.

peke 2020-05-24 12:17 developer ~0058241 Last edited: 2020-05-24 12:28	While testing other bugs I have found out that there is a need for Global Trim Space on tracks Metadata. Space influence sort also and also Duplicate Titles fail because There is a space in front of title/artist. I guess it is very useful if it is added like Ignore THE and it can use same dialog. That way users can easily exclude some and include others. EDIT: MM4/5 do not allow adding Space in tags (eg. for inline edit, properties and auto tag from filename) and it trims it immediately. But do not Trim it automatically if file tags already contain it.

peke 2020-05-24 15:17 developer ~0058253 Last edited: 2020-05-24 15:20	It is possible that this would also be fixed with auto trimming at 0016653 ? So only question is left if and we would add this feature or not. and even yes it is lower priority.

Ludek 2020-05-25 10:49 developer ~0058258 Last edited: 2020-05-25 11:50	I see that in course of 0014599 we already added - trimming of title (so that [Files to Edit > Duplicate titles] shows all duplicities) - trimming TAB from all fields (original issue in 0014599) - trimming spaces and CR+LF from album, artist, genre fields and thus solves the grouping inconsistencies described in my note 0014599:0049614 Based on my tests in 2250: - for the newly scanned track the track fields like title/artist/ablum are auto-trimmed in MM5 - for already existing tracks it shows track title with leading space in Properties dialog (until edited), but not in file listing (like Peke's example with his DB above) - playlist names are not trimmed on import and also MM5 allows to add leading space to playlists (originally reported in 0016631 ) So the current issue are: 1) we don't allow adding leading/trailing spaces in track title field when editing, but show the leading space for already imported files in Properties => we should either allow leading space and show it properly in listings or disallow leading space completelly (some pros and cons already discussed in 0014599 in the past) 2) we allows leading space to be added to the playlist name and we also show the leading space in the playlist listing => we might want this to unify with track title, but some users enters the space on purpose (e.g. when they want some playlists to be on the top of the list)