1202 'Split multiple...' checkbox missing [4938]

Beta Testing for Windows Products and plugins

Moderator: Gurus

chrisjj
Posts: 5007
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

1202 'Split multiple...' checkbox missing [4938]

Post by chrisjj »

1202 'Split multiple attribute fields with' checkbox missing

Though http://www.ventismedia.com/mantis/view.php?id=4938 is marked 'resolved', the checkbox for 'Split multiple attribute fields with' is missing.
Chris
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1202 'Split multiple attribute fields with' checkbox missing

Post by Owyn »

Added Note to Mantis.
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
nynaevelan
Posts: 5559
Joined: Wed Feb 07, 2007 11:07 pm
Location: New Jersey, USA
Contact:

Re: 1202 'Split multiple attribute fields with' checkbox missing

Post by nynaevelan »

I am not sure why the checkbox is needed. If there is a checkbox and it is not selected, doesn't that mean MM would not use a separator for multi-value fields?? What kind of impact would that have on the db?

Nyn
3.2x - Win7 Ultimate (Zen Touch 2 16 GB/Zen 8GB)
Link to Favorite Scripts/Skins

Join Dropbox, the online site to share your files
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1202 'Split multiple attribute fields with' checkbox missing

Post by Owyn »

nynaevelan wrote:I am not sure why the checkbox is needed. If there is a checkbox and it is not selected, doesn't that mean MM would not use a separator for multi-value fields?? What kind of impact would that have on the db?

Nyn
Ummm. You could be right. Will let Rusty/Jiri resolve the issue.
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
chrisjj
Posts: 5007
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1202 'Split multiple attribute fields with' checkbox missing

Post by chrisjj »

Owyn wrote:Added Note to Mantis.
Thanks Owyn.
Chris
chrisjj
Posts: 5007
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1202 'Split multiple attribute fields with' checkbox missing

Post by chrisjj »

> If there is a checkbox and it is not selected, doesn't that mean MM would not use a separator for multi-value fields??

Yes.

> What kind of impact would that have on the db?

None... if as the UI says this option is just for Appearence.
Chris
nynaevelan
Posts: 5559
Joined: Wed Feb 07, 2007 11:07 pm
Location: New Jersey, USA
Contact:

Re: 1202 'Split multiple attribute fields with' checkbox missing

Post by nynaevelan »

chrisjj wrote:> If there is a checkbox and it is not selected, doesn't that mean MM would not use a separator for multi-value fields??

Yes.

> What kind of impact would that have on the db?

None... if as the UI says this option is just for Appearence.
Ok, it's a little early in the morning and I do not have any caffeine in me yet, but if it is only for Appearances sake, what is the point of making the change? What difference does it make in having a checkbox if the db will just use the default of ";"? This option gives you the opportunity to select your own separator, so if a checkbox is added and you uncheck it then the db will just use the default, so just don't make any change to it, no need to add a checkbox.... As stated I don not have any caffeine in me so maybe I am missing the point, can you clarify if I am?

Nyn
3.2x - Win7 Ultimate (Zen Touch 2 16 GB/Zen 8GB)
Link to Favorite Scripts/Skins

Join Dropbox, the online site to share your files
chrisjj
Posts: 5007
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1202 'Split multiple attribute fields with' checkbox missing

Post by chrisjj »

> if it is only for Appearances sake, what is the point of making the change?

The point is to provide the old MM behaviour, wherein e.g. Artist is a treated as single-value field for display.

> What difference does it make in having a checkbox if the db will just use the default of ";"?

Actually I don't care what the DB uses, provided the checkbox works as designed.

> This option gives you the opportunity to select your own separator, so if a checkbox
> is added and you uncheck it then the db will just use the default

Not if it accords with the design! Unchecking a checkbox on "Split..." will disable splitting, not just disable selection of the split character.
Chris
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1202 'Split multiple...' checkbox missing BUG#4938

Post by Owyn »

Ummm. Perhaps the best solution is to require:
1) That the field always be present (i.e no checkbox),
2) That the permitted values be one of the special characters, excluding space
3) Maybe use a non-extending drop-down selection for the field.

----------------------
Thinking about it further, maybe just ";" and "/" are reasonable values which reflect current practices.
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
chrisjj
Posts: 5007
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1202 'Split multiple...' checkbox missing BUG#4938

Post by chrisjj »

Owyn wrote:Ummm. Perhaps the best solution is to require:
1) That the field always be present (i.e no checkbox),
I see nothing best about that. Best is to make good on the claim that the check box has been added, and hence on the long-standing promise to restore single-value operation.
Chris
Mizery_Made
Posts: 2283
Joined: Tue Aug 29, 2006 1:09 pm
Location: Kansas City, Missouri, United States

Re: 1202 'Split multiple...' checkbox missing BUG#4938

Post by Mizery_Made »

"Multi-Value" is supported by ID3, a subject which you voiced your opinion on several times in the past. So now that the MM Team have changed the way the tag is saved (which you suggested), and given people the option of how these split values are shown (which I believe you also voiced your opinion on, wanting "/" instead of ";" much like the actual saving of the tag), you now want them to offer to NOT do this?

Taking away Multi-Value support could have an impact on the database, as the artists have been split as they should in the Artists Table. Just because a checkbox was proposed in the description for that "Bug" doesn't mean it has to be included in the final product, plans often change in the process of finalizing the idea and putting the feature in, it was surely figured out along the line that a checkbox is unneeded.

"Best is to make good on the claim that the check box has been added, and hence on the long-standing promise to restore single-value operation." Might I ask where this "long-standing" promise was made? I can't for the life of me recall any of the developers making this promise. However, if you want it so bad, what stops you from changing the splitting value to a character that appears nowhere in your files, then changing that character in your files back to "/" or ";" which then should give you exactly what you want. *Shrugs*
chrisjj
Posts: 5007
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1202 'Split multiple...' checkbox missing BUG#4938

Post by chrisjj »

> you now

What's with the "now"? I proposed it six months ago:
1170 Objection to proposal to release
http://www.mediamonkey.com/forum/viewto ... 67#p148467
Make the new behaviour with respect to ';' an option, off by default.
> want them to offer to NOT do this?

Um, they already have offered:
http://www.mediamonkey.com/forum/viewto ... 10#p149310
We really do see the need for change in this area, and we are planning on making it,
> Taking away Multi-Value support could have an impact on the database,
> as the artists have been split as they should in the Artists Table.

I sure do hope taking it away would have an impact on all the artists here that were split when they SHOULDN'T have been. However, note that the option under discussion is presented as for Appearence only.

> it was surely figured out along the line that a checkbox is unneeded.

I see nothing sure about that. Especially since it does not accord with the Mantis entry.

> Might I ask where this "long-standing" promise was made?

Sorry - s/promised/planned/

> what stops you from changing the splitting
> value to a character that appears nowhere in your files

The simple fact that I don't know of any character that appears nowhere in my existing 20K files and nowhere in the next 20K files I add.
Chris
Teknojnky
Posts: 5537
Joined: Tue Sep 06, 2005 11:01 pm
Contact:

Re: 1202 'Split multiple...' checkbox missing BUG#4938

Post by Teknojnky »

I am not sure that a check box is applicable, since this would appear to be a required entry.

Something must be used to separate multiple artists, I haven't tried it, but if you want the appearance of not having a separator, try a blank space. Since I have no idea what, if any consequences this would have on the library database, I would of course highly recommend making a backup copy before doing so.

You may or may not realize it, but I think your posts here and elsewhere on the board can sometimes be a bit confrontational.
Post Reply