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!

CommentFileSizeAuthor
#1 textimage_email_support.patch1.02 KBDevElCuy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DevElCuy’s picture

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

Deciphered’s picture

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.

Deciphered’s picture

Status: Active » Fixed

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

DevElCuy’s picture

Thank you for both, guidelines and commit!

Blessings!

Status: Fixed » Closed (fixed)

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