Only render visible forms

Review Request #8726 - Created Feb. 8, 2017 and submitted

Barret Rennie
Djblets
release-0.8.x
8729
djblets

When rending a ConfigPagesView, we check for page visibility but we
never check for form visibility. This would lead to supposedly not
visible forms being rendered, but only if there was another form on the
page was visible. Presumably this wasn't caught earlier due to Review
Board only having one form per page.

Ran unit tests.

Review Bot
  1. Tool: PEP8 Style Checker
    Processed Files:
        djblets/configforms/pages.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        djblets/configforms/pages.py
    
    
  2. 
      
David Trowbridge
  1. Ship It!
  2. 
      
Barret Rennie
Barret Rennie
Barret Rennie
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.8.x (c013e92)
Loading...