Use 'user settings' for reporter/editor in Tickets and Wiki

[daniel]

Trac should automatically use name and email from user settings/prefs to fill out contact information for tickets, wiki edits and comments.

[daniel]

Fixed in [619].

[cap]

util.get_reporter_id() simply returns the authname if a user is authenticated. I can't see any code to match up authnames with email addresses elsewhere, so it seems that authenticated users can't get email notification. A fix for #464 would fix this, but it seems like a better solution to both tickets would use the email session var regardless of authentication status.

[daniel]

I believe the previous reopening is due to a misunderstanding.

The ticket refers to the email address stored as a session variable, which is separate from authentication, and matched up in Session.py.

See [617] (the chunk for macros.cs especially).

[cap]

I was a unsure about reopening this, but [617] doesn't change my original complaint. Let me describe the behavior I was objecting to:

Without being authenticated, I click on the settings link and enter my full name and email address (Cap Petschulat <cap@…>). I create a new ticket and the reporter field is filled in automatically with name and email (I think via the call to get_reporter_id). This is as expected.

Now I authenticate myself (username cap) via the login link. I click on settings and fill in a name and email address. When I create a new ticket, the reporter field is populated with my authname alone--no email address.

If session variables really should be ignored in this case, the option to set them should probably be disabled once the user is authenticated. My statement that authenticated users can't receive email notification was overly dramatic, but I still think something is off here.

[cap]

After looking more deeply, you are right in noting that [617] behaves correctly. Unfortunately, we also have [653], which causes the issue I've raised. I see now that the chunk in macros.cs was changed to match the logic of get_reporter_id, which simply returns the authname when the user is not anonymous. Unless there's some good reason for the changes in [653] (which deals with an unrelated problem), I'll submit a patch tomorrow to revert back to the behavior of [617].

[jonas]

The reason why the ticket and wiki author is set to the authname instead of the name and email is because it's less likely to change. I think this is the best we can do until we add a user table to the database.

[richard.leach@…]

Is there an easy (e.g. trac.ini parameter) way of reverting to the behavior of having to type in the reporter when creating a new ticket? (Or having a tick box "I am the Reporter"?)

In our setup, we'll often create tickets on behalf of people who will want to recieve email notifications (since they are the real Reporters). Having to create a ticket, then immediately edit it to change the Reporter (and remembering to do so) is a pain!

[cboos]

What prevents you from clicking the logout link before creating a ticket on behalf of someone?

[cmlenz]

Please file a separate ticket. AFAICT the issue is not really related to this ticket.