Thanks for this module. Although I tested both stable and dev version and convert limit is set to 1, returned links in node page set to unlimited.

CommentFileSizeAuthor
#1 bug_1872624_convert_limit.patch1.08 KBparavibe

Comments

paravibe’s picture

StatusFileSize
new1.08 KB

Hello,

Thank you for testing. Tommorow will be a new dev release and this patch is already applied.

paravibe’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, bug_1872624_convert_limit.patch, failed testing.

paravibe’s picture

Status: Needs work » Needs review

#1: bug_1872624_convert_limit.patch queued for re-testing.

paravibe’s picture

Status: Needs review » Closed (fixed)

Path test fail because its already applied.

1mundus’s picture

Reopening this, because it's only partly fixed.

Let's imagine that you have a text with multiple words that have to be "converted" with this module, e. g.:

First, Second, Third, First, Second, Third

If you have "0" option on for link limit, everything works fine. When you change it to 1, it will convert only one link on the page, not one instance of each word. In the previous example, only first occurence of "First" would be converted to a link.

1mundus’s picture

Status: Closed (fixed) » Active
paravibe’s picture

Yes you are right! I will try to fix this.

paravibe’s picture

Status: Active » Fixed

Hello again,
Fixed in dev.

Status: Fixed » Closed (fixed)

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