View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004371 | MMW v4 | Burning / Disc Handling | public | 2008-02-06 12:35 | 2008-02-08 20:29 |
| Reporter | Ludek | Assigned To | |||
| Priority | urgent | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 3.0 | ||||
| Fixed in Version | 3.0 | ||||
| Summary | 0004371: "Table of Contents CD-ROM Z: not read successfully." message is annoying | ||||
| Description | By reading forums I found out that a lot of users complain that "Table of Contents CD-ROM Z: not read successfully." message is annoying: http://www.mediamonkey.com/forum/viewtopic.php?t=25943 http://www.mediamonkey.com/forum/viewtopic.php?t=25893 The question is "why we show this message"?? Because this error means that MediaMonkey tried to access the CD-ROM drive in order to find out whether there is an audio CD content, but it was not able to read TOC successfully, i.e. it found out that there is no audio CD inserted in the drive. But why user should be bothering by this message? I guess that there is no need to show this message. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 1135 | ||||
|
|
Assigning to Jiri for confirmation that I can remove this message completely. And for review whether this message could be useful in some situations, because Jiri initially add this message before I start working on MM. |
|
|
I agree that the message can be quite unnecessary in some cases. That said, it probably shouldn't be completely eliminated since it's good to know about problems when they happen. Currently, there are 4 instances of this message in CDUtils.pas. Based on the reports, particularly the third one causes problems and should be eliminated. The first and fourth seem to be quite useful as they are related to ripping. The second instance doesn't seem to be used at all, it's commented out in CDDBUtils.pas. |
|
|
Yes, that is exactly what I meant: To remove it only from GetInfoFromAudioCD() procedure. Fixed in build 1135. |
|
|
Verified no regressions. |
