From fc414d7962fcee878462416a9f5e61b16aedf0fd Mon Sep 17 00:00:00 2001
From: rebortg <github@ghlr.de>
Date: Wed, 2 Sep 2020 22:31:29 +0200
Subject: [PATCH] Fixes #5066: Permission for Report View

---
 netbox/extras/views.py             | 4 ++--
 netbox/templates/home.html         | 4 ++--
 netbox/templates/inc/nav_menu.html | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/netbox/extras/views.py b/netbox/extras/views.py
index cd813d280..56477a59f 100644
--- a/netbox/extras/views.py
+++ b/netbox/extras/views.py
@@ -315,7 +315,7 @@ class ReportListView(ContentTypePermissionRequiredMixin, View):
     Retrieve all of the available reports from disk and the recorded JobResult (if any) for each.
     """
     def get_required_permission(self):
-        return 'extras.view_reportresult'
+        return 'extras.view_report'
 
     def get(self, request):
 
@@ -347,7 +347,7 @@ class ReportView(ContentTypePermissionRequiredMixin, View):
     Display a single Report and its associated JobResult (if any).
     """
     def get_required_permission(self):
-        return 'extras.view_reportresult'
+        return 'extras.view_report'
 
     def get(self, request, module, name):
 
diff --git a/netbox/templates/home.html b/netbox/templates/home.html
index 85332ae1b..e06b67469 100644
--- a/netbox/templates/home.html
+++ b/netbox/templates/home.html
@@ -276,7 +276,7 @@
             <div class="panel-heading">
                 <strong>Reports</strong>
             </div>
-            {% if report_results and perms.extras.view_reportresult %}
+            {% if report_results and perms.extras.view_report %}
                 <table class="table table-hover panel-body">
                     {% for result in report_results %}
                         <tr>
@@ -285,7 +285,7 @@
                         </tr>
                     {% endfor %}
                 </table>
-            {% elif perms.extras.view_reportresult %}
+            {% elif perms.extras.view_report %}
                 <div class="panel-body text-muted">
                     None found
                 </div>
diff --git a/netbox/templates/inc/nav_menu.html b/netbox/templates/inc/nav_menu.html
index 23808bc04..bf3d349cc 100644
--- a/netbox/templates/inc/nav_menu.html
+++ b/netbox/templates/inc/nav_menu.html
@@ -518,7 +518,7 @@
                         <li{% if not perms.extras.view_script %} class="disabled"{% endif %}>
                             <a href="{% url 'extras:script_list' %}">Scripts</a>
                         </li>
-                        <li{% if not perms.extras.view_reportresult %} class="disabled"{% endif %}>
+                        <li{% if not perms.extras.view_report %} class="disabled"{% endif %}>
                             <a href="{% url 'extras:report_list' %}">Reports</a>
                         </li>
                     </ul>