Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
The contact.module of the Drupal HEAD does now nearly the same as the feedback.module.
kbahey and I talked about to bring them together.
To have just one module.
a) Here comes a list of features which you can find in feedback.module and which are currently missing in contacts.module
- add the following fields to contacts.module:
- Postal Address
- Phone Number
- A real "Message Subject" (You say "What?? we have one!" - this one is used in the false way. Be patient i'll tell you later what i mean)
- make it configurable if these fields are shown on the Sites Contact-Page and/or on the Users Contact-Page
- Make it configurable which field is required and which one not
- The Subject field is currently used to select on which team/department the message belongs to.
this field should be renamed to topic/category/department (just one of them / maybe let the user decide)
and a Subject field with the default behavour should be introduced. - let the "Additional information:"-field be formatted. So add form_format()
- Feedback has a feature "Log all feedback attempts to watchdog"
- it also allows you to add an special "Subject Prefix:" to the e-mail which will be sent.
- and to customize the Message which will be shown after the user pressed submit. ("Result Message - Success". "Result Message - Failure")
(personally i think this is not really needed) - it also allows you to define a "Navigation link text" and a "Feedback Page Header" - but the most users (including me) were a little bit confused about what that means
(personally i think this is not really needed)
b) here comes my list what i want to see beside the things mentioned above in contact.module:
- why does contact have two places to configure it (admin/contact and admin/settings/contact)
i think they should be merged into one. So that it is easier for the User to get into this. - Help texts, everywhere :D
- a new permission:
"Let me use all Users Contact page" - allow specific Roles to contact the User through the Contact-Page, also if he deactivated it
Please tell me what you think about it and if you have other ideas.
I want to start to merge the feedback module into the contact module soon
Yours sincerely
Tobias Maier
Comment | File | Size | Author |
---|---|---|---|
#15 | contact.module_5.patch | 9.65 KB | Tobias Maier |
#13 | contact.module_4.patch | 10.97 KB | Tobias Maier |
#12 | contact.module_3.patch | 10.76 KB | Tobias Maier |
#11 | updates.inc_3.patch | 1.13 KB | Tobias Maier |
#10 | database.pgsql_5.patch | 634 bytes | Tobias Maier |
Comments
Comment #1
Tobias Maier CreditAttribution: Tobias Maier commentedi forgot to mention that it also sends the referrer in the E-Mail
what I also want to include
Comment #2
lennart CreditAttribution: lennart commentedHi Tobias,
I am currently using both feedback.module and contact.module, so for me a merging would generally make sense.
I agree with most of your ideas except; I do not think " the Message which will be shown after the user pressed submit. ("Result Message - Success". "Result Message - Failure") " is not needed.
I think this should stay.
It gives comfort and certainty to the user to know that the mail has been sent succesfully
Regards
Lennart
Comment #3
Tobias Maier CreditAttribution: Tobias Maier commentedi think also that this messages should stay there, but I think they don't have to be customizable, like in Feedback.
Comment #4
kbahey CreditAttribution: kbahey commentedTobias,
Thanks for the comparison.
As the author of Feedback, I see a lot of overlap between contact and feedback. I would like to see them merged, and become part of core.
contact is a good base to build on, and has some extra features (most importantly the ability to contact a specific user, as well as list different topic/category and route the message accordingly (think of sales, support, marketing, ..etc or complaint, comment, request, ...etc.)
Add to all that one feature present to neither: a checkbox to send a copy of the email to the originator for filing.
If both merge and add the above feature, I will be very happy, and feedback can then rest in peace starting from 4.7.
Comment #5
Tobias Maier CreditAttribution: Tobias Maier commentedhere are the first patches regarding this:
please test them all and integrate them all as fast as possible, because i want to develop the next steps on top of them
Thanks
Comment #6
Tobias Maier CreditAttribution: Tobias Maier commentedand here comes the first all-in-one patch:
please try it, and tell me if it works for you:
This is just the first step to solve the list above so there come much more enhancements later...
Changelog:
Comment #7
Tobias Maier CreditAttribution: Tobias Maier commentedthe patch for the mysql-table
Comment #8
Tobias Maier CreditAttribution: Tobias Maier commentedcontact.module updated again
-->now it runs :D
Comment #9
Dries CreditAttribution: Dries commented- You need to provide an update for updates.inc too.
- You need to provide an update for database.pgsql.
- Please drop the filter stuff for now.
- Please don't capitalize the words category and subject in
+ form_set_error('subject', t('You must enter a Subject.'));
and+ form_set_error('category', t('You must select a valid Category.'));
.- If there is only one category, we probably don't need a hidden category either?
Comment #10
Tobias Maier CreditAttribution: Tobias Maier commentedif I don't send it hidden I have to count the number of available categories at the field validation.
I think this is the best way, because I don't have so much sql queries.
why? if it has security reasons than it would be a solution only to allow the dafault filter.
here is the update
- removed filter
- changed the words to lowercase
- update for updates.inc
- update for database.pgsql
I was not able to test the pqsql things...
Comment #11
Tobias Maier CreditAttribution: Tobias Maier commentedupdates.inc.patch
Comment #12
Tobias Maier CreditAttribution: Tobias Maier commentedcontact.module.patch
it includes the hidden input field
maybe I release one without it...
Comment #13
Tobias Maier CreditAttribution: Tobias Maier commentedand here comes the patch without the hidden form field
Comment #14
Dries CreditAttribution: Dries commentedCommitted to HEAD. Thanks.
Comment #15
Tobias Maier CreditAttribution: Tobias Maier commentedafter the first part was commited to HEAD
I want to start with the second:
* added phone number
* added postal address
* you can configure which field should be shown and which one not
* you can configure which field is required
* referrer is sent with site-wide contact form
* settings page is a little bit changed (I hope you like it)
* [bugfix/feature] whitespaces were trimmed on $edit[]
* [bugfix] site_name readded to subject...
the email which will be sent could _maybe_ improved.
the best would be html but this is not allowed :(
Comment #16
Tobias Maier CreditAttribution: Tobias Maier commentedComment #17
Crell CreditAttribution: Crell commentedPatch in #15 needs to be updated to the new form API.
Comment #18
Tobias Maier CreditAttribution: Tobias Maier commentedI will release seperate patches for all relevant bugfixes/features
Comment #19
Jaza CreditAttribution: Jaza commentedStill needed? I'm not sure... I'd like to see contact.module be upgraded with the ability to add custom fields, rather than simply add a whole lot more system-provided fields to it.
anyway, moving to 6.x-dev queue.
Comment #20
BioALIEN CreditAttribution: BioALIEN commentedJaza beat me to it. I was just about to say, contact.module should follow the path of profile.module and allow for the addition of custom fields, specify required fields etc. This will also slow down the spam invasion as not all contact forms will be identical (without the need for additional contrib modules).
Put me down for testing :)
Comment #21
NancyDruJust my two cents worth: The Contact_List module adds functionality that I can't believe isn't already there. It should be a minor merge effort. The main thing it adds is the ability to "pre-select" the category (recipient) from a link (
<a href="/contact/Sales+Department">contact the Sales Department</a>
).I also like the "Contact List" intercept rather than the drop down selector, but others may not. The Contact_List also adds a bit more customization to the contact form.
Comment #22
bradlis7 CreditAttribution: bradlis7 commentedI've got a patch for the referrer at Issue# 183661. It's currently not working, but any discussion about it should go there.
Comment #23
bradlis7 CreditAttribution: bradlis7 commentedIn reply to #21, see Issue 183678.
Comment #24
catchComment #25
behindthepage CreditAttribution: behindthepage commentedI vote also for customizable fields as stated in #19 and #20.
Would this be a good time to consider merging Contacts Forms as well so that when you specify a category eg. contact/Sales_Department you can have a customizable "additional info" and choose whether the drop down category menu shows. If you are going to add half the features of my module the other half is only another step. Although I think it is important to keep the admin as simple as possible and if that means leaving features out and having add on modules then I am all for that.
I am happy to contribute some code for customizable fields and for merging Contact Forms.
Regards
Geoff
Comment #26
Dave ReidLet's make sure we have all separate issues for each feature please. Marking this issue as a duplicate.