[
{
"Id": "98195",
"ThreadId": "29471",
"Html": "First - thanks to Chesso for the great work and effort thus far.
\r\n
\r\nI need to add a large amount of files to a zip file (the file sizes are very small), but I need to verify which files get added successfully. I am unable to add files successfully to an existing .zip archive (using version 1.5.0.7). Simple example:
\r\n
\r\n Using oZip As New ZipFile("D:\\Data_Archive.zip") 'Create initial archive
\r\n oZip.AddItem("D:\\Data.bin", "")
\r\n oZip.Save()
\r\n End Using
\r\n
\r\n Using oZip2 As New ZipFile 'Start adding new files to zip file.
\r\n oZip2.UpdateItem("D:\\Data2.bin", "")
\r\n oZip2.Save("D:\\Data_Archive.zip")
\r\n End Using
\r\n
\r\nThis code generates "Unable to move the replacement file to the file to be replaced. The file to be replaced has retained its original name.". The temp directory is set correctly. I am I overlooking something simple, or should I be going about this a different way? Any suggestions are welcome.
\r\n
\r\nJon
\r\n",
"PostedDate": "2008-06-11T09:05:47.25-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "98206",
"ThreadId": "29471",
"Html": "Hey JonBoy,
\r\n
\r\nYes, this may be a bug.
\r\n
\r\nI think the error you are getting is telling you that that the original zip file "D:\\Data_Archive.zip" is not being overwritten.
\r\nThe documentation for the Save("filename") method says: "Existing files will be overwritten with great prejudice." It seems this is not the actual behavior.
\r\n
\r\nBut in addition to the bug, I think there is some confusion on your part regarding how to use the library.
\r\n
\r\nYour second stanza opens a new ZipFile, a totally new zipfile. The no-argument constructor you use (New ZipFile) says - this is going to be a new zip file.
\r\nSo in fact you are not Updating any entries in an existing zipfile. You are creating a totally new, independent zip file.
\r\nAnd by calling UpdateItem() on that totally new zipfile, you are adding the item to the zipfile.
\r\n
\r\nOk, then you try to save into a zip archive that already exists. The file is not being overwritten. This is the bug.
\r\n
\r\nif you want to update a zip file you would do something like this:
\r\n
\r\n Using oZip2 As New ZipFile 'Start adding new files to a new zip file.
\r\n oZip.AddItem("D:\\Data.bin", "")
\r\n oZip2.Save("D:\\Data_Archive.zip")
\r\n End Using
\r\n
\r\n Using oZip As New ZipFile("D:\\Data_Archive.zip") 'Open the existing
\r\n oZip2.UpdateItem("D:\\Data2.bin", "")
\r\n oZip.Save()
\r\n End Using
\r\n
\r\nOr, another option is like this:
\r\n
\r\n Using oZip2 As New ZipFile("D:\\Data_Archive.zip") 'open a new or existing archive.
\r\n oZip.UpdateItem("D:\\Data.bin", "")
\r\n oZip2.Save("D:\\Data_Archive.zip")
\r\n End Using
\r\n
\r\n Using oZip As New ZipFile("D:\\Data_Archive.zip") 'open the existing archive
\r\n oZip2.UpdateItem("D:\\Data2.bin", "")
\r\n oZip.Save()
\r\n End Using
\r\n
\r\n",
"PostedDate": "2008-06-11T09:51:41.483-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "98207",
"ThreadId": "29471",
"Html": "Ahh, the bug I think has to do with using a different volume for the temp file, versus the final file.
\r\n
\r\nThis is workitem 5284
\r\n
\r\nhttp://www.codeplex.com/DotNetZip/WorkItem/View.aspx?WorkItemId=5284
\r\n",
"PostedDate": "2008-06-11T09:53:49.07-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "98218",
"ThreadId": "29471",
"Html": "Thanks for the quick response Cheeso.
\r\n
\r\nI did notice that line in the documentation as well, but it seemed to go against what I found in discussion http://www.codeplex.com/DotNetZip/Thread/View.aspx?ThreadId=28509 when the functionality was added.
\r\n
\r\nThe clarification on the ctors and .save method for new vs. existing archives makes sense now. Thanks for creating the work item for the .temp file and archive being on a different directory.
\r\n
\r\n
\r\n",
"PostedDate": "2008-06-11T10:41:53.333-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "98232",
"ThreadId": "29471",
"Html": "I did notice that line in the documentation as well, but it seemed to go against what I found in discussion http://www.codeplex.com/DotNetZip/Thread/View.aspx?ThreadId=28509 when the functionality was added.
\r\nWe seem to have a little foncusion here.
\r\nThe discussion
\r\nhttp://www.codeplex.com/DotNetZip/Thread/View.aspx?ThreadId=28509
\r\n
\r\nis referring to updating entries within a zipfile. The line in the documentation you are commenting on - I think you mean the part that says - "Existing files will be overwritten with great prejudice." deals with updating a zip archive on the disk.
\r\n
\r\nThey are related but not quite the same thing. What I mean is -
\r\na common scenario will be to read in an existing ZipFile, then update items in the zip file, then save the zipfile to the disk. I think this is what you were trying to do, JonBoy.
\r\nThe discussion at 28509 said we have now added UpdateFile() and UpdateItem() as methods on the ZipFile class. That takes care of "updating items in the zip file". The thing you are doing is saving the updated file, back to the same place. This is the part that is covered by "Existing files will be overwritten with great prejudice."
\r\n
\r\nExcept there was a bug dealing with multiple volumes (C:\\ versus d:\\).
\r\n
\r\nAs I said I put a fix into the source for this. If you want to workaround the problem with an existing binary, set the TempFileFolder to be on a disk that is the same as the disk you are saving your ZipFile to. Eg, d:\\temp. If you want the fix, you can build the latest source yourself or wait for a new binary from me.
\r\n
\r\nMake sense?
\r\n",
"PostedDate": "2008-06-11T11:56:40.777-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "98237",
"ThreadId": "29471",
"Html": "I understood the different volume issue from your previous response (I changed my files\\directories to C: and verified that). Already copied down the latest source.
\r\n
\r\n>> The thing you are doing is saving the updated file, back to the same\r\nplace. This is the part that is covered by "Existing files will be\r\noverwritten with great prejudice."
\r\nRoger that Capt! Understood.
\r\n",
"PostedDate": "2008-06-11T12:22:52.857-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
}
]