Possibility to use username if full name is missing in the comments framework

[Thejaswi Puthraya]

The ticket summary mentions it all. After a discussion with Jacob on IRC, it was decided to have this though having the full name is always recommended.

[Thejaswi Puthraya]

git-patch of the latest checkout

[Ben Spaulding]

I ran into the same issue and found this ticket. Just to reiterate, the view should use the get_full_name() or username because that’s what the model itself does. It just makes sense.

I have attached my patch just because I already had it made and it’s a little shorter. (I don’t care what color the bike shed it is, really.)

[Ben Spaulding]

A little shorter version.

[Malcolm Tredinnick]

*sigh* You both should be smacked for not providing a test for the fix. :-(

Fortunately, I also stumbled across this and have a local patch with tests that I'll commit shortly.

[Malcolm Tredinnick]

(In [9118]) Fixed #8803 -- Allow authenticated users without first_name/last_name values set to post comments.

[Malcolm Tredinnick]

(In [9125]) [1.0.x] Fixed #8803 -- Allow authenticated users without first_name/last_name values set to post comments.

Backport of r9118 from trunk.