I believe function _notify_content($node, $notify) has problems processing URLs:

1. If they contain target and title - example:
<a href="http://www.drupal.org" target="_blank" title="www.drupal.org">Drupal</a>
would be listed as:
[x] http://www.drupal.org\" target=\"_blank\" title=\"www.drupal.org

2. If an URL contains anchors - example:
<a href="http://www.drupal.org#whatever">whatever</a>
would be listed as:
[x] http://www.drupal.org%2523whatever

Both are not proper links...

CommentFileSizeAuthor
#1 notify_94118.patch.txt512 bytesstella

Comments

stella’s picture

stella commented
Status: Active » Needs review
StatusFileSize
new notify_94118.patch.txt512 bytes

attached patch works for me for both Drupal 4.7 and Drupal 5. I didn't have any problems with the anchors in either version, but did have problems with the first example.

beginner’s picture

beginner commented
Version: 4.7.x-1.x-dev » 5.x-1.x-dev
Status: Needs review » Needs work

Oh my god! I hate regexes!

Can you explain this one for me, please?

I noticed og module had the same problem. They are using the same regex.
If the patch is good for them, it is probably good here, too.

beginner’s picture

beginner commented

see og.module issue:
http://drupal.org/node/157664

beginner’s picture

beginner commented

Actually, the example with the anchor (drupal.org#whatever) works for me, too.

matt2000’s picture

matt2000 commented
Status: Needs work » Closed (fixed)

Hasn't been replicated in 3 years, closing.