In testing #336255: CCK datestamp fields totally broken and fields without timezone handling mostly broken and the new CCK date testsite install profile, I noticed a bug in signup.install. In signup_insert_default_signup_info(), reminder_days_before is set to 0, whereas the rest of the code treats this as if it defaults to 1. So, the DB doesn't match the reality you see when you visit admin/settings/signup unless you save that page. This means that all the signup-enabled nodes created by the install profile are configured to send reminders 0 days in advance, which is a non-sensical setting that's normally not possible in the code.

Comments

dww’s picture

Comment #1

dww
we/he/they
commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new 340377_signup_insert_default_signup_info.1.patch 746 bytes

This fixes it (same patch applies to both DRUPAL-5--2 and HEAD).

dww’s picture

Comment #2

dww
we/he/they
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD, DRUPAL-5--2, and backported to DRUPAL-5 and committed there, too.

Comment #3

Status: Fixed » Closed (fixed)

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