I installed this on a test site running Drupal 4.7RC3. It shows up as expected in the list of filters, etc. However, I cannot detect any effect of this on the content. No addresses are converted/filtered as far as I can see.

Comments

jarea’s picture

Track this issue

jjeff’s picture

You are looking at the html source, correct? If everything works right, there shouldn't be any visual difference on the page.

jarea’s picture

Using browser view source, I am seeing a plaintext email address not an encoded one.

pwolanin’s picture

I haven't tried this module recently- as above, viewing the source demonstrated that the e-mail address was not encoded.

gtcaz’s picture

I have many email addresses that are not being obfuscated or autolinked. I cannot determine a rational pattern as to why some are processed and others are not, although it is repeatable (non-random). Some emails are consistently not processed, while others (on the same page) are. I'm running the current CVS version of the 4.7 branch.

gtcaz’s picture

Here's a strange test case. I have Textile installed.

1jim@fbz.com

Jim Foo
Foo, Bar & Baz, P.C.
335 N. South Way
Blankville, Utopia 55555
(800) 555-1212
(800) 555-5555(fax)
2jim@fbz.com
Admitted 1996 (MN); 2002 (AZ)

3jim@fbz.com

If I put Invisimail first in the filter ordering, only the 2jim address is processed. If I put Invisimail second (behind Textile), then only the 1jim1 and 3jim addresses are processed.

If I run Invisimail with only the line break converter (Invisimail -10, Line Break Converter 10), only the 2jim address is processed.

gtcaz’s picture

I'm going to try the regex from urlfilter in this module, or perhaps add an invisimail option to the urlfilter module.

gtcaz’s picture

Title: not working with 4.7RC3 » Invisimail doesn't consistently process email addresses
Priority: Normal » Critical
gtcaz’s picture

Status: Active » Reviewed & tested by the community
StatusFileSize
new707 bytes

The regex from urlfilter does indeed work. Perhaps email encoding could be a urlfilter option?

gtcaz’s picture

Status: Reviewed & tested by the community » Needs work

There are still cases where the email isn't processed. The crazy thing is, it's working for me locally under XAMPP, but on my server some emails are not processed unless I add a space before them when they are at the beginning of a line. Another strange thing is sometimes I can add a single letter and the email will be processed. Take that letter out and the email isn't processed. I'm at a loss as to how to debug this.

gtcaz’s picture

I also have some email addresses that are failing under my remote server and XAMPP. Something about the block of text they're in.

sfarestam’s picture

StatusFileSize
new712 bytes

I updated the regex in the previous patch to take into account the fact that

tags may contain style information etc.

ilfelice’s picture

You may want to try to rearrange your filters so that "Encode email addresses" is processed first.

Go to administer > input formats, click on the Configure link of the input format that invisimail is being used with, and click on the "rearrange" tab.

Hope it helps.

gtcaz’s picture

I've tried it with all different orders (see my Post #6 above). Thanks.

AmrMostafa’s picture

StatusFileSize
new842 bytes

Why does the regex care about

and

  • ?

    I've stumbled into the same issue, created a patch and was gonna submit an issue but found this one, so may be it's better to post it here. Patch attached.

  • AmrMostafa’s picture

    Sorry didn't know HTML was enabled in comments. First line should read: Why does the regex care about <p> and <li>?

    Crell’s picture

    Status: Needs work » Fixed

    4.7 is no longer supported. The 6.x-1.1 release has a new, more robust regex that should catch any reasonable case.

    Status: Fixed » Closed (fixed)

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