Work around a page breakage with open_an_issue.

Review Request #4193 — Created June 1, 2013 and submitted

Information

Review Board
release-1.7.x

Reviewers

Work around a page breakage with open_an_issue.

Some users have hit an odd problem where the open_an_issue profile field
was returning None. This indicates possibly that there was an error
fetching the field.

The only thing I can think of is that 'user', in this case, doesn't mean
what we think it means. Really, we should be using request.user and not
user anyway. It's been changed to use that. It's also now using a
conditional to output true/false instead of the hacky call to lower().
Reproduced the problem by replacing 'get_profile' with 'get_profile2' in the
template. With lower(), the result is blank, causing a JavaScript error.
With the conditional, I at least get false.
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
      Ignored Files:
        reviewboard/templates/reviews/reviewable_base.html
    
    
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-1.7.x (eaccbd4)
QU
  1. adsfjalsdjflaksjflkadf
    
    1. Please do not use this server for testing. You can use http://demo.reviewboard.org (guest:guest) for that.
  2. 
      
Loading...