Author Last,First displayed without comma

[onomou]

The name of a question author or ancestor is displayed as Last,First without any space after the comma, as is the list of question ancestors. This appears in the question editor and question preview. I could not find any discussion about this, so here it is. I think it would be best to follow convention and put a space afterward.

Example:

Places in the code I found this happen:

• IMathAS/course/moddataset.php

  Line 1034 in 8eb72ac

  Author: <?php echo Sanitize::encodeStringForDisplay($line['author']); ?> <input type="hidden" name="author" value="<?php echo Sanitize::encodeStringForDisplay($author); ?>">

• IMathAS/course/testquestion.php

  Line 321 in ef54add

  Sanitize::encodeStringForDisplay($line['author']));

• IMathAS/course/testquestion2.php

  Line 431 in 37c7973

  Sanitize::encodeStringForDisplay($line['author']));

Calls that produce a list without spaces after the commas:

• Sanitize::encodeStringForDisplay($line['author']))
• Sanitize::encodeStringForDisplay($line['ancestors'])
• Sanitize::encodeStringForDisplay($line['ancestorauthors'])

[drlippman]

The author name is stored that way in moddataset, line 554. Those other places are just displaying the stored value.

I agree it would be better with a space, but it would cause some challenges with matching previous author names to prevent duplicates when edits are made. But I might be able to work around that.