I would like to see a pclzip_api module where all it does is provide an api interface to pclzip. I think this would be useful to other module developers wanting to implement zip/unzip functionality. Looking through the code the current module seems to include cck functions which wouldn't be necessary in an api.

Comments

ssm2017 Binder’s picture

ssm2017 Binder commented

this module is just an implementation of the pclzip library in a simple way
this module is not an universal zip api
pclzip is already a kinf of zip api, so i will not build an api of the api

about the api :
this module is only made to handle the pclzip library
this module is not made to zip/unzip
everyone can build another module that will use it to zip/unzip
this module is only :
- checking if the library exists
- build an object that can be used to zip/unzip by other modules
- send the zip file if someone build a zip

about the cck implementation :
i have added this cck check because i need it for submodules and didnt want to repeat the code but if you dont have these modules and if you dont have cck, nothing will appear

if you have some ideas about patches, feel free to send them and depending on them we can speak about this and maybe add you as a co-maintainer
if you wan to build some sub modules using this module, feel free to publish them

30equals’s picture

30equals commented
Status: Active » Closed (won't fix)

Old threat cleanup.