Posted by develCuy on February 3, 2009 at 3:26am
| Project: | Textimage |
| Version: | 5.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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!
Comments
#1
Just 3 lines needed for the task, works fine out of the box!
#2
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
Committed to: HEAD, DRUPAL-6--2 and DRUPAL-5--2.
#4
Thank you for both, guidelines and commit!
Blessings!
#5
Automatically closed -- issue fixed for 2 weeks with no activity.