Hi,
I am attaching a patch for 6.x port of this module.

CommentFileSizeAuthor
#5 advcontact_296783_5.patch1.33 KBmikey_p
#1 advcontact-6.x.patch2.61 KBmeba
advcontact-6.x.patch1.64 KBmeba

Comments

meba’s picture

meba commented
StatusFileSize
new advcontact-6.x.patch2.61 KB

Rerolling, fixed a typo in help and fixed a bug that this string wasn't exported into translation files.

neclimdul’s picture

neclimdul
He/Him
commented

Thanks! I'll take a look.

neclimdul’s picture

neclimdul
He/Him
commented

Could you leave the help text the way it was? If there is a problem can we handle it separately so its easier to review?

neclimdul’s picture

neclimdul
He/Him
commented
Status: Needs review » Needs work

per my comments

mikey_p’s picture

mikey_p commented
StatusFileSize
new advcontact_296783_5.patch1.33 KB

Removing help text modifications.

Simplest. Update. Ever.

meba’s picture

meba commented
Status: Needs work » Needs review
a_c_m’s picture

a_c_m commented
Status: Needs review » Reviewed & tested by the community

Works for me, really simple patch so i think RTBC.

jweberg’s picture

jweberg commented

Any idea when this will be rolled out as a stable 6.x version?

sinuz’s picture

sinuz commented

subscribing

awry’s picture

awry commented

The patch basically works for me. There's some weird grammar in one spot, though, which I changed like so:

--- advcontact.module   (revision 78)
+++ advcontact.module   (working copy)
@@ -38,7 +38,7 @@
           '#required' => TRUE,
         );
         $form['contact_information']['#value'] =
-          t('You can leave us a message about using the contact form below regarding: @category', array('@category' => $cat));
+          t('Use the form below to send a message to our @category team. Thanks for your interest.', array('@category' => $cat));
       }
     }
neclimdul’s picture

neclimdul
He/Him
commented
Status: Reviewed & tested by the community » Fixed

Thanks guys, took a little of both patches(no message change). Forgot to credit but thanks a lot!

Status: Fixed » Closed (fixed)

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