On Tue, Apr 14, 2009 at 11:33 AM, Mitch Chapman <mchapman87...@gmail.com>wrote:
>
> Is it bad form in Django to add "transient" instance variables to
> model instances?
>
> I'm rendering a set of search results. Each result record is a User
> instance. I want to show whether or not each record is a friend of
> the user who is conducting the search.
>
> Right now my view is annotating each search result record to indicate
> whether or not it is a friend of the requesting user. This works, but
> I'm not sure what might break as a result of adding the instance
> variable:
>
> matched_users = User.objects.filter(...)
> friends_set = set([...])
> for u in matched_users:
> u.is_my_friend = (u.username in friends_set)
>
> I don't need to store this information permanently. I just want to
> use it to simplify template rendering, e.g.
>
> {% for user in matched_users %}
> ...
> {% if user.is_my_friend %}
> {{user.username}} is already your friend.
> {% endif %}
> ...
>
> Is there a different, recommended way to do this? Thanks for the help!
> >
>
In my view this is perfectly acceptable. Django is just python and that's
completely legal Python and even idiomatic. Indeed just adding attribute is
how annotations work in the development version.
Alex
--
"I disapprove of what you say, but I will defend to the death your right to
say it." --Voltaire
"The people's good is the highest law."--Cicero
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django users" group.
To post to this group, send email to django-users@googlegroups.com
To unsubscribe from this group, send email to
django-users+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/django-users?hl=en
-~----------~----~----~----~------~----~------~--~---
