Bugs item #1604873, was opened at 2006-11-28 17:37
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=642714&aid=1604873&group_id=105970

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: extensions
Group: v3.0
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Rob (rmacfadyen)
Assigned to: Scott Kurtzeborn (scotk)
Summary: SqlExtension will incorrectly drop an existing database!

Initial Comment:
The SqlExtension will delete an existing database on rollback (this is an 
existing bug). However... if the ConfirmOverwrite attribute is set to "yes" and 
the user answers "No" to the prompt "Database X already exists do you wish to 
continue" the install is halted and _rolled back_.

This results in the database being deleted!

Here's the xml:

<Sql:SqlDatabase Id="db01" 
  Server="." Database="Payroll"
  CreateOnInstall="yes" ConfirmOverwrite="yes"
  DropOnUninstall="no"
/>

Deleting an existing database without explicit instructions to do so is a 
_very_ bad thing. On a production SQL server the database could be quite 
important (eg. DeveloperPayrollDb) and possibly not backed up. This could 
result in a serious data loss situation (just an ugly scenario).

The SQL extension may need to record its intention to create a database, and 
then the rollback "drop database" would only be invoked if there was an 
intention to create a database. Maybe this means the check of an existing 
database and confirm overwrite will need to move into an earlier phase (where 
it can write a property).


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=642714&aid=1604873&group_id=105970

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
WiX-devs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wix-devs

Reply via email to