Typo Mistake for Subscription Message

drupal_jon - October 29, 2009 - 12:10
Project:MailChimp
Version:5.x-2.x-dev
Component:Code
Category:task
Priority:normal
Assigned:spatz4000
Status:closed
Description

Just spotted an amusing spelling mistake...

Line 213 mailchimp.module:

    $items[] = array(
      'path'     => 'mailchimp/subscribe/message',
      'callback' => 'mailchimp_subscribe_message',
      'title'    => t( 'MailChump Subscription Message' ),
      'type'     => MENU_CALLBACK,
      'access'   => true,
    );

Note the spelling mistake for the title 'MailChump Subscription Message'.

Also, perhaps the title should be added to the admin interface as well as the message.

#1

spatz4000 - October 29, 2009 - 13:20
Version:5.x-2.0» 6.x-2.x-dev
Assigned to:Anonymous» spatz4000
Status:active» needs review

Here's a patch for the development branch.

AttachmentSize
mailchimp-617924.patch 837 bytes

#2

LouBabe - October 29, 2009 - 18:28
Version:6.x-2.x-dev» 5.x-2.x-dev
Status:needs review» fixed

Type committed to 5.x-2.x branch. Note this is NOT in the 6.x-2.x branch as indicated. I'm not really maintaining 5.x anymore, but happy to fix the typo.

#3

System Message - November 12, 2009 - 18:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.