[drupal-devel] [feature] Add subject to contact module

Goba drupal-devel at drupal.org
Sat Feb 19 12:07:45 UTC 2005


 Project:      Drupal
 Version:      cvs
 Component:    base system
 Category:     feature requests
 Priority:     normal
 Assigned to:  kbahey
 Reported by:  kbahey
 Updated by:   Goba
-Status:       fixed
+Status:       patch
 Attachment:   http://drupal.org/files/issues/Drupal.fix-contact-module-ws.patch (1.42 KB)

Whitespace fixes after the patch...


Goba



Previous comments:
------------------------------------------------------------------------

February 2, 2005 - 03:26 : kbahey

Attachment: http://drupal.org/files/issues/contact.module-subject.patch (1.93 KB)

I find it very undescriptive when I recieve a message from Drupal with
the subject "message from username".
This patch adds a "subject" field for the contact.module which the user
can fill, and would tell you what they want right away.
Oh, and it helps group the 'conversation' on Gmail into something
meaningful.
(Note, I have not tested this since I do not have a CVS installation at
the moment. Appreciate if someone can test it).


------------------------------------------------------------------------

February 9, 2005 - 21:20 : kbahey

Did this patch make it? Any objections to it?


------------------------------------------------------------------------

February 10, 2005 - 21:02 : Dries

I'll commit this patch when there is more demand for it.


------------------------------------------------------------------------

February 10, 2005 - 21:12 : killes at www.drop.org

I'd like to see this applied. An email got to have a subject.


------------------------------------------------------------------------

February 18, 2005 - 16:34 : kbahey

This is badly needed.
Just today I got a message from walkah and Gmail grouped it in a
previous conversation with someone else, just because the subject is
"message from kbahey".
We have to get this to CVS.


------------------------------------------------------------------------

February 18, 2005 - 16:42 : Morbus Iff

+1 from me.


------------------------------------------------------------------------

February 18, 2005 - 18:10 : tangent

There is an added newline in the foreach which seems unnecessary. Some
lines also have tabs which disrupts the indentation.
Otherwise, +1 on the functionality.


------------------------------------------------------------------------

February 18, 2005 - 18:38 : Anonymous

+1 from me too. Since this is a simple useability extension, I hope that
it will get into 4.6.


------------------------------------------------------------------------

February 18, 2005 - 18:38 : Goba

The previous 'me' was me :)


------------------------------------------------------------------------

February 18, 2005 - 19:50 : Dries

Committed to HEAD.


------------------------------------------------------------------------

February 18, 2005 - 22:35 : kbahey

I think we missed something in this patch.
The form_textfield has an argument that the maximum is 8 characters.
This is too short. Should be 30 characters or more. Line 125 in HEAD.


------------------------------------------------------------------------

February 19, 2005 - 09:17 : Dries

Fixed in HEAD.  Thanks.


-- 
View: http://drupal.org/node/16590
Edit: http://drupal.org/project/comments/add/16590





More information about the drupal-devel mailing list