View Issue Details

IDProjectCategoryView StatusLast Update
0010800MMW v4Main Panelpublic2013-09-21 04:27
Reporterlowlander Assigned To 
PriorityhighSeverityminorReproducibilityalways
Status closedResolutionno change required 
Product Version4.1 
Summary0010800: in-place editing keeps ;; in multi-value field (regression)
DescriptionWhen having 2 ; in a multi-value field (either due to manual entering or removing a value between 2 ;) while in-place editing MediaMonkey won't remove ;; while it will do so in the Properties dialog.
TagsNo tags attached.
Fixed in build

Activities

Ludek

2013-06-19 14:51

developer   ~0036560

Last edited: 2013-06-19 14:52

I see the same for both filelisting edits and edits via Properties.

In both cases ;; is accepted. By watching DUnit tests it was intentional:


CheckEquals( 'AB;;BA; Beatles', VisualString2MultiString('AB;;BA; Beatles'));
CheckEquals( 'AB;;BA; Beatles', VisualString2MultiString('AB;BA| Beatles'));
CheckEquals( 'Big; Test; Of; Man;;y; Artists', VisualString2MultiString(' Big | Test|Of| Man;y | Artists '));

i.e. from the tests it looks that double ;; is escaped ; when ; is used also as separator.

lowlander

2013-06-20 19:07

developer   ~0036590

I see an inconsistency.

In Properties > Artist Seal;; D 12 changes to Seal; D 12
In Filelisting (Now Playing node) > Artist Seal;; D 12 stays as Seal;; D 12

Ludek

2013-06-21 10:44

developer   ~0036599

For me artist(s) 'Seal;; D 12' doesn't change to 'Seal; D 12' in file properties. I can enter double ;; and after clicking [OK] and reopening properties the double ;; is still there.

Couldn't it be just auto-complete feature that is causing the issue for you? You can verify by pressing space key until the auto-complete disappears.

lowlander

2013-06-26 18:49

developer   ~0036664

Adding spaces in Properties does leave it as Artist Seal;; D 12 so I guess this can be chalked up to inconsistent auto-complete handling in MediaMonkey.

Ludek

2013-07-01 08:40

developer   ~0036691

OK, resolving as it was auto-complete issue.

peke

2013-09-21 04:27

developer   ~0037626

Verified 1659