Consider having an explicit return of a variable at the end of the function rather than a single nested return.

The `href` temporary variable could also be removed now, something like;

```
var markUp;
if (record.pointOfTruthLink) {
    markUp = this._makeExternalLinkMarkup(record.pointOfTruthLink.href, OpenLayers.i18n('metadataLinkText'));
}

return markUp;
```