Hotfix bbcode regex by alexdryden · Pull Request #5 · UIUCLibrary/archon

alexdryden · 2026-03-20T14:27:15Z

No description provided.

This fixes a number of issues with the substitution parsing. If further work is needed, we should investigate a library like github.com/chriskonnertz/bbcode. This commit includes fixes for: -match only matching tags -fix substitution syntax -allow https

packages/core/lib/archonobject.inc.php

test_bbcode.php

Co-authored-by: graykr <graykr@users.noreply.github.com>

alexdryden · 2026-03-22T18:32:37Z

I think ideally we would not call this bbcode_to_html because it doesn't produce html, but that probably isn't really strictly necessary.

@graykr After applying your suggestions I was getting some double mailto:mailto. Probably the easiest way to make sure this is really doing what you expect is to look at the test cases, which I made a bit easier to read. And adding/correcting any mistakes in the test cases might be the clearest way to indicate the expected behavior.

test_bbcode.php

These are some other ways that email links can appear Co-authored-by: graykr <graykr@users.noreply.github.com>

alexdryden added 3 commits March 19, 2026 17:45

fix: various issues with regex

590d66b

This fixes a number of issues with the substitution parsing. If further work is needed, we should investigate a library like github.com/chriskonnertz/bbcode. This commit includes fixes for: -match only matching tags -fix substitution syntax -allow https

chore: add some ad hock tests for bbcode_to_html

92cc2ee

refactor: combine mail and email patterns

338bd2c