CVS edit link for weaponx

I wish to contribute with a few modules and themes I have made for my Drupal website (http://howlingmoon.info)

The first module i wish to create is described in this issue:
http://drupal.org/node/692998
This module allows users to zip an entire gallery as .zip or .cbz

I am also working on some themes. You may see the main theme i am working on at my website.

CommentFileSizeAuthor
#1 pclzip_zipfolder_galleries.zip18.49 KBweaponx

Comments

weaponx’s picture

weaponx commented
Assigned: Unassigned » weaponx
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new pclzip_zipfolder_galleries.zip18.49 KB

This is module for the above application.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: weaponx » Unassigned
Issue tags: +Module review
beeradb’s picture

beeradb commented
Status: Needs review » Needs work

The uploaded file contains a license.txt, which cannot be committed to the drupal.org CVS. Please double check the CVS application guidelines.

You should also take another pass at the coding standards. Although I didn't do a in depth review, a first pass showed some left over development code, invalid function names (clean_title), and commenting issues (they should be complete sentences).

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

Hello, and thank you for applying for a CVS account.
I am just adding a further note to what already reported by beerad; may you add something more about the module you propose to be added in CVS repository? This would allow us to understand better why you decided to create a separate module, and not report a feature request for an existing module.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs work » Closed (won't fix)

There have not been replies from the OP in the past 7 days. I am marking this report as won't fix.