minor changes from aaronw's review

diff --git a/mediagoblin/app.py b/mediagoblin/app.py
index 982e570c146a2a1346ab6f2a0a6f6a7a39e93025..ecd3de0e677331f56908cabd01c4c1c1d4009887 100644 (file)
--- a/mediagoblin/app.py
+++ b/mediagoblin/app.py
@@ -100,6 +100,8 @@ class MediaGoblinApp(object):
 
         # Check if authentication plugin is enabled and respond accordingly.
         self.auth = check_auth_enabled()
+        if not self.auth:
+            app_config['allow_comments'] = False
 
         # Set up storage systems
         self.public_store, self.queue_store = setup_storage()

diff --git a/mediagoblin/templates/mediagoblin/bits/frontpage_welcome.html b/mediagoblin/templates/mediagoblin/bits/frontpage_welcome.html
index 261536940c40a5d3c33bda5f42bd46f0f45dd37b..9ef28a4d9082a951462723dd363d6fdea45dd92b 100644 (file)
--- a/mediagoblin/templates/mediagoblin/bits/frontpage_welcome.html
+++ b/mediagoblin/templates/mediagoblin/bits/frontpage_welcome.html
@@ -22,17 +22,17 @@
     <h1>{% trans %}Hi there, welcome to this MediaGoblin site!{% endtrans %}</h1>
     <img class="right_align" src="{{ request.staticdirect('/images/frontpage_image.png') }}" />
     <p>{% trans %}This site is running <a href="http://mediagoblin.org">MediaGoblin</a>, an extraordinarily great piece of media hosting software.{% endtrans %}</p>
-               {% if auth %}
+    {% if auth %}
     <p>{% trans %}To add your own media, place comments, and more, you can log in with your MediaGoblin account.{% endtrans %}</p>
     {% if allow_registration %}
       <p>{% trans %}Don't have one yet? It's easy!{% endtrans %}</p>
       {% trans register_url=request.urlgen('mediagoblin.auth.register') -%}
         <a class="button_action_highlight" href="{{ register_url }}">Create an account at this site</a>
         or
-                       {%- endtrans %}
-               {% endif %}
-               {% endif %}
-                       {% trans %}
+      {%- endtrans %}
+    {% endif %}
+    {% endif %}
+      {% trans %}
         <a class="button_action" href="http://wiki.mediagoblin.org/HackingHowto">Set up MediaGoblin on your own server</a>
       {%- endtrans %}
 

diff --git a/mediagoblin/templates/mediagoblin/user_pages/media.html b/mediagoblin/templates/mediagoblin/user_pages/media.html
index 5420752f859210889e585e91dd0f50fe2691cd07..fb892fd788ab1e364a2eea976dcd33009c594fd3 100644 (file)
--- a/mediagoblin/templates/mediagoblin/user_pages/media.html
+++ b/mediagoblin/templates/mediagoblin/user_pages/media.html
@@ -86,7 +86,7 @@
       <p>{{ media.description_html }}</p>
     {% endautoescape %}
     {% if comments %}
-      {% if app_config['allow_comments'] and auth %}
+      {% if app_config['allow_comments'] %}
         <a
           {% if not request.user %}
             href="{{ request.urlgen('mediagoblin.auth.login') }}"