#667: Error rendering forbidden pages
------------------------+---------------------------------
Reporter: olemis | Owner: rjollos
Type: defect | Status: review
Priority: blocker | Milestone: Release 8
Component: ui design | Version: 0.7.0
Resolution: | Keywords: permissions, chrome
------------------------+---------------------------------
Comment (by rjollos):
The patch appears to be effectively the same as the one in comment:3. Is
there a difference I'm not seeing, or just style differences? I'm fine
with using the `.get('mainnav', []))` since I think it's cleaner.
Is the `setdefault` needed? There won't be any iteration if `mainnav`
isn't in the dictionary, so it doesn't appear that the line with
`setdefault` would even be hit.
--
Ticket URL: <https://issues.apache.org/bloodhound/ticket/667#comment:9>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker
