Bug 17322 - Leading and trailing spaces in names should be trimmed and not significant
: Leading and trailing spaces in names should be trimmed and not significant
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.6.0
: PC Windows Server 2003
: P2 normal with 5 votes (vote)
: 7.6.1
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-07-20 16:39 UTC by Jim McAtee
Modified: 2011-08-10 22:38 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jim McAtee 2011-07-20 16:39:44 UTC
Three different albums tagged with

ARTIST='Jack Johnson'
ARTIST=' Jack Johnson'
ARTIST='Jack Johnson '

produce three different artist/contributors in the library. Looking at the database, the NAMESORT and NAMESEARCH columns are stripped of the leading and trailing spaces, but the NAME column is not.

If this behavior is intentional, feel free to close as WORKSFORME.
Comment 1 Mike Walsh 2011-07-20 17:41:21 UTC
related:

bug 16956  (start around comment 19 or so)

bug 10324  (comment 73, so called fud)

http://forums.slimdevices.com/showthread.php?t=86176
Comment 3 Mike Cappella 2011-07-28 16:34:17 UTC
Although I voted for this, I personally don't care much.

Perhaps it will reduce support calls.  Its a Do What I Mean, Not What I Say type of fix.
Comment 4 Ron Olsen 2011-07-29 19:39:24 UTC
I think this change makes sense to improve the user experience with SBS.

It could be confusing to see three artists named Jack Johnson and not know why they are not one artist.

Or else make trimming leading and trailing blanks from tags a scanning option on the SBS Settings page. Default: yes.
Comment 5 Mike Walsh 2011-07-29 19:43:15 UTC
i would add that all we are doing is asking for SBS to act as it ALWAYS did prior to the diacritics fix.  this problem, and it certainly is a problem, is an unintended side effect of that fix.

there is no compelling reason NOT to fix this issue, and return things to the previous behavior which was sensible and the state of things for years.
Comment 6 SVN Bot 2011-08-10 21:01:31 UTC
 == Auto-comment from SVN commit #33041 to the slim repo by agrundman ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33041 ==

Fixed bug 17322, strip leading/trailing spaces from the following: contributors, genres, years, album titles
Comment 7 Mike Walsh 2011-08-10 22:38:56 UTC
thx andy!