1

hide OpenID mentions when LDAP is activated

I think that LDAP is used only into companies. In my case that I imagine to be the most frequent one in companies, if LDAP is used there is no other provider activated.

So I would like to remove the panel on the right explaining why OpenID is good for me.

I think that I can do it myself by creating a custom skin and updating the file templates/authopenid/signin.html but in my opinion it should be done natively if ldap is activated.

Evgeny's avatar
13.2k
Evgeny
updated 2011-05-12 13:11:58 -0600
Samuel's avatar
425
Samuel
asked 2011-05-12 10:00:56 -0600
edit flag offensive 0 remove flag close merge delete

Comments

add a comment see more comments

3 Answers

1

done in 0.6.85 those messages are automatically hidden when there are no login "buttons".

Evgeny's avatar
13.2k
Evgeny
answered 2011-05-12 13:11:20 -0600
edit flag offensive 0 remove flag delete link

Comments

add a comment see more comments
1

The message displayed @ logout had been missed. Here is the patch :

diff --git a/askbot/skins/default/templates/logout.html b/askbot/skins/default/templates/logout.html
index d9ab69f..2cdf495 100644
--- a/askbot/skins/default/templates/logout.html
+++ b/askbot/skins/default/templates/logout.html
@@ -3,13 +3,15 @@
 {% block title %}{% spaceless %}{% trans %}Logout{% endtrans %}{% endspaceless %}{% endblock %}
 {% block content %}
 <h1>{% trans %}You have successfully logged out{% endtrans %}</h1>
-<p>{% trans %}However, you still may be logged in to your OpenID provider. Please logout of your provider if you wish to do so.{% endtrans %}</p>
-{% if settings.FACEBOOK_KEY and settings.FACEBOOK_SECRET %}
-    <div id="fb-root"></div>
-    <script src="http://connect.facebook.net/en_US/all.js"></script>
-    <script>
-        FB.init({appId: '{{settings.FACEBOOK_KEY}}', status: true, cookie: true, xfbml: true});
-    </script>
+{% if have_buttons %}
+    <p>{% trans %}However, you still may be logged in to your OpenID provider. Please logout of your provider if you wish to do so.{% endtrans %}</p>
+    {% if settings.FACEBOOK_KEY and settings.FACEBOOK_SECRET %}
+        <div id="fb-root"></div>
+        <script src="http://connect.facebook.net/en_US/all.js"></script>
+        <script>
+            FB.init({appId: '{{settings.FACEBOOK_KEY}}', status: true, cookie: true, xfbml: true});
+        </script>
+    {% endif %}
 {% endif %}
 {% endblock %}
 {% block endjs %}
Samuel's avatar
425
Samuel
answered 2011-06-01 10:08:24 -0600
edit flag offensive 0 remove flag delete link

Comments

You still need to initialize the "have_buttons" variable :), in your case it will be always false.
Evgeny's avatar Evgeny (2011-06-02 01:13:23 -0600) edit
pushed this to the repo, with some extra refactoring.
Evgeny's avatar Evgeny (2011-06-02 02:07:21 -0600) edit
I have reverted my local update and integrate your last commits from the github:master. It works fine. Thanks.
Samuel's avatar Samuel (2011-06-03 09:17:57 -0600) edit
add a comment see more comments
0

thanks. verified.

Samuel's avatar
425
Samuel
answered 2011-05-13 04:05:02 -0600
edit flag offensive 0 remove flag delete link

Comments

add a comment see more comments