Issue Details (XML | Word | Printable)

Key: CRM-4633
Type: New Feature New Feature
Status: Closed Closed
Resolution: Fixed/Completed
Priority: Minor Minor
Assignee: David Greenberg
Reporter: Tom Kirkpatrick
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
CiviCRM

Add Do Not SMS communication preference option

Created: 12/Jun/09 03:54 PM   Updated: 08/Dec/09 02:11 AM
Component/s: Core CiviCRM
Affects Version/s: 2.2.5
Fix Version/s: 3.0

Time Tracking:
Not Specified

File Attachments: 1. Text File civicrm_2.2.5_do_not_sms.patch (17 kB)



 Description  « Hide
currently we have Do Not Phone, Mail, Email, and Trade... but what about SMS? This is a very common method of communication and I think the option should be included in core rather tan making people create a custom field to handle this - which does not display alongside the other communication preferences without template hacks.

Attached is a patch that *should* add the support for this. I wrote this patch against the 2.2.5 release. The only think I haven't included is the ALTER TABLE statement for the update script since I wasn't sure where this should go. The SQL for that would be:

ALTER TABLE `civicrm_contact`
  ADD `do_not_sms` tinyint(4) default '0' AFTER `do_not_trade`;

And civicrm_cache needs to be cleared, but I assume that happens on an upgrade automatically (took me a while to figure out while my changes hadn't taken effect!)

 All   Comments   Work Log   Change History   FishEye      Sort Order: Ascending order - Click to sort in descending order
Neha Kulkarni added a comment - 23/Jun/09 01:49 PM
checked in rev 22057

Kurund Jalmi added a comment - 16/Jul/09 07:11 AM
Batch move to verification

Yashodha Chaku added a comment - 16/Jul/09 11:28 AM
assigning for 3.0 QA

Michał Mach added a comment - 18/Aug/09 01:53 PM
verified

Yashodha Chaku added a comment - 09/Nov/09 06:30 AM
assigning for 3.1 verification