View Issue Details

IDProjectCategoryView StatusLast Update
0000918MMW v4Burning / Disc Handlingpublic2005-10-24 04:47
Reporterrusty Assigned To 
PriorityurgentSeverityfeatureReproducibilityalways
Status closedResolutionfixed 
Fixed in Version2.5 
Summary0000918: Erase CD Wizard
DescriptionThis is part of the Burn CD wizard. It provides basic erase functionality as follows:

--------------------------------------------------------------
| Burn CD Wizard - Erase CD
|-------------------------------------------------------------
| Choose erase method:
| (o) Normal erase
| (o) Quick erase
|
|
|
|-----------------------------------------------------------
| Total: xx KB, yy:zz [Options] [Back] [Erase] [Cancel]
------------------------------------------------------------

If the user clicks [Erase], then MM checks for a CD and erases 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 erasable. Please try another one.

If a CD is erasable, 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 'Operation Complete'. Clicking [Ok] should return the user to the Burn CD wizard's first page.
Additional InformationParent bug: 0000058 CD Burning Wizard
TagsNo tags attached.
Fixed in build904

Activities

rusty

2004-01-26 04:04

administrator   ~0002890

pushedfrom22

Ludek

2005-10-07 23:54

developer   ~0005964

Last edited: 2005-10-07 23:56

It has been done, but there is a technical problem that OnEraseProgress method is called from a thread different from the thread the IDevice13::Erase13 method has been called.
-> it causes The progress bar doesn't show a percentage although we can write up percentage by using debug view.

We have it discussed with Jiri and there is a need for finding the right thread and it can be hard to implement.

Rusty, please review this bug and eventually assign to Jiri for fixing the described problem.

rusty

2005-10-09 02:58

administrator   ~0005968

The erase functionality seems to work, however, there are a couple of problems:

1) The fact that there is no visual feedback of the progress of the erasing is a critical issue. When I first tried it, I thought that it simply wasn't working, so I tried:
a) to terminate the process, which didn't work
b) to manually remove the CD--which caused the machine to need to be rebooted, because although I didn't realize, the CD _was_ being erased and manual removal during this process caused the CD drive to stop working unless I rebooted.

2) When the user enters the erase function, the wizard terminates after the erase operation is complete or after an error has occured. This is very annoying because the user had accessed the erase function after having selected which tracks they want to burn, but when they try to erase the CD, the wizard closes and the user then has to make his selection again.

e.g.
-User selects 100 tracks to burn & clicks burn
-User inserts CD, but realizes that there's insufficient space so he backs up to the main screen and accesses the erase functionality
-An error occurs because the user tried to erase a CD-R
-->Wizard closes!!

I'm leaving this bug open rather than opening a new one, because I don't think that this feature can be considered complete in it's current state.

peke

2005-10-09 09:20

developer   ~0005979

Regarding Erase CD Rusty You are right. That was same problem With CDRWin older versions, also with nero older versions.

Both firms solved the problem by Locking the drive eject button during erase.

One more note: Some of Burners do not return valid percentage of erase process finished like TEAC CDR-W540E which results in Nero for example that erase progress bar jumps from 50% to 100% within Sec. on the end of erase. Also NEC DVD-RW ND-3540 return valid info for either erase or data disc burning. May I suggest that there we do not use progress but just like Nero estimated time to complete "Less than Minute"

peke

2005-10-09 16:41

developer   ~0005980

If CD-Rom Door is opened and I initiate Erase CD. It is not checked for open door so MM exits with "No CD has been detected. Please Insert CD to erase." then closes Erasing. Also in some cases "Disc Burning Wizard - Erase settings" do not show all of mine Bureners in the list just one that do not have Eraseable CD in it and CD that needs to be erased is in Other one. After few restarts of Erase CD all are shown.
I suggest that "Disc Burning Wizard - Erase settings" are shown before any testing of available CDs for erase. This way It will be checked for exact CD-Rom user wants to use in erase process and also check for Ejected CD Tray can be made for desired CD-Rom. Process will look something like:
            a) Eject CD Tray for Selected CD-Rom In Case CD is not Writable or selected CD-Rom is Empty (This way MM will check for Ejected Tray)
            b) Show Dialog with text "Please insert empty or rewritable CD/DVD!" and options "OK" and "CANCEL"

jiri

2005-10-10 18:34

administrator   ~0005986

1) I fixed this in SVN revision #427. Ludek should review if it works for Erasing as expected.

2) Remains for Ludek.

Ludek

2005-10-20 16:06

developer   ~0006023

2) I have resolved it by implementing Multiple CD/DVD burning in build 902, because after [!Burn] button clicking you will be asked for erasing CD/DVD RW which isn't blank. You can answer what you want - yes or no. This feature is implemented for every type of burning.

Checking for Ejected Tray is implemented as Peke suggested.
Checking for Ejected Tray is implemented for burning data/audio disc too.

rusty

2005-10-21 10:22

administrator   ~0006028

Last edited: 2005-10-21 10:25

Tested in 902. Disc erase status works nicely. As far as other issues, there are still some usability/cosmetic issues:

2)
a) When the user clicks 'Erase CD' from the front page of the wizard, any content that has been selected is lost. What should happen is that the 'Erase CD' action occurs without ever exiting the wizard completely OR that when the wizard is exited, selection isn't lost. That way the user can proceed with a burn without having lost their selection.

3) Once the user clicks 'Burn' the workflow is a bit confusing in the sense that multiple dialogs appear and the user is unclear on why the dialogs have appeared and on what dialogs will appear next. For example here's what happens when I try to burn a CD-RW that is already finalized:
-Click 'Burn'
a)-->Selected Tracks exceed available disc space. Do you want to proceed?
-Click 'OK'
b)-->The inserted CD/DVD is rewritable. Do you want to erase it first?
-Click 'No'
c)-->The inserted CD/DVD cannot be written on. Do you want to erase it first? + four buttons that are a bit confusing about what they do.

This could be simplified as follows:
a)-->The selected tracks exceed available disc space. Do you want to proceed?
-Click 'OK'
b)If unwriteable CD-RW
-->The inserted CD/DVD cannot be written on unless it is first erased. Do you want to erase it?
[Try another disc] [Erase disc] [Cancel]

If CD-RW contains content already
-->The inserted CD/DVD is not empty. Do you want to erase it before burning?
[Try another disc] [Erase disc] [Cancel]

c) There is no longer a need for this step!

4) Cosmetic issues (some of these have already been addressed in 3):
a)
The inserted CD/DVD is rewritable. Do you want to erase it first?
-->
The inserted CD/DVD is not empty. Do you want to erase it before burning?

b)
The inserted CD/DVD cannot be written on. Do you want to erase it first?
-->
The inserted CD/DVD cannot be written on unless it is first erased. Do you want to erase it?

c) No disc is available. Please insert a disc to be burned...
-->
Please insert a disc to be burned...

d) Status Messages: No mention of 'flushing cache' needs to be made--it sounds like a technical detail that is irrelevant to the user

e) Status message: '(Waiting for a medium)' --> '(Waiting for a new disc)'

Ludek

2005-10-22 18:15

developer   ~0006055

3) and 4) have been fixed in build 904.
Marked as resolved for testing.

rusty

2005-10-24 04:47

administrator   ~0006063

Verified 904.