in testing some other stuff, i just noticed that we're needlessly storing all our settings in the {variable} table, because of how we're using system_settings_form(). given the non-standard way we're handling the default signup settings in the {signup} table, we should consider if we want to use system_settings_form() at all, and perhaps just add those buttons manually, and just have our own submit handler (which we already have and need) do whatever manipulations of the {variable} table are required...

CommentFileSizeAuthor
#1 signup_settings_no_dups_5x.patch.txt2.28 KBdww

Comments

dww’s picture

Status: Active » Needs review
StatusFileSize
new2.28 KB
dww’s picture

Status: Needs review » Fixed

committed to HEAD and DRUPAL-5

Anonymous’s picture

Status: Fixed » Closed (fixed)