Fork me on GitHub
Subscribe 5

Ticket #530 (fixed bug)

End slash for url is outside the bbcode

  • Created: 2011-11-02 15:35:06
  • Reported by: daris
  • Assigned to: Reines
  • Milestone: 1.4.8
  • Component: parser
  • Priority: normal

History

daris 2011-11-02 18:10:49

Most of users/devs don't read test forum, submit a ticket next time smile

quy 2011-12-02 17:05:16

In Jeff's new parser, it has a linkify function that fixes this issue. OK to use it instead of do_clickable?

Franz 2011-12-02 19:58:37

If it takes care of everything the current do_clickable() does, then sure, go ahead.

Franz 2011-12-02 20:43:12

  • Owner set to quy.

quy 2011-12-05 04:27:52

Commit bc3a86a to fluxbb fluxbb-1.4

#530: Fixed ending slash from being placed outside of the URL tag

quy 2011-12-05 04:28:56

  • Status changed from open to fixed.

Reines 2012-01-04 12:35:27

  • Status changed from fixed to open.

Seems to have introduced a new issue: https://fluxbb.org/forums/viewtopic.php … 015#p44015

Reines 2012-01-04 23:03:28

Working fine locally for me too :s

Reines 2012-01-05 15:34:46

Commit fcf840c to fluxbb fluxbb-1.4

Hopefully fixing parsing URLs ending in slashes. #530

Reines 2012-01-05 15:50:50

  • Status changed from open to fixed.

I think this seems to have fixed it.

quy 2012-01-05 16:32:22

On my Linux server (PHP 5.1.6, PCRE 6.6 06-Feb-2006), the setting to make clickable links is enabled, but it is not doing it. This is happening with v1.4.7 too. Could the issue be related to using an older version of PCRE?

quy 2012-01-05 16:37:06

I am still on PunBB 1.2.23 and plan to upgrade to FluxBB 1.4.8 shortly. "Make clickable links" is working with the PunBB install which is on the same server as FluxBB 1.4.7/1.4.8.

Reines 2012-01-05 16:56:51

It is possible I guess. The regular expression has been changed since 1.2 to make it UTF-8 compatible (as well as some other changes by the look of it).

Is there some way you can check if your PHP/PCRE supports unicode character properties? (i.e. /p{L}).

Reines 2012-01-05 17:17:21

  • Owner changed from quy to Reines.

quy 2012-01-05 17:20:47

This maybe the case. I will check with my hosting provider. Thanks.