There are a lot of cleanups to do in contact.module to help bring this up to documentation and conding standards.

CommentFileSizeAuthor
#9 599186-contact-cleanup-D7-2.patch27.43 KBDave Reid
#7 599186-contact-cleanup-D7.patch28.3 KBDave Reid
#6 599186-contact-cleanup-D7.patch28.3 KBDave Reid
#4 599186-contact-cleanup-D7.patch24.77 KBDave Reid
#3 599186-contact-cleanup-D7.patch24.76 KBDave Reid
#1 599186-contact-cleanup-D7.patch19.02 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
Issue tags: +API change
FileSize
599186-contact-cleanup-D7.patch19.02 KB

Just a few things so far...so much to clean up.
- Renames vague functions like contact_admin_categories() and contact_admin_edit() to contact_category_list() and contact_category_edit_form(). Much easier to tell what each function does.
- Simplified the logic in contact_category_edit_form().
- Cleans up contact_load() for simpler code.
- Cleans up message variables in contact_mail().

Status: Needs review » Needs work

The last submitted patch failed testing.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs work » Needs review
FileSize
599186-contact-cleanup-D7.patch24.76 KB
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
FileSize
599186-contact-cleanup-D7.patch24.77 KB

Thank god for simpletest. Patch with 100% less stupid mistakes.

Status: Needs review » Needs work

The last submitted patch failed testing.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs work » Needs review
FileSize
599186-contact-cleanup-D7.patch28.3 KB

ARG. Wrong patch.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
FileSize
599186-contact-cleanup-D7.patch28.3 KB

Fixed a few spelling errors.

Dries’s picture

Dries CreditAttribution: Dries commented 
+++ modules/contact/contact.admin.inc	8 Oct 2009 16:04:08 -0000
@@ -9,28 +9,35 @@
+  // If no categories were found, add a link to create a new category.

No link is added so the comments looks incorrect.

Otherwise this looks like a nice clean-up.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
FileSize
599186-contact-cleanup-D7-2.patch27.43 KB

Re-rolled for recent commits and removed that comment.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Fixed

Committed. Thanks for the reroll.

Status: Fixed » Closed (fixed)
Issue tags: -API change

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +API change

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.