[#IZPACK-24] Installer recursively removes install directory when cancelled. - jira.codehaus.org

IzPack

Installer recursively removes install directory when cancelled.

Created: 03/Mar/08 11:30 AM Updated: 31/Jul/09 08:55 AM Resolved: 07/Mar/08 04:52 PM

Return to search

Component/s:

Installer

Affects Version/s:

None

Fix Version/s:

4.0.0

Time Tracking:

Original Estimate:

Not Specified

Remaining Estimate:

0 minutes

Time Spent:

30 minutes

Environment:

Ubuntu Linux 7.1

Description

« Hide

When the installer is running and is processing the Install Pack to write the files to the disk, cancelling the process by clicking the close window button at the top right of the window brings up a dialog asking if you are sure that you want to cancel the installation.

If you select "Yes", the installation is cancelled and the directory that the software was installing to is removed along with all its files and sub-directories.

This is not really acceptable in that the user could, for example, realise that they have chosen the wrong directory to install to, cancel the installation and consequently lose data and directories.

Sort Order:

[ Permalink | « Hide ]

Julien Ponge added a comment - 03/Mar/08 11:43 AM

You are right, this behavior can cause problems.... This will be fixed by somebody for version 4.0.0.

If you have time, could you please investigate and make a patch?

Julien Ponge made changes - 03/Mar/08 11:43 AM

Field	Original Value	New Value
Fix Version/s		4.0.0 [ 14071 ]

Julien Ponge made changes - 07/Mar/08 04:43 PM

Assignee

Julien Ponge [ jponge ]

[ Permalink ]

Julien Ponge logged work - 07/Mar/08 04:48 PM

Time Worked:	30 minutes
Identified where the issue was, removed the unsafe code.

Julien Ponge made changes - 07/Mar/08 04:49 PM

Time Spent		30 minutes [ 1800 ]
Remaining Estimate		0 minutes [ 0 ]

[ Permalink | « Hide ]

Julien Ponge added a comment - 07/Mar/08 04:52 PM

The unsafe wiping code has been removed.

Thanks for reporting it.

Julien Ponge made changes - 07/Mar/08 04:52 PM

Resolution		Fixed [ 1 ]
Status	Open [ 1 ]	Resolved [ 5 ]

Julien Ponge made changes - 10/Mar/08 04:49 AM

Status

Resolved [ 5 ]

Closed [ 6 ]

[ Permalink | « Hide ]

Thomas Demande added a comment - 31/Jul/09 08:55 AM

Can I have an idea of what has been changed in the code?
We do have to get rid of this, but still having < 4.0.0 version, for Java 4 compatibility, so knowing what's to be done would greatly help!

Thanks