Mercurial > kallithea
changeset 1643:bdb6fbc8265f
implements #293 gravatar link should be disabled when use_gravatar = false
author | Marcin Kuzminski <marcin@python-works.com> |
---|---|
date | Mon, 31 Oct 2011 23:10:11 +0200 |
parents | c0d8171ade36 |
children | 7e2695ffab3b |
files | rhodecode/lib/base.py rhodecode/lib/helpers.py rhodecode/templates/admin/users/user_edit.html rhodecode/templates/admin/users/user_edit_my_account.html |
diffstat | 4 files changed, 24 insertions(+), 16 deletions(-) [+] |
line wrap: on
line diff
--- a/rhodecode/lib/base.py Sun Oct 30 20:30:24 2011 +0200 +++ b/rhodecode/lib/base.py Mon Oct 31 23:10:11 2011 +0200 @@ -10,6 +10,7 @@ from pylons.templating import render_mako as render from rhodecode import __version__ +from rhodecode.lib import str2bool from rhodecode.lib.auth import AuthUser from rhodecode.lib.utils import get_repo_slug from rhodecode.model import meta @@ -24,6 +25,7 @@ def __before__(self): c.rhodecode_version = __version__ c.rhodecode_name = config.get('rhodecode_title') + c.use_gravatar = str2bool(config.get('use_gravatar')) c.ga_code = config.get('rhodecode_ga_code') c.repo_name = get_repo_slug(request) c.backends = BACKENDS.keys() @@ -32,8 +34,6 @@ self.sa = meta.Session() self.scm_model = ScmModel(self.sa) - #c.unread_journal = scm_model.get_unread_journal() - def __call__(self, environ, start_response): """Invoke the Controller""" # WSGIController.__call__ dispatches to the Controller method @@ -56,11 +56,13 @@ class BaseRepoController(BaseController): """ - Base class for controllers responsible for loading all needed data - for those controllers, loaded items are + Base class for controllers responsible for loading all needed data for + repository loaded items are - c.rhodecode_repo: instance of scm repository (taken from cache) - + c.rhodecode_repo: instance of scm repository + c.rhodecode_db_repo: instance of db + c.repository_followers: number of followers + c.repository_forks: number of forks """ def __before__(self): @@ -76,7 +78,6 @@ redirect(url('home')) - c.repository_followers = \ - self.scm_model.get_followers(c.repo_name) + c.repository_followers = self.scm_model.get_followers(c.repo_name) c.repository_forks = self.scm_model.get_forks(c.repo_name)
--- a/rhodecode/lib/helpers.py Sun Oct 30 20:30:24 2011 +0200 +++ b/rhodecode/lib/helpers.py Mon Oct 31 23:10:11 2011 +0200 @@ -454,8 +454,8 @@ #============================================================================== def gravatar_url(email_address, size=30): - if not str2bool(config['app_conf'].get('use_gravatar')) or \ - email_address == 'anonymous@rhodecode.org': + if (not str2bool(config['app_conf'].get('use_gravatar')) or + not email_address or email_address == 'anonymous@rhodecode.org'): return url("/images/user%s.png" % size) ssl_enabled = 'https' == request.environ.get('wsgi.url_scheme')
--- a/rhodecode/templates/admin/users/user_edit.html Sun Oct 30 20:30:24 2011 +0200 +++ b/rhodecode/templates/admin/users/user_edit.html Mon Oct 31 23:10:11 2011 +0200 @@ -29,10 +29,13 @@ <div class="field"> <div class="gravatar_box"> <div class="gravatar"><img alt="gravatar" src="${h.gravatar_url(c.user.email)}"/></div> - <p> - <strong>${_('Change your avatar at')} <a href="http://gravatar.com">gravatar.com</a></strong><br/> - ${_('Using')} ${c.user.email} - </p> + <p> + %if c.use_gravatar: + <strong>${_('Change your avatar at')} <a href="http://gravatar.com">gravatar.com</a></strong> + <br/>${_('Using')} ${c.user.email} + %else: + <br/>${c.user.email} + %endif </div> </div> <div class="field">
--- a/rhodecode/templates/admin/users/user_edit_my_account.html Sun Oct 30 20:30:24 2011 +0200 +++ b/rhodecode/templates/admin/users/user_edit_my_account.html Mon Oct 31 23:10:11 2011 +0200 @@ -29,8 +29,12 @@ <div class="gravatar_box"> <div class="gravatar"><img alt="gravatar" src="${h.gravatar_url(c.user.email)}"/></div> <p> - <strong>${_('Change your avatar at')} <a href="http://gravatar.com">gravatar.com</a></strong><br/> - ${_('Using')} ${c.user.email} + %if c.use_gravatar: + <strong>${_('Change your avatar at')} <a href="http://gravatar.com">gravatar.com</a></strong> + <br/>${_('Using')} ${c.user.email} + %else: + <br/>${c.user.email} + %endif </p> </div> </div>