Issue status update for http://drupal.org/node/27633 Post a follow up: http://drupal.org/project/comments/add/27633 Project: Drupal Version: cvs Component: contact.module Category: tasks Priority: normal Assigned to: Anonymous Reported by: Tobias Maier Updated by: Tobias Maier Status: patch Attachment: http://drupal.org/files/issues/updates.inc_3.patch (1.13 KB) updates.inc.patch Tobias Maier Previous comments: ------------------------------------------------------------------------ Wed, 27 Jul 2005 23:00:09 +0000 : Tobias Maier The contact.module of the Drupal HEAD does now nearly the same as the feedback.module. kbahey [1] 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 [1] http://drupal.org/user/4063 ------------------------------------------------------------------------ Thu, 28 Jul 2005 08:40:21 +0000 : Tobias Maier i forgot to mention that it also sends the referrer in the E-Mail what I also want to include ------------------------------------------------------------------------ Thu, 28 Jul 2005 10:30:36 +0000 : lennart Hi 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 ------------------------------------------------------------------------ Thu, 28 Jul 2005 10:48:13 +0000 : Tobias Maier i think also that this messages should stay there, but I think they don't have to be customizable, like in Feedback. ------------------------------------------------------------------------ Thu, 28 Jul 2005 13:03:54 +0000 : kbahey Tobias, 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. ------------------------------------------------------------------------ Sat, 30 Jul 2005 02:32:58 +0000 : Tobias Maier here are the first patches regarding this: * [bugfix] if contact.module is unconfigured the form gets shown [2] * [bugfix/new feature] hide subject if there is just one [3] * [new feature] input format for contact.module [4] * [bugfix] Side-wide contact form: doesn't save entered text on input error reload [5] 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 [2] http://drupal.org/node/27757 [3] http://drupal.org/node/27822 [4] http://drupal.org/node/27824 [5] http://drupal.org/node/27139 ------------------------------------------------------------------------ Sat, 30 Jul 2005 13:30:40 +0000 : Tobias Maier Attachment: http://drupal.org/files/issues/contact.module_1.patch (10.38 KB) and 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: * Includes bugfix: if contact.module is unconfigured the form gets shown [6] * includes bugfix: if contact.module is unconfigured the form gets shown [7] * includes bugfix: Side-wide contact form: doesn't save entered text on input error reload [8] * renamed subject to category * added a real Subject-field * includes input format for contact.module [9] * includes hide subject if there is just one [10] * changes in contact-table (just MYSQL) * subject renamed to category * category is now PRIMARY KEY [6] http://drupal.org/node/27757 [7] http://drupal.org/node/27757 [8] http://drupal.org/node/27139 [9] http://drupal.org/node/27824 [10] http://drupal.org/node/27822 ------------------------------------------------------------------------ Sat, 30 Jul 2005 13:31:33 +0000 : Tobias Maier Attachment: http://drupal.org/files/issues/database.mysql_5.patch (720 bytes) the patch for the mysql-table ------------------------------------------------------------------------ Sat, 30 Jul 2005 14:26:00 +0000 : Tobias Maier Attachment: http://drupal.org/files/issues/contact.module_2.patch (11.02 KB) contact.module updated again -->now it runs :D ------------------------------------------------------------------------ Sat, 30 Jul 2005 15:16:38 +0000 : Dries - 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? ------------------------------------------------------------------------ Sat, 30 Jul 2005 16:42:02 +0000 : Tobias Maier Attachment: http://drupal.org/files/issues/database.pgsql_5.patch (634 bytes) "If there is only one category, we probably don't need a hidden category either? " if 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. "Please drop the filter stuff for now. " 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...