Merge pull request #3439 from stsewd/dont-show-build-ideas

Don't show "build ideas" to unprivileged users

Author: ericholscher

readthedocs/templates/builds/build_detail.html

@@ -97,26 +97,28 @@
       </span>
     </div>
 
-    {% if not build.success and build.commands.count < 4 %}
-      <div class="build-ideas">
-        <p>
-          {% url 'wipe_version' build.version.project.slug build.version.slug as wipe_url %}
-          {% blocktrans %}
-            Having trouble with your build environment?
-            Try <a href="{{ wipe_url }}">reseting it</a>.
-          {% endblocktrans %}
-        </p>
-      </div>
-    {% elif not build.success and "setup.py install" in build.commands.last.output %}
-      <div class="build-ideas">
-        <p>
-          {% url 'projects_advanced' build.version.project.slug as advanced_url %}
-          {% blocktrans %}
-            Don't want <em>setup.py install</em> called?
-            Change the <strong>Install Project</strong> setting in your <a href="{{ advanced_url }}">advanced settings</a>.
-          {% endblocktrans %}
-        </p>
-      </div>
+    {% if request.user|is_admin:project %}
+      {% if not build.success and build.commands.count < 4 %}
+        <div class="build-ideas">
+          <p>
+            {% url 'wipe_version' build.version.project.slug build.version.slug as wipe_url %}
+            {% blocktrans %}
+              Having trouble with your build environment?
+              Try <a href="{{ wipe_url }}">reseting it</a>.
+            {% endblocktrans %}
+          </p>
+        </div>
+      {% elif not build.success and "setup.py install" in build.commands.last.output %}
+        <div class="build-ideas">
+          <p>
+            {% url 'projects_advanced' build.version.project.slug as advanced_url %}
+            {% blocktrans %}
+              Don't want <em>setup.py install</em> called?
+              Change the <strong>Install Project</strong> setting in your <a href="{{ advanced_url }}">advanced settings</a>.
+            {% endblocktrans %}
+          </p>
+        </div>
+      {% endif %}
     {% endif %}
 
     {% if build.output %}