View Issue Details

IDProjectCategoryView StatusLast Update
0016754MMW 5FileMonitor / Find Missingpublic2020-06-30 23:38
Reporterrusty Assigned To 
PriorityurgentSeveritytweakReproducibilityalways
Status closedResolutionfixed 
Product Version5.0 
Target Version5.0Fixed in Version5.0 
Summary0016754: Clarify 'Failed to import' value in scan results dialog
DescriptionAs described at 0016738, the scan results dialog is a bit misleading because 'Failed to import <x> files' doesn't clearly distinguish why files failed to import vs 'Did not import <x> files (did not match import criteria)'. A new string '(invalid format)' could clarify this. e.g.

Failed to import <x> files (invalid format)

note: this wording assumes that any File Type that has been checked off to be scanned, but that fails to scan for whatever reason will be counted in this category.
TagsNo tags attached.
Fixed in build2259

Relationships

related to 0016738 closedLudek MMW v4 Non Media Files should not be calculated as skipped (regression) 
related to 0002437 closedjiri MMW v4 Some MP3 files cannot be scanned by MediaMonkey 

Activities

peke

2020-06-26 16:51

developer   ~0058669

Note: that if usr do not have access rights to read files they are still Fail To import even they are not Invalid format.

Ludek

2020-06-26 20:13

developer   ~0058671

Last edited: 2020-06-26 20:14

I agree with Peke that the "(invalid format)" appendix isn't needed at all as reasons for failed imports are various. e.g. access rights, AV when reading tag because of non-standard value, unexpected format or tag, unexpected audio data, etc.

So my understanding is that:
- only the files with extension not configured to scan (and too short files) should be listed as "did not match import criteria"
- others files that failed to scan for whatever reason should be listed as "failed to import"

Ludek

2020-06-26 20:25

developer   ~0058672

Last edited: 2020-06-26 20:27

My test results are different from Peke's tests results in 0016738 - item 2 (I tested with MM5 though)

I created test folder with:
- one standard MP3 file
- one zero bytes MP3 file (empty txt file with MP3 extension)
- one txt file including "bla bla" renamed to MP3 extension

=> all three files were imported to MM5 library.

Expected: only the first file should be imported and the others two should be listed as "failed to import".

michal

2020-06-29 08:12

developer   ~0058675

So we are changing behavior introduced long time ago in the issue 0002437.
I have no idea, why it was introduced at all, for me, importing empty/corrupted files also does not make sense.

michal

2020-06-29 12:28

developer   ~0058676

Fixed in build 2259. Files in wrong format now fail with error "error while reading file info".

peke

2020-06-30 23:38

developer   ~0058704

Verified 2259

Now it looks more reasonable.

Test note: along with Ludek tests in 0016754:0058672 I have also tested with

WAV,FLAC, OGG to MP3 and all return as Fail to import which is correct and now when saving Results Log it is clear where and what files fail.