Wrong users highlighted as core devs in trac

[Alex Couper]

I assigned a ticket to myself anonymously and was given Core status:

https://code.djangoproject.com/ticket/20649#comment:4

I've now reassigned it to a registered version of me and all is well.

[Baptiste Mispelon]

The highlighting of core developers is done client-side with javascript.

As it turns out, the regular expression used to detect usernames stops at whitespace so your username ends up being Alex, a username belonging to Alex Gaynor who is indeed a core developer.

I'll look into this and see if the regex can be fixed or if we need to change something in trac.

Thanks.

[Aymeric Augustin]

Computer.

:D

[Baptiste Mispelon]

After digging a bit deeper into this issue, I'm starting to think the only sane fix would be to make everone named Alex a core developer...

The problem we have is that trac sometimes likes to use the full name of a user instead of his/her username when displaying a comment.
This means that we can't check reliably whether a user is a core dev or not.

I "improved" the username-extracting code to reject everything containing an "@" character.
This won't fix the core issue, but it should at least get rid of the false positives.

I hope you enjoyed your "core dev" highlight while it lasted ;)

​https://github.com/django/code.djangoproject.com/pull/7

[Aymeric Augustin]

I applied the patch and I confirmed it fixes the issue.