From 5ae0cbaa551ca1457bb9b84ccad72836db1bf5fc Mon Sep 17 00:00:00 2001 From: Elrond Date: Thu, 25 Apr 2013 14:16:17 +0200 Subject: [PATCH] No lazy_* needed here; Add ReallyLazyProxy.__repr__. One should use lazy_* only if you have a good reason. This one found by our unit tests! For example add_message adds the message to the session, the session needs to be serialized (with json) and well, LazyProxy is not serializable. To aid in debugging, gave our ReallyLazyProxy a __repr__. --- mediagoblin/edit/views.py | 2 +- mediagoblin/tools/translate.py | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/mediagoblin/edit/views.py b/mediagoblin/edit/views.py index b0d8dd5d..34b7aaca 100644 --- a/mediagoblin/edit/views.py +++ b/mediagoblin/edit/views.py @@ -29,7 +29,7 @@ from mediagoblin.decorators import (require_active_login, active_user_from_url, get_media_entry_by_id, user_may_alter_collection, get_user_collection) from mediagoblin.tools.response import render_to_response, redirect -from mediagoblin.tools.translate import lazy_pass_to_ugettext as _ +from mediagoblin.tools.translate import pass_to_ugettext as _ from mediagoblin.tools.text import ( convert_to_tag_list_of_dicts, media_tags_as_string) from mediagoblin.tools.url import slugify diff --git a/mediagoblin/tools/translate.py b/mediagoblin/tools/translate.py index 00677863..b20e57d1 100644 --- a/mediagoblin/tools/translate.py +++ b/mediagoblin/tools/translate.py @@ -50,6 +50,13 @@ class ReallyLazyProxy(LazyProxy): def value(self): return self._func(*self._args, **self._kwargs) + def __repr__(self): + return "<%s for %s(%r, %r)>" % ( + self.__class__.__name__, + self._func, + self._args, + self._kwargs) + def locale_to_lower_upper(locale): """ -- 2.25.1