Fixed thread_view content check

[Lawrence]

In the thread view template, hasContent for the bottom pageNavLinkGroup will always be true. This is caused by this line:

<div class="linkGroup"{xen:if '!{$ignoredNames}', ' style="display: none"'}><a href="javascript:" class="muted JsOnly DisplayIgnoredContent Tooltip" title="{xen:phrase show_hidden_content_by_x, "names={xen:helper implode, $ignoredNames, ', '}"}">{xen:phrase show_ignored_content}</a></div>

If you replace the above line with the following code, the hasContent check has a chance to be false (and this does not affect anything if a user is ignoring a member):

<xen:if is="{$ignoredNames}">
                <div class="linkGroup"><a href="javascript:" class="muted JsOnly DisplayIgnoredContent Tooltip" title="{xen:phrase show_hidden_content_by_x, "names={xen:helper implode, $ignoredNames, ', '}"}">{xen:phrase show_ignored_content}</a></div>
</xen:if>

Only a small bug, but no reason to perform a content check when it will always resolve to be true,

 

[Mike]

The approach used here is the standard approach for these ignore links, though I have to admit I don't really recall why. That said, there are a lot of other cases where this would be displayed even if this were changed, so removing the content check is probably the easiest.