[ { "Id": "544616", "ThreadId": "240647", "Html": "\r\n

Hi,

\r\n

I am fairly new to Asp.net (with C#) and have a GridView that displays files in a folder on the server along with a checkbox template field. I am trying to use DotNetZip to add selected files from the GridView to a zip file for the user to download. Below\r\n is the code I have so far, but the zip file it creates appears to be empty.

\r\n

GridView Markup:

\r\n

\r\n
\r\n
<asp:GridView ID="gvFiles" runat="server" AutoGenerateColumns="false">\r\n    <Columns>\r\n        <asp:TemplateField>\r\n            <ItemTemplate>\r\n                <asp:CheckBox ID="chkSelect" runat="server" />\r\n            </ItemTemplate>\r\n        </asp:TemplateField>\r\n        <asp:TemplateField HeaderText="File Name">\r\n            <ItemTemplate>\r\n                <asp:Label ID="lblFileName" runat="server" Text='<%# Eval("FileName") %>' />\r\n                <asp:Label ID="lblFilePath" Visible="false" runat="server" Text='<%# Eval("FilePath") %>' />\r\n            </ItemTemplate>\r\n        </asp:TemplateField>\r\n    </Columns>\r\n</asp:GridView>\r\n<asp:Button ID="btnDownload" runat="server" Text="Download" \r\n    onclick="btnDownload_Click" />\r\n
C# Code-Behind:
\r\n
\r\n
\r\n
using System;\r\nusing System.Text;\r\nusing System.IO;\r\nusing System.Collections.Generic;\r\nusing System.Linq;\r\nusing System.Web;\r\nusing System.Web.UI;\r\nusing System.Web.UI.WebControls;\r\nusing Ionic.Zip;\r\n\r\npublic partial class DownloadFiles : System.Web.UI.Page\r\n{\r\n    protected void Page_Load(object sender, EventArgs e)\r\n    {\r\n        var files = Directory.GetFiles(Server.MapPath("~/Uploads/Folder3"));\r\n\r\n        gvFiles.DataSource = from f in files\r\n                             select new\r\n                             {\r\n                                 FileName = Path.GetFileName(f),\r\n                                 FilePath = f\r\n                             };\r\n        gvFiles.DataBind();\r\n    }\r\n    protected void btnDownload_Click(object sender, EventArgs e)\r\n    {\r\n        var filePath = string.Empty;\r\n\r\n        string downloadFileName = "Files.zip";\r\n        Response.ContentType = "application/zip";\r\n        Response.AddHeader("content-disposition", "filename=" + downloadFileName);\r\n\r\n        using (ZipFile zip = new ZipFile())\r\n        {\r\n            foreach (GridViewRow row in gvFiles.Rows)\r\n            {\r\n                CheckBox cb = (CheckBox)row.FindControl("chkSelect");\r\n                filePath = (row.FindControl("lblFilePath") as Label).Text;\r\n\r\n                if (cb != null && cb.Checked)\r\n                {\r\n                    zip.AddFile(Server.MapPath(filePath), "Files");\r\n                }\r\n            }\r\n            zip.Save(Response.OutputStream);\r\n        }\r\n    }\r\n}\r\n
Can anyone see how I could get this to work? I would appreciate any help.

Thanks,


James
\r\n
\r\n

\r\n", "PostedDate": "2011-01-05T17:56:32.567-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "544999", "ThreadId": "240647", "Html": "\r\n

James - a couple things for you.

\r\n

1. There's an example in the DotNetZip source distribution that  zips up a file based on checkbox status, and then downloads that zipfile to the browser.  You may wanna check it out.  It does almost exactly what you are describing.

\r\n

2. If I was debugging this, I would attach to the ASPNET process with the debugger and step through the code so I could see it clearly.  I'd also want to wrap the zip creation logic in a try..catch... to handle exceptions.   I might also use\r\n the Fiddler2 tool to examine the HTTP request and response on the wire, to make sure it looks like I expect. 

\r\n

What you are trying to do should work.   Nothing in your code jumps out at me as obviously wrong.   It could very well be a logical error in your code - if you are getting a valid, correct zip file, but the problem is, the zip is empty,\r\n then ... maybe you are not adding the files you thought you were adding.  If on the other hand you are seeing an empty response, that is something different.   

\r\n

Good luck to you.

\r\n

-Cheeso

\r\n", "PostedDate": "2011-01-06T11:18:22.387-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "803986", "ThreadId": "240647", "Html": "

Anyone figure this out??  I am needing a very similar solution but in ASP.NET VB.

\r\n

Thanks,
Andrew

", "PostedDate": "2012-03-01T13:03:14.813-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "804180", "ThreadId": "240647", "Html": "

I tested this code. the only problem is with this line

\r\n
 zip.AddFile(Server.MapPath(filePath), \"Files\");

There is no need to use Server.MapPath as Filepath is already having a physical path so use like this:

zip.AddFile(filePath, \"Files\");

If this does not work then paste you snippet.

Please let me know in case of any concerns.
\r\n
", "PostedDate": "2012-03-01T22:16:02.337-08:00", "UserRole": null, "MarkedAsAnswerDate": null } ]