LoCo Team Portal

Merge lp://staging/~mhall119/loco-team-portal/fixes-589226 into lp://staging/loco-team-portal

  1. fixes-589226
  2. Merge into 0.2
Proposed by Michael Hall
Status: Merged
Merged at revision: 145
Proposed branch: lp://staging/~mhall119/loco-team-portal/fixes-589226
Merge into: lp://staging/loco-team-portal
Diff against target: 12 lines (+1/-1)
1 file modified
loco_directory/events/views.py (+1/-1)
To merge this branch: bzr merge lp://staging/~mhall119/loco-team-portal/fixes-589226
Reviewer Review Type Date Requested Status
Dave Walker Approve
Review via email: mp+26722@code.staging.launchpad.net

Description of the change

AnonymousUser doesn't have group associations, so calling request.user.groups, when the user is not logged in, throws this error. I added a check that request.user.is_authenticated() before doing that, to short-circuit the if condition before it tries that.

To post a comment you must log in.
Revision history for this message
Dave Walker (davewalker) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches