Summary

Only render visible forms

Review Request #8726 — Created Feb. 8, 2017 and submitted Feb. 9, 2017, 3:47 p.m.

Information

Owner

brennie

Repository

Djblets

Branch

release-0.8.x

Bugs

Depends On

Blocks

8729

Reviewers

Groups

djblets

People

Description

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.

Testing Done

Ran unit tests.

Expand All
Collapse All

Tool: PEP8 Style Checker
Processed Files:
    djblets/configforms/pages.py



Tool: Pyflakes
Processed Files:
    djblets/configforms/pages.py

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-0.8.x (53e0926)

Status:: Re-opened

Status:: Completed
Change Summary:: Pushed to release-0.8.x (c013e92)