On 3/25/06, coulix <[EMAIL PROTECTED]> wrote: > > i got it working : > > def _module_get_top_users(limit): > cursor = db.cursor() > cursor.execute(""" > SELECT auth_users.username, COUNT(*) AS count > FROM auth_users, recettes_recipes > WHERE auth_users.id=recettes_recipes.owner_id > GROUP BY username > ORDER BY count DESC > LIMIT %s > """% limit) > res = cursor.fetchall() > return res
Actually, you should let .execute() do parameter substitution: def _module_get_top_users(limit): cursor = db.cursor() cursor.execute(""" SELECT auth_users.username, COUNT(*) AS count FROM auth_users, recettes_recipes WHERE auth_users.id=recettes_recipes.owner_id GROUP BY username ORDER BY count DESC LIMIT %s """, (limit,)) res = cursor.fetchall() return res Although in this situation, doing your own parameter substitution (using %) is probably not harmful. -- The Pythonic Principle: Python works the way it does because if it didn't, it wouldn't be Python. --~--~---------~--~----~------------~-------~--~----~ 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 [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/django-users -~----------~----~----~----~------~----~------~--~---