Integration with email cck field

develCuy - February 3, 2009 - 03:26
Project:Textimage
Version:5.x-2.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

What a cool module!
But I miss support for email field, since textimage only supports text cck fields, integration will help with e-mail obfuscation.

Blessings!

#1

develCuy - February 3, 2009 - 03:32

Just 3 lines needed for the task, works fine out of the box!

AttachmentSize
textimage_email_support.patch 1.02 KB

#2

Deciphered - February 8, 2009 - 22:09

Hi develCuy,

Had a look at the patch and had to make one minor change; $formatter reports the Textimage preset, which isn't guaranteed to always be 'email', so using $filed['type'] instead.

Other than that, looks good and will be in the next dev release.

Also, you forgot to flag the issue as a patch, I won't do so now as the patch is partially irrelevant, but it would be helpful to do so in the future.

Cheers,
Deciphered.

#3

Deciphered - February 8, 2009 - 22:27
Status:active» fixed

Committed to: HEAD, DRUPAL-6--2 and DRUPAL-5--2.

#4

develCuy - February 9, 2009 - 16:33

Thank you for both, guidelines and commit!

Blessings!

#5

System Message - February 23, 2009 - 16:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.