I was not able to find a module that did what my module does. The closest thing I found was this: http://drupal.org/node/496544

I'm submitting this because there seems to be a need for such a thing as described below:

  • This module allows nodes of the "giveaway" cck type to contain information about things a website can giveaway.
  • it provides a thank you page and email sent upon signing up for the giveaway
  • it also provides the administrator to download the sign up list in excel format via a button

It's basically a webform with an thank you email and page after you sign up. For years we just used webform to do what we needed for giveaways but found it really didn't cut it for us so I wrote this module. In the next version I'll include the cck type in the install function.

Project page link: http://drupal.org/sandbox/dshumaker/1307946

git repository link: http://drupalcode.org/sandbox/dshumaker/1307946.git/tree/HEAD

This module is for drupal 6.19 and up.

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Postponed (maintainer needs more info)

but found it really didn't cut it for us

Can you go into more detail on that?

For this, it's better to use double quotes and not escape the ' - that helps translators.
t('This is a test description and wish I didn\'t need it.'),

Need quotes on access_thank_you

'access callback' => access_thank_you,

I think access_thank_you should also have a check for node_access('view') on the node.

I noticed some very small code style issues. Please run the coder module on "minor" setting to help catch these. The coding standards have even more information in this area.

dsm won't work unless you add a dependency on devel, but I think really you want drupal_set_message.

This should use a db_query placeholder like elsewhere in the module:

db_query("select id, nid,firstname,lastname,subscribe, email,datestamp from giveaway where nid=" . $nid . " order by id");

As far as I know this comment isn't true. I'd like to hear more details on it to understand under what scenarios it is happening:

// Can't use the passed in node parameter because it is not passed when url aliases are used and the giveaway nodes are used as aliases.
88 // otherwise would just use $node->nid for the database call.

You need a "core = 6.x" in your .info

I think the review should be postponed until the giveaway content type is exported and created automatically.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Postponed (maintainer needs more info) » Closed (won't fix)

There have not been any reply on the past two months; I am changing the status to won't fix.