View Issue Details

IDProjectCategoryView StatusLast Update
0000917MMW v4Burning / Disc Handlingpublic2005-11-01 15:50
Reporterrusty Assigned To 
PriorityurgentSeverityfeatureReproducibilityalways
Status resolvedResolutionfixed 
Summary0000917: Burn Audio CD (first page of wizard related to Audio CD)
DescriptionThis bug description is deprecated but is left for historical reasons. The updated spec is in comment 5987 in this bug.
------------------------------------------------------------------------------

This functionality is a component of the Burn CD Wizard, and is accessed when the user chooses to burn an Audio CD from within the first panel of the wizard. It appears as follows:

------------------------------------------------------------
| Burn CD Wizard - Audio CD
|-----------------------------------------------------------
| --Audio-------------------------------------------------
| | [x] Apply Volume Levelling
| | [ ] Apply active plug-ins
| | [ ] Insert 2 second gap between tracks
| --------------------------------------------------------
|
| --[x] Write CD-Text-------------------------------------
| |
| | CD Title: __________________________________________
| | CD Artist: _________________________________________
| --------------------------------------------------------
|
| --Burn Options------------------------------------------
| | Drive: ________v . . . . [ ] Eject CD after burn
| | Speed: ________v . . . . --Action-------------------
| | Method: _______v . . . . | [ ] Simulate Burn
| | Finalization: __v . . . . | [ ] Write CD
| | . . . . . . . . . . . . . .-------------------------
| --------------------------------------------------------
|-----------------------------------------------------------
| Total: xx KB, yy:zz [Options] . . [Back] [Burn] [Cancel]
------------------------------------------------------------

If the user clicks [Burn], then MM checks for a CD and begins burning it if possible. If not possible it should report meaningful errors for:
-No CD has been detected. Please insert a CD to erase.
-The inserted CD is not empty. Do you want to erase it first?
-The inserted CD cannot be written on. Do you want to erase it first?
-The inserted CD cannot be written on. Please insert another CD.
-The inserted CD has x MB available, and you need y MB. Do you want to erase it?
-The inserted CD has x MB available, and you need y MB. Insert another CD.
-etc.... (please document any other errors that should be tested)

If a CD is writable, a small dialog indicating the operations status should be displayed. The window should have a progress bar and a time indication of how much longer is needed. Once complete, the progress bar should be full and the dialog should indicate 'Burn CD Operation Complete. [Ok]'. Clicking [Ok] should return the user to the Playlist Editor.

Notes:
-Volume Leveling: see related information in bug #698

-'Apply active plug-ins' records the tracks exactly as they would have sounded had they played on MM with the currently active plug-ins

-CD-text section should be disabled if not supported by the active drive.

-'Drive' drop down contains drive that's automatically detected by the burning engine. It is active only when more than a single burner is in the system.

-'Speed' only lists valid speeds associated with the chosen drive. Default to highest supported speed.

-'Method': [Disk at once]; Track at once

-'Finalization': [Finalize]; Allow further writing

-Note: the current dialog does NOT give a warning when the amount of audio stored to the CD > x minutes. All that happens is that the time is bolded and in blue!! What should happen is that a warning symbol appears under the Total Time indicator saying (in bold red) 'Exceeds CD Capacity!'
Additional InformationParent bug: 0000058 CD Burning Wizard

2-second delay requirement: http://www.songs-db.com/forum/viewtopic.php?t=2211
TagsNo tags attached.
Fixed in build910

Activities

rusty

2004-01-26 04:18

administrator   ~0002893

pushedfrom22

jiri

2005-10-09 06:56

administrator   ~0005977

Last edited: 2005-10-11 03:03

What exactly from this bug needs to be implemented? It seems to me that all should remain as is, only:

1. Space used/available/total should not use B/MB/GB as units but should use minutes instead.

<RS> Yes

2. [ ] Apply active plug-ins doesn't seem to be that critical. Anyway, shouldn't it be [ ] Apply active DSP plug-ins?

<RS> Yes

3. [ ] Insert 2 second gap between tracks - this is actually the difference between DAO and TAO, isn't it?

<RS> From what I see, this isn't the case--silence removal can apply to both types of write methods.

4. CD-text section should be disabled if not supported by the active drive. - problem is that in that moment CD-drive isn't selected yet (in case user has >1).

<RS> My new proposal will deal with this nicely.

5. Re. Note: the current dialog does NOT give a warning when the amount of audio stored to the CD > x minutes. All that happens is that the time is bolded and in blue!! What should happen is that a warning symbol appears under the Total Time indicator saying (in bold red) 'Exceeds CD Capacity!'
   - This should probably happen on the last screen of the wizard when user did already choose CD drive.
  
<Agreed>

rusty

2005-10-11 03:09

administrator   ~0005987

Please ignore previously suggested implementations in this bug. i.e. to solve this bug, all that is required is to modify the first panel of the Disc Burning Wizard that pertains to Audio CDs.

The proposed implementation deals with the issues that Jiri described and makes the wizard consistent with the changes made on the data burning elements of the wizard:

------------------------------------------------------------------------------
| Disc Burning Wizard
|-----------------------------------------------------------------------------
| Audio CD Tracks and Format
| Drag and drop the tracks if you wish to change the order in which they appear
| on the CD, and set the audio characteristics of the CD.
|------------------------------------------------------------------------------
| CD Layout:
| --------------------------------------------------
| | Title . . . . . . . . .Artist . . . . . Length .|
| | blah . . . . . . . . . .blah . . . . . . Blah . |
| | blah . . . . . . . . . .blah . . . . . . Blah . |
| | blah . . . . . . . . . .blah . . . . . . Blah . |
| | blah . . . . . . . . . .blah . . . . . . Blah . |
| | blah . . . . . . . . . .blah . . . . . . Blah . | [ ] Level Volume
| | blah . . . . . . . . . .balh . . . . . . Blah . | [ ] Apply active DSP plug-ins
| ---------------------------------------------------
|----------------------------------------------------------------------------
| Total time: xx min. yy sec. . . . . . . . . . . . . . . . . . .[Back] [Next] [Cancel]
-----------------------------------------------------------------------------

The changes above are:
1) The title and text look and feel and content are modified (consistent with data cd wizard)
2) The 'Total time:' indicator location is made consistent with the data CD wizard
3) Level volume wording is simplified
4) Apply active dsp plugins is added
5) If xx:yy > CD capacity then it should appear as 'xx min. yy sec ! (exceeds CD capacity)' in red

jiri

2005-10-11 04:49

administrator   ~0005992

4) To Ludek: Might be harder to implement, we'll probably need to discuss this.

peke

2005-10-11 20:10

developer   ~0006000

Last edited: 2005-10-11 20:15

4. I have valid working solution, whith who I need to talk to or should I write proposal.

To Jiri: Main problem here is that WinAPI do not allow one DLL to be loaded twice. if it is all ready loaded in Memory for that program in this case MM. Will talk over IM if needed.

Ludek

2005-11-01 15:49

developer   ~0006139

1,2,3,5 fixed in build 910.

4) remains to do.