Trac should not detect GetMouse() as wiki entry

[axeld]

When we use function names in the text (like GetMouse()), Trac will turn "GetMouse" into a missing page. Even worse, when I search for "GetMouse" the search returns empty, even though there are several occurences in the bug descriptions (for example bug #901).

[wkornewald]

You can escape GetMouse by putting a "!" in front: GetMouse

The search field at the top acts as a quickjump if it detects a certain pattern (you can also enter revision numbers like hrev123 or ticket numbers like #1161 there). In this case, Trac recognizes a wiki word and tries to bring you to the wiki page. In order to work around this, you can enter GetMouse ("!" at the beginning disables the quickjump when searching).

Is that good enough? It can only be disabled by hacking the Trac source, but it might be a one-liner, so not too difficult. OTOH, the less modifications we have the better. And the quickjump can be very handy. :)

[wkornewald]

Well, if you insist on it you can still reopen the ticket. Otherwise, there's no reason to keep it open. :)

[axeld]

I think this functionality is misplaced in the search field - it should be the other way around, ie. it should only do a quickjump if you prepend the name with '!'. Also, it shouldn't detect names when they end in ().

[wkornewald]

Replying to axeld:

I think this functionality is misplaced in the search field - it should be the other way around, ie. it should only do a quickjump if you prepend the name with '!'.

Done. BTW, the search results still show a hint that a quickjump is possible. With '!' you can explicitly do a quickjump.

Also, it shouldn't detect names when they end in ().

I don't understand that one. I think my patch makes this change superfluous, anyway.

[axeld]

Replying to wkornewald:

Done. BTW, the search results still show a hint that a quickjump is possible. With '!' you can explicitly do a quickjump.

Thanks, that definitely sounds like the best solution.

Also, it shouldn't detect names when they end in ().

I don't understand that one. I think my patch makes this change superfluous, anyway.

Go to #901 comment 7 and see what it does to GetMouse(). That only makes the text harder to read, and doesn't server any purpose :-) But anyway, the search thing was the most important.

[wkornewald]

Replying to axeld:

Replying to wkornewald:

Done. BTW, the search results still show a hint that a quickjump is possible. With '!' you can explicitly do a quickjump.

Thanks, that definitely sounds like the best solution.

I had to change '!' to ':' because otherwise it's inconsistent. '!' is used to prevent "wikifying".

Also, it shouldn't detect names when they end in ().

I don't understand that one. I think my patch makes this change superfluous, anyway.

Go to #901 comment 7 and see what it does to GetMouse(). That only makes the text harder to read, and doesn't server any purpose :-)

Well, here I can't change anything because that's normal wiki syntax. What you can do is prefix wiki words with '!', so GetMouse() doesn't get transformed.

BTW, you can write comment:ticket:901:7 (weird syntax :).