I an only getting one glossified link per paragraph, even though there are multiple keywords to different pages in the paragraph. I have the following configuration set:

1. Link From everything. Link to everything.
2. Set to TRUE: Only link first occurrence of term.
3. I am using both the "Make use of title" and the internal method to create links.
4. My target "To" node in the attached screen shot has the following title: Photo Gallery
5. My target "To" node in the attached screen shot has the following comma separated keywords added to it: gallery, images, photos, pictures

The attached screen shot of the "From" node contains the following paragraph: (Yes, it is garbage text while I am testing this issue.)

Be sure to contact us and pictures photos gallery we can help. You can also request a consultation.

Only "contact us" is being glossified (a different target "To" node) in that paragraph while the follow keywords are not: pictures, photos and gallery

A later paragraph does glossify the keyword: photos

I am using TinyMCE 3.3.2, although I am not sure how that would affect it. If it helps, the site is running:
Drupal 6.19
MySQL database 5.0.91
PHP 5.2.14

Anything else needed?

CommentFileSizeAuthor
#1 CORRECT-one-glossify-per-paragraph.png88.58 KBArchnemysis
one-glossify-per-paragraph.png81.95 KBArchnemysis

Comments

Archnemysis’s picture

Archnemysis commented
StatusFileSize
new CORRECT-one-glossify-per-paragraph.png88.58 KB

Ugh. Uploaded the wrong screen shot. This is the correct one where the later paragraph does glossify the keyword: photos.

Ivo.Radulovski’s picture

Ivo.Radulovski commented
Status: Active » Postponed

Hi and thanks for your interest in glossify, i suspect this could be related to the "only first" bug. I will try to update the module today and would like to ask you to give the next dev release a shot.

omerida’s picture

omerida commented

Fixed in #917366: Performance patch - the way the replacement function works, it was only replacing inside the deepest child of each paragraph, in a sense. Once you add a link to a paragraph, it would only look to replace within that A tag, if that makes sense.

Ivo.Radulovski’s picture

Ivo.Radulovski commented

Yes i noticed that, it seems like simplehtmldom does not (as i at first thought) have a seperate text-node, i've already worked on a fix but i'll see how far we can merge our fixes, thanks a lot however for all your effort. I will try to get working on glossify asap.

WorldFallz’s picture

WorldFallz commented
Status: Postponed » Closed (cannot reproduce)

seems to be fixed in the current dev.