Only render visibility for authenticated users
Review Request #7302 — Created May 15, 2015 and submitted — Latest diff uploaded
Previously, when an unauthenticated user attempted to view the 'All
Review Requests' page, every summary would beNone
. This was because
theSummaryColumn
would attempt to evalute the
review_request.visibility
attribute, but it is only defined when the
user is authenticated. This would lead to an exception being raised in
the rendering andNone
being returned instead. This patch fixes the
issue by only attempting to render the review request visibility when
the user viewing the datagrid is authenticated.
The archived and muted state is correctly shown.
The summaries are correctly rendered in the datagrid.