This is a simple module to remove the subject field from the site-wide contact form.

Simply enable to module from admin/build/modules to remove the subject field.

Version: Drupal 6
Project page: http://drupal.org/sandbox/GavA/1919486
Git: git clone --branch 6.x-2.x http://git.drupal.org/sandbox/GavA/1919486.git contact_no_subject

A Google search shows there is a lot of interest around removing the subject field from the contact form, but as of yet no module for this. This module provides a quick and easy way for users who are not comfortable with editing code to remove the contact form subject field.

Comments

PawelR’s picture

PawelR commented

In my opinion this module is to simple - it's just one alter function with one line of code.
That kind of tweak can be done in the template.

jbloomfield’s picture

jbloomfield commented

Agree with PawelIR, there is no need for a module to do this. Can be done in the template.

joseffriedrich’s picture

joseffriedrich commented

A module to pass the review process should have a minimum of 120 lines code.

Also there a minor coding standards problems:

FILE: /var/www/drupal-7-pareview/pareview_temp/contact_no_subject.module
--------------------------------------------------------------------------------
FOUND 2 ERROR(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
2 | ERROR | Missing file doc comment
3 | ERROR | Function comment short description must be on a single line
--------------------------------------------------------------------------------
klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented

We are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)

arun ak’s picture

arun ak commented
Status: Needs review » Needs work

Hi,

I checked your module. There is only 9 lines of code in your module file.

"$form['subject']['#access'] = FALSE;"

For only this purpose there is no requirement of an additional module. If you wish to continue the development of this module, then add more additional features with it.

Regards
ARUN

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs work » Needs review

Of course we also accept short modules for one specific purpose. This project is too short to approve you as git vetted user. We are currently discussing how much code we need, but everything with less than 120 lines of code or less than 5 functions cannot be seriously reviewed. However, we can promote this single project manually to a full project for you.

ayesh’s picture

ayesh commented

Some suggestions from my manual review:
- Add contact module as a dependency.
- Remove the master branch (See moving from master to major branch)
- In the comment block, specify you are implementing hook_form_FORM_ID_alter. See the doc.

I'm not sure you will get the Full Project permissions for this module, but this module is not duplicating some other functionality as far as I know, and is useful in certain occasions.

Good luck!

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented
Title: Contact No Subject » [D6] Contact No Subject
Status: Needs review » Needs work

Yeah, this module is too simple - it would be easier to write a blog post describing your technique. Not sure setting #access to FALSE is the best way to disable the subject field anyway.

----
Top Shelf Modules - Enterprise modules from the community for the community.

PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

I'm a robot and this is an automated message from Project Applications Scraper.