We had to make a few fields disappear in our installation of the module here. I have therefore made which fields are displayed configurable.

With this patch, the fields 'name', 'email', 'adddress', 'city', 'state', 'zip', 'phone' and 'fax' can be turned off in the module's configuration.

CommentFileSizeAuthor
#1 optional_0.patch5.42 KBanarcat
optional.patch3.91 KBanarcat

Comments

anarcat’s picture

anarcat commented
StatusFileSize
new optional_0.patch5.42 KB

note that this patch depends on the factor_1.patch of issue 52280.

i've also attached a modified version of the patch that will properly accept forms without the disabled fields. :)

georgehotelling’s picture

georgehotelling commented
Status: Needs review » Needs work

Thanks for your work on this module. There's a few changes that need to be made before this is ready to be committed:

1. In citizenspeak.theme.php: theme_citizenspeak_statistics_page() the ZIP Code statistics should be turned off if the ZIP Code field is disabled. It also shouldn't call _citizenspeak_get_zips() from citizenspeak_report() since it won't be using the result.

2. The generated reports should not include fields that are turned off, otherwise it will be confusing to campaign creators who see fields in their reports that aren't in their forms.

3. theme_citizenspeak_message() should not include fields that are turned off either.

Rosamunda’s picture

Rosamunda commented

+1!