search

buxeasywork / django-rosetta

Automatically exported from code.google.com/p/django-rosetta
MIT License
0 stars 0 forks source link

Fix for broken "Suggest" feature (also adds "Suggest All") #88

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
When you, for example, suggest this ---

    Listen to %(chant)s football chant

and "suggest" gives you this --

    H..ren Sie %(Gesang)s Fu..all Gesang        (ignore the ... missing chars)

It's just broken your code right there, and will more than likely give you 
server 500 errors!!!

The patch attached fixes the problem, keeping your variables sacred.

It also adds "Suggest All" feature, and works around a minor issue with recent 
Django trunk.

Original issue reported on code.google.com by litchfie...@gmail.com on 26 Oct 2010 at 8:49

Attachments:

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago

Original comment by litchfie...@gmail.com on 27 Oct 2010 at 12:00

Attachments:

GoogleCodeExporter commented 9 years ago 
I've done a similar suggest all patch myself. There is an issue though with 
google returning with an error message about API abuse is you translate too 
many too quickly.

Your patch for keeping the variables appears to work sometimes and sometimes it 
doesn't even on the same text. It certainly takes out most of the effort though.

I've attached your patch modified to apply against django-rosetta-0.5.6.

Original comment by Barry.C....@gmail.com on 2 Mar 2011 at 11:55

Attachments: