Always show usernames in reaction tooltips

[n0toose]

Even if GetDisplayName() is normally preferred elsewhere, this change
provides more consistency, as usernames are also always being shown
when participating in a conversation taking place in an issue or
a pull request. This change makes conversations easier to follow, as
you would not have to have a mental association between someone's
username and someone's real name in order to follow what is happening.

This behavior matches GitHub's. Optimally, both the username and the
full name (if applicable) could be shown, but such an effort is a
much bigger task that needs to be thought out well.

[n0toose]

Screenshot:

I may have forgotten to update a test, I'll push another commit soon if that's the case.

EDIT: Nope, passes locally.

[silverwind]

Shouldn't pass locally :)

--- FAIL: TestIssueReactionCount (0.03s)
    reaction_test.go:112: 
        	Error Trace:	/home/runner/work/gitea/gitea/models/issues/reaction_test.go:112
        	Error:      	Not equal: 
        	            	expected: "User One, < U<se>r Tw<o > ><"
        	            	actual  : "user1, user2"
        	            	
        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-User One, < U<se>r Tw<o > ><
        	            	+user1, user2
        	Test:       	TestIssueReactionCount

[n0toose]

Shouldn't pass locally :)

That's because I did make test-sqlite! THAT was a pitfall, a mistake on my end makes much more sense.