#5872 closed enhancement (fixed)
auto-linking for semicolon-separated lists in xprofile
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.8 | Priority: | normal |
Severity: | normal | Version: | 1.1 |
Component: | Extended Profile | Keywords: | has-patch |
Cc: | jon.reeve@… |
Description
Xprofile will automatically create hyperlinks out of many comma-separated lists, but doesn't behave well when semicolons are used instead of or in conjunction with commas.
A lot of our users use semicolons to separate items in their interests lists, especially when some of those items contain commas:
Interests: interest one; interest two; comma, interest three.
A solution would be to allow semicolons as delineators in addition to commas. I'll submit a patch for this.
Since I haven't submitted many patches to BP yet, please let me know if I'm doing the patch submission correctly.
Attachments (3)
Change History (15)
#6
@
8 years ago
- Keywords has-patch added; needs-refresh removed
Updated patch to almost match what @jreeve provided originally, with some slight codesniffer cleanups.
#7
@
8 years ago
- Milestone changed from Future Release to 2.8
Don't patch things and leave them in Future Release, they'll never get seen!
#8
@
8 years ago
- Keywords needs-patch added; has-patch removed
@tw2113 The only change I would suggest is that we check for commas before semi-colons. If a string has both, splitting it on the commas will match existing behaviour. (I think. Only had a quick read).
Hi @jreeve!
Thank you for the patch. :) It looks to be in the correct format and, at a quick glance, the code change itself seems to be pretty good. During this current release cycle (2.1), we had spoken a bit about the auto-linking behaviour in xprofile -- see #787 -- so this kind of sits together with that nicely.
In the end, we ran out of time/volunteers to get the changes discussed in that ticket ready for 2.1, so I'm going to move your ticket to our 2.2 milestone, which means we'll come back and revisit this ticket when we start work on the 2.2 cycle, which will probably be about a month from now. It'd be great to have your thoughts on the #787 ticket as well.
Thanks again for the patch.