Allow browser and repos aliases for source

[Ryan J Ollos]

Would it be possible to add the browser and repos aliases for source?: TracLinks#VersionControlrelatedlinks.

There may also be other aliases that could be added in process_request​.

[Peter Suter]

Replying to rjollos:

Would it be possible to add the browser and repos aliases for source?: TracLinks#VersionControlrelatedlinks.

Sure, I only added what I typically use myself. But patch welcome. :)

There may also be other aliases that could be added

The only other alias documented o TracLinks seems to be bug:. (But wasn't there also issue:?)

in process_request​.

Wouldn't adding them to the Javascript match regexes​ be enough?

[Peter Suter]

(But wasn't there also issue:?)

Ah that's new in Trac 1.2: ​t:ticket:12192

[Ryan J Ollos]

My JavaScript skills are pretty mediocre, but here's a first attempt at a patch: t13048.diff​.

[Ryan J Ollos]

I forgot to include in patch, bump version to 1.2.

[Peter Suter]

Thanks for the patch. However I don't think the version test for issue: is worth it. Sorry for the wasted effort. (Also it would fail for Trac 2.0 ;)) So I propose just going with this:

wikiautocomplete.js

@@ -164 +164 @@
         },
 
         { // Tickets
-            match: /((?:^|[^{])#|\bticket:)(\d*)$/,
+            match: /((?:^|[^{])#|\bticket:|\bbug:|\bissue:)(\d*)$/,
             search: search('ticket'),
             index: 2,
             template: function (ticket, term) {
@@ -208 +208 @@
         },
 
         { // Source
-            match: /\b(source:|log:)([^@\s]*(?:@\S*)?)$/,
+            match: /\b(source:|browser:|repos:|log:)([^@\s]*(?:@\S*)?)$/,
             search: search('source'),
             index: 2,
             template: template,

[Ryan J Ollos]

In case you are waiting for me before committing, I have no preference. I just want to have the aliases available and hope to have trac-hacks.org on Trac release 1.2 soon.

[Peter Suter]

Thanks, committed in r16221.