Okay I have been over and over this all weekend and this darn thing just won't work. Here's all the code that I think matters.
\r\n\r\n
What happens is that I get the zip file, but the file is totally empty. It has the filenames, but they are 0 bytes. Why be this?
\r\n\r\n
/// <summary>\r\n /// Outputs the zip file stream to the response output stream\r\n /// </summary>\r\n /// <param name="docIDs">An array of document id's to include in the zip file.</param>\r\n private void SendZipResponse(int[] docIDs)\r\n {\r\n SPSite site = SPContext.Current.Site;\r\n Response.Clear();\r\n //set response parameters for the zip attachment\r\n Response.Buffer = false;\r\n Response.Expires = 0;\r\n Response.ContentType = "application/x-zip-compressed";\r\n String guid =Guid.NewGuid().ToString();\r\n string disHeader = string.Format("Attachment;Filename=\\"download-{0}.zip\\"", guid);\r\n Response.AppendHeader("Content-Disposition", disHeader);\r\n\r\n //create a lock as the zip output stream is not thread safe\r\n lock (lockObj)\r\n {\r\n try{\r\n using (ZipFile zip = new ZipFile())\r\n {\r\n hs = new Hashtable(docIDs.Length);\r\n\r\n foreach (int docID in docIDs)\r\n {\r\n //get a managed file for the binary file data \r\n //and metadata\r\n ManagedFile mf = new ManagedFile(site);\r\n mf.ID = docID;\r\n mf.Load();\r\n //get a bytes array from the document \r\n\r\n hs.Add(Path.GetFileName(mf.URL), mf);\r\n zip.AddEntry(Path.GetFileName(mf.URL), WriteEntry);\r\n \r\n }\r\n zip.Save(Response.OutputStream);\r\n Response.OutputStream.Flush();\r\n }\r\n }\r\n catch (Exception ex){\r\n Console.Out.Write("storp");\r\n }\r\n \r\n Response.Close();\r\n\r\n }\r\n }\r\n\r\n private void WriteEntry(String filename, Stream output)\r\n {\r\n ManagedFile value = (ManagedFile)hs[filename];\r\n output = new MemoryStream(value.GetBinary());\r\n\r\n }\r\n\r\n
\r\n
and then in the ManagedFile class:
\r\n\r\n
/// <summary>\r\n /// Returns a byte array of the underlying file data\r\n /// </summary>\r\n /// <returns>An array of bytes</returns>\r\n public byte[] GetBinary()\r\n {\r\n //open the web based on the managed file's web property\r\n using (SPWeb web = activeSite.OpenWeb(this.Web))\r\n {\r\n //get the item by the managed file's item property (guid).\r\n SPListItem item = web.Lists[this.List].Items[this.Item];\r\n\r\n //open the file creating a binary stream\r\n return item.File.OpenBinary(); \r\n }\r\n }\r\n\r\n
\r\n", "PostedDate": "2011-02-06T18:52:25.24-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "562233", "ThreadId": "244895", "Html": "\r\n
in the WriteDelegate, you need to actually write the data to the output stream. It is not sufficient to simply assign a value to the output stream variable, which is what your code does.
\r\nLike this:
\r\nprivate void WriteEntry(String filename, Stream output)\r\n {\r\n ManagedFile value = (ManagedFile)hs[filename];\r\n byte[] b = value.GetBinary();\r\n output.Write(b,0,b.Length);\r\n }\r\n\r\n\r\n
\r\n", "PostedDate": "2011-02-07T14:17:36.743-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "562615", "ThreadId": "244895", "Html": "\r\n
Yeap, that was the trick! Works like a charm now. Also by setting that buffer to FALSE, I get a nice effect of the Download starting right away and then just streaming along as it goes!
\r\nThat probably seems silly, but we're using this function on our Organizational Meetings page and most of our users are non tech savy, so they need something to pop up right away or they think it's broken and they complain and I loose my job and my kids (2\r\n boys, 6 and 4) go hungry! Well okay, maybe not that bad.
\r\nOne last question, if I wanted to pass the docID variable to the WriteEntry method, so that I didn't have to have a member variable like "hs" defined, would I have to modify the DotNetZip source? or is there a way to provide an override without touching\r\n the DotNotZip DLL?
\r\nSee how in the whole code I have to have a HashTable called "hs" defined at the class level so that I can get the ManagedFile I am interested in? I wish I could just pass the DocID variable to WriteEntry, but since it's a delegate the signature is set in\r\n stone it seems.
\r\n\r\n\r\n
using System;\r\nusing System.Collections;\r\nusing System.IO;\r\nusing System.Web;\r\nusing System.Web.UI;\r\nusing Ionic.Zip;\r\nusing Microsoft.SharePoint;\r\n\r\nnamespace MISO.IR.ECM.SP.ApplicationPages\r\n{\r\n /// \r\n /// A page that zips a set of files based on query string parameters\r\n /// \r\n public class ZipFiles : Page\r\n {\r\n Hashtable hs;\r\n // private object lockObj = new object();\r\n\r\n /// \r\n /// Overrides the base OnLoad method for the zip file response.\r\n /// \r\n /// <param name="e" />Event Args\r\n protected override void OnLoad(EventArgs e)\r\n {\r\n base.OnLoad(e);\r\n\r\n //get the query string of document id's\r\n object queryString = Request.QueryString["docIDs"];\r\n\r\n //validate value exists\r\n if (queryString == null)\r\n return;\r\n\r\n //convert the dash '-' seperated string values to an integer array.\r\n int[] docIDs = GetDocIDs(queryString.ToString());\r\n\r\n //send the zip file back in the response as an attachment\r\n SendZipResponse(docIDs); \r\n }\r\n\r\n /// \r\n /// Outputs the zip file stream to the response output stream\r\n /// \r\n /// <param name="docIDs" />An array of document id's to include in the zip file.\r\n private void SendZipResponse(int[] docIDs)\r\n {\r\n SPSite site = SPContext.Current.Site;\r\n Response.Clear();\r\n //set response parameters for the zip attachment\r\n Response.Buffer = false;\r\n Response.Expires = 0;\r\n Response.ContentType = "application/x-zip-compressed";\r\n String guid = Guid.NewGuid().ToString();\r\n string disHeader = string.Format("Attachment;Filename=\\"download-{0}.zip\\"", guid);\r\n Response.AppendHeader("Content-Disposition", disHeader);\r\n\r\n //create a lock as the zip output stream is not thread safe\r\n //lock (lockObj)\r\n //{\r\n using (ZipFile zip = new ZipFile())\r\n {\r\n hs = new Hashtable(docIDs.Length);\r\n foreach (int docID in docIDs)\r\n {\r\n ManagedFile mf = new ManagedFile(site);\r\n mf.ID = docID;\r\n mf.Load();\r\n hs.Add(Path.GetFileName(mf.URL), mf);\r\n zip.AddEntry(Path.GetFileName(mf.URL), WriteEntry);\r\n }\r\n zip.Save(Response.OutputStream);\r\n }\r\n HttpContext.Current.ApplicationInstance.CompleteRequest();\r\n }\r\n\r\n private void WriteEntry(String filename, Stream output)\r\n {\r\n ManagedFile value = (ManagedFile)hs[filename];\r\n byte[] b = value.GetBinary();\r\n output.Write(b, 0, b.Length);\r\n }\r\n\r\n /// \r\n /// Converts a dash '-' seperated string of numbers to an integer array\r\n /// \r\n /// <param name="queryString" />The string to parse.\r\n /// An array on integers\r\n private int[] GetDocIDs(string queryString)\r\n {\r\n //split the string into an array of number string values\r\n string[] docIDs = queryString.ToString().Split(new char[] { '-' }, StringSplitOptions.RemoveEmptyEntries);\r\n \r\n //create a matching sized integer array\r\n int[] retArray = new int[docIDs.Length];\r\n\r\n int intVal = 0;\r\n\r\n //loop through the string values parsing them into the integer array\r\n for (int i = 0; i < docIDs.Length; i++)\r\n {\r\n if (!int.TryParse(docIDs[i], out intVal))\r\n throw new ArgumentException("Document ID's must be integer values.");\r\n\r\n retArray[i] = intVal;\r\n }\r\n\r\n return retArray;\r\n }\r\n }\r\n}\r\n\r\n",
"PostedDate": "2011-02-08T05:57:59.063-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "565635",
"ThreadId": "244895",
"Html": "You can avoid making the Hashtable an instance variable by converting your WriteEntry to be an Action defined within SendZipResponse. For this you need at least .NET 3.5, I think. An Action is basically a delegate, but it is defined within the SendZipResponse() method, and can be used only there (unless you pass it out via a separate call). Because it is defined within the SendZipResponse() method, the WriteEntry action can access local variables that are in-scope. Move your hashtable variable into that method, and the WriteEntry Action can access it, via closures.
\nAlso, I'd recommend using the generic Dictionary, instead of the Hashtable type. It gives you a typesafe hashtable. For that you will need to add a \"using System.Collections.Generic;\" At the top of your source file. And, I'd recommend Response.Close() in lieu of mumblefroo.CompleteRequest().
\nAll those changes look like this in code:
\n
private void SendZipResponse(int[] docIDs)
{
SPSite site = SPContext.Current.Site;
var hs = new Dictionary<string,ManagedFile>();
var WriteEntry = new Action<String,Stream>((filename,output) => {
byte[] b = hs[filename].GetBinary();
output.Write(b, 0, b.Length);
});
Response.Clear();
//set response parameters for the zip attachment
Response.Buffer = false;
Response.Expires = 0;
Response.ContentType = \"application/x-zip-compressed\";
String guid = Guid.NewGuid().ToString();
string disHeader = string.Format(\"Attachment;Filename=\\\"download-{0}.zip\\\"\", guid);
Response.AppendHeader(\"Content-Disposition\", disHeader);
//create a lock as the zip output stream is not thread safe
//lock (lockObj)
//{
using (ZipFile zip = new ZipFile())
{
foreach (int docID in docIDs)
{
ManagedFile mf = new ManagedFile(site);
mf.ID = docID;
mf.Load();
hs.Add(Path.GetFileName(mf.URL), mf);
zip.AddEntry(Path.GetFileName(mf.URL), WriteEntry);
}
zip.Save(Response.OutputStream);
}
HttpContext.Close();
}
Thanks for working so hard on this Cheeso!
\r\nUnfortunately this is on SharePoint 2007 install, so it's .Net 2.0 only! :(
\r\nI wasn't able to get the sample code working. Action<t> in 2.0 can only take the 1 parameter. I couldn't get the method generated as an Action delegate that made DotNetZip happy with WriteEntry having 2 parameters.
\r\nWe did alot of testing and it seems that the Dictionary<string, managedfile> is proving very thread safe and VERY fast. We're planning to go with that.
\r\n\r\n
Once it's done and in Production, I will share the link with you, Cheeso, to see your hard work in action!
", "PostedDate": "2011-02-15T06:38:53.753-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "566541", "ThreadId": "244895", "Html": "ok, well if you are using .NET 2.0, then you can use the WriteDelegate type. like this:
\r\nIonic.Zip.WriteDelegate WriteEntry = \r\n new Ionic.Zip.WriteDelegate((filename,output) => {\r\n byte[] b = hs[filename].Whatever...\r\n output.Write(b, 0, b.Length);\r\n });\r\n \r\n\r\n\r\n
I'm glad the Dictionary is working for you.
\r\nAnyway, good luck.
\r\n", "PostedDate": "2011-02-15T08:31:53.353-08:00", "UserRole": null, "MarkedAsAnswerDate": null } ]