[Translate-cvs] SF.net SVN: translate:[15672] src/trunk/Pootle/local_apps/pootle_store/ views.py

[<al...@us...>]

Revision: 15672
          http://translate.svn.sourceforge.net/translate/?rev=15672&view=rev
Author:   alaaosh
Date:     2010-08-27 17:11:18 +0000 (Fri, 27 Aug 2010)

Log Message:
-----------
only count suggestion if one was created

Modified Paths:
--------------
    src/trunk/Pootle/local_apps/pootle_store/views.py

Modified: src/trunk/Pootle/local_apps/pootle_store/views.py
===================================================================
--- src/trunk/Pootle/local_apps/pootle_store/views.py	2010-08-27 17:10:16 UTC (rev 15671)
+++ src/trunk/Pootle/local_apps/pootle_store/views.py	2010-08-27 17:11:18 UTC (rev 15672)
@@ -353,13 +353,15 @@
                     sub.save()
 
             elif cansuggest and 'suggest' in request.POST:
-                #HACKISH: django 1.2 stupidly modifies instance on model form validation, reload unit from db
-                prev_unit = Unit.objects.get(id=prev_unit.id)
-                prev_unit.add_suggestion(form.cleaned_data['target_f'], get_profile(request.user))
-                SuggestionStat.objects.get_or_create(translation_project=translation_project,
-                                                           suggester=get_profile(request.user),
-                                                           state='pending',
-                                                           unit=prev_unit.id)
+                if form.instance._target_updated:
+                    #HACKISH: django 1.2 stupidly modifies instance on
+                    # model form validation, reload unit from db
+                    prev_unit = Unit.objects.get(id=prev_unit.id)
+                    sugg = prev_unit.add_suggestion(form.cleaned_data['target_f'], get_profile(request.user))
+                    if sugg:
+                        SuggestionStat.objects.get_or_create(translation_project=translation_project,
+                                                             suggester=get_profile(request.user),
+                                                             state='pending', unit=prev_unit.id)
         else:
             # form failed, don't skip to next unit
             edit_unit = prev_unit


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.