[ { "Id": "447792", "ThreadId": "213959", "Html": "

Hi, I have a code that generates XLS/CSV/XLSX file.
The code sends the file using ms.writeto(response.outputstream) to the client as a download dialog box.

I wanted to add a feature to download a compressed version of the file, so instead of sending the stream of the file, I zip it first and then send the stream of the zip file instead of the original XLS file.

I have been trying to do that but for some reason, using the code below, I get a ZIP file with an empty XLS file inside.

Please help me, I have been trying to solve it for few hours with no luck at all. 

Thanks,

\r\n

Lior

Response.ContentType = "application/x-zip-compressed"

Response.AddHeader("Content-Disposition", "attachment; filename=" + reportName + ".zip")

\r\n

Dim ms As MemoryStream = New MemoryStream

\r\n

If FileFormat = "CSV" Then

\r\n

ef.SaveCsv(ms, CsvType.CommaDelimited)

\r\n

ElseIf FileFormat = "XLS" Then

\r\n

ef.SaveXls(ms)

\r\n

ElseIf FileFormat = "XLSX" Then

\r\n

ef.SaveXlsx(ms)

\r\n

End If

\r\n

ef.SaveXlsx(ms)

\r\n

Using zip As New ZipFile()

\r\n

zip.CompressionLevel = Ionic.Zlib.CompressionLevel.BestCompression

\r\n

Dim entry As ZipEntry = zip.AddEntry("report.xls", ms)

entry.SetEntryTimes(Now(), Now(), Now())

\r\n

zip.Save(Response.OutputStream)

\r\n

End Using

", "PostedDate": "2010-05-26T11:54:30.183-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "447931", "ThreadId": "213959", "Html": "

add

\r\n
ms.Seek(0, SeekOrigin.Begin)\r\n
\r\n

before you call

\r\n
using zip as New ZipFile()\r\n
\r\n

 

", "PostedDate": "2010-05-26T19:34:43.63-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "447964", "ThreadId": "213959", "Html": "

Worked like a charm.

Thank you very much Cheeso. :)

", "PostedDate": "2010-05-26T22:12:06.057-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "448114", "ThreadId": "213959", "Html": "

You could simplify the whole thing and eliminate the use of the temporary MemoryStream by using the WriteDelegate syntax of AddEntry. It was designed specifically for your situation.

\r\n
\r\n
Private Sub WriteEntry (ByVal filename As String, ByVal output As Stream) \r\n    If FileFormat = "CSV" Then\r\n        ef.SaveCsv(stream, CsvType.CommaDelimited)\r\n    ElseIf FileFormat = "XLS" Then\r\n        ef.SaveXls(stream)\r\n    ElseIf FileFormat = "XLSX" Then\r\n        ef.SaveXlsx(stream)\r\n    End If\r\nEnd Sub\r\n\r\nPublic Sub Run()\r\n\r\n    Response.ContentType = "application/x-zip-compressed"\r\n    Response.AddHeader("Content-Disposition", "attachment; filename=" + reportName + ".zip")\r\n    Using zip As New ZipFile()\r\n        zip.CompressionLevel = Ionic.Zlib.CompressionLevel.BestCompression\r\n        Dim entry As ZipEntry = _\r\n            zip.AddEntry("report.xls", New WriteDelegate(AddressOf WriteEntry))\r\n        entry.SetEntryTimes(Now(), Now(), Now())\r\n        zip.Save(Response.OutputStream)\r\n    End Using\r\n\r\nEnd Sub\r\n
\r\n
\r\n

But your original approach will work fine.

", "PostedDate": "2010-05-27T04:34:13.05-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "450204", "ThreadId": "213959", "Html": "

I'm having a similar problem.  I use the following function to zip a set of files and return a MemoryStream to an Attachment object to be added to an Email object.  The zip file attaches OK but when I open it I find that it is empty.  If I Save() the zip file to a directory then it contains the files.  Any ideas?

\r\n
\r\n
        public Stream ZipFiles(string[] files)\r\n        {\r\n            MemoryStream stream = new MemoryStream();\r\n            stream.Seek(0, SeekOrigin.Begin);\r\n            using (ZipFile zip = new ZipFile())\r\n            {\r\n                foreach (String filename in files)\r\n                {\r\n                    ZipEntry e = zip.AddFile(filename);\r\n                }              \r\n                //zip.Save(@"test.zip");\r\n                zip.Save(stream);\r\n            }\r\n            return stream;\r\n        }\r\n
\r\n
\r\n

Thanks!

\r\n

Ian

", "PostedDate": "2010-06-01T11:55:06.637-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "450211", "ThreadId": "213959", "Html": "

yes - Your .Seek() should come AFTER you save the zipfile to the stream.

", "PostedDate": "2010-06-01T12:10:09.677-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "450226", "ThreadId": "213959", "Html": "

That did it.  Thanks, Cheeso!  What threw me off was your first comment at the top of this thread.  I read it again and it still seems to me that it suggests putting the Seek() before the "using" which would also put it before the Save().  Hopefully this will help others from getting confused also.

\r\n

Thanks for the great utility!

\r\n

Ian

", "PostedDate": "2010-06-01T12:38:00.563-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "450243", "ThreadId": "213959", "Html": "

Well in the situation at the top of the thread, the memorystream was the *source* of the data or a ZipEntry. 

\r\n

In your case the MemoryStream was the *destination* of the ZipFile.

\r\n

In either case, the situation was similar:  after writing to the MemoryStream, you must invoke MemoryStream.Seek() before trying to Read it.

\r\n\r\n

 

", "PostedDate": "2010-06-01T12:59:30.87-07:00", "UserRole": null, "MarkedAsAnswerDate": null } ]