Hey there! First off, thanks for doing this! I'm just starting to figure it out, but it looks like it's going to suit my project very well.

I just tried my first download, and checked drupal's status report, and saw this:

---
Cannot modify header information - headers already sent by (output started at /Users/me/Sites/drupal/sites/site.dev/modules/pclzip/pclzip.module:116) in /Users/me/Sites/drupal/includes/common.inc on line 328.
---

The download still worked perfectly. Just a heads up.
Cheers!

Comments

ssm2017 Binder’s picture

ssm2017 Binder commented

thank you for reporting
can you try with adding "exit;" after the "readfile" function in pclzip.module line 116 ? like :

readfile($pclzip_temp_file);
exit;
stoptime’s picture

stoptime commented

That seems to have done the trick!

ssm2017 Binder’s picture

ssm2017 Binder commented
Status: Active » Fixed

cool :) i have commited it to the cvs

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.