[or-cvs] [bridgedb/master 2/3] Moved indexed variables to a list as Nickm suggested

nickm at torproject.org nickm at torproject.org
Wed Sep 29 18:04:43 UTC 2010


Author: Christian Fromme <kaner at strace.org>
Date: Mon, 27 Sep 2010 21:47:24 +0200
Subject: Moved indexed variables to a list as Nickm suggested
Commit: d006b87ee23a27f3f03491ec85c4c518c18cf6d3

---
 lib/bridgedb/I18n.py   |   60 ++++++++++++++++++++---------------------------
 lib/bridgedb/Server.py |   33 +++++++++++---------------
 2 files changed, 40 insertions(+), 53 deletions(-)

diff --git a/lib/bridgedb/I18n.py b/lib/bridgedb/I18n.py
index 4999b5b..928f4cc 100644
--- a/lib/bridgedb/I18n.py
+++ b/lib/bridgedb/I18n.py
@@ -17,67 +17,59 @@ def _(text):
     """
     return text
 
-HTML_1 = _("""
+# All text that needs translation goes here
+BRIDGEDB_TEXT = [
+        _("""
     Here are your bridge relays:
-    """)
-
-HTML_2 = _("""
+    """),
+        _("""
     Bridge relays (or "bridges" for short) are Tor relays that aren't listed
     in the main directory. Since there is no complete public list of them,
     even if your ISP is filtering connections to all the known Tor relays,
     they probably won't be able to block all the bridges.
-    """)
-
-HTML_3 = _("""
+    """),
+        _("""
     To use the above lines, go to Vidalia's Network settings page, and click
     "My ISP blocks connections to the Tor network". Then add each bridge
     address one at a time.
-    """)
-
-HTML_4 = _("""
+    """),
+        _("""
     Configuring more than one bridge address will make your Tor connection
     more stable, in case some of the bridges become unreachable.
-    """)
-
-HTML_5 = _("""
+    """),
+        _("""
     Another way to find public bridge addresses is to send mail to
     bridges at torproject.org with the line "get bridges" by itself in the body
     of the mail. However, so we can make it harder for an attacker to learn
     lots of bridge addresses, you must send this request from a gmail or
     yahoo account.
-    """)
-
-EMAIL_1 = _("""
+    """),
+        _("""
 [This is an automated message; please do not reply.]
-""")
-
-EMAIL_2 = _("""
+    """),
+        _("""
 Here are your bridge relays:
-""")
-
-EMAIL_3 = _("""
+    """),
+        _("""
 Bridge relays (or "bridges" for short) are Tor relays that aren't listed
 in the main directory. Since there is no complete public list of them,
 even if your ISP is filtering connections to all the known Tor relays,
 they probably won't be able to block all the bridges.
-""")
-
-EMAIL_4 = _("""
+    """),
+        _("""
 To use the above lines, go to Vidalia's Network settings page, and click
 "My ISP blocks connections to the Tor network". Then add each bridge
 address one at a time.
-""")
-
-EMAIL_5 = _("""
+    """),
+        _("""
 Configuring more than one bridge address will make your Tor connection
 more stable, in case some of the bridges become unreachable.
-""")
-
-EMAIL_6 = _("""
+    """),
+        _("""
 Another way to find public bridge addresses is to visit
 https://bridges.torproject.org/. The answers you get from that page
 will change every few days, so check back periodically if you need more
 bridge addresses.
-""")
-
-NO_BRIDGES = _("(no bridges currently available)")
+    """),
+        _("(no bridges currently available)")
+]
diff --git a/lib/bridgedb/Server.py b/lib/bridgedb/Server.py
index dd5066d..cd1d760 100644
--- a/lib/bridgedb/Server.py
+++ b/lib/bridgedb/Server.py
@@ -75,7 +75,7 @@ class WebResource(twisted.web.resource.Resource):
             answer = "".join("%s\n" % b.getConfigLine(self.includeFingerprints)
                              for b in bridges)
         else:
-            answer = _(I18n.NO_BRIDGES)
+            answer = _(I18n.BRIDGEDB_TEXT[-1])
 
         logging.info("Replying to web request from %s.  Parameters were %r", ip,
                      request.args)
@@ -88,18 +88,17 @@ class WebResource(twisted.web.resource.Resource):
 
     def buildHTMLMessageTemplate(self):
         """DOCDOC"""
-        template_string = "<html><body>" \
-                             + "<p>" + _(I18n.HTML_1) \
-                             + "<pre id=\"bridges\">" \
-                             + "%s" \
-                             + "</pre></p>" \
-                             + "<p>" + _(I18n.HTML_2) + "</p>" \
-                             + "<p>" + _(I18n.HTML_3) + "</p>" \
-                             + "<p>" + _(I18n.HTML_4) + "</p>" \
-                             + "<p>" + _(I18n.HTML_5) + "</p>" \
-                             + "</body></html>"
+        html_msg = "<html><body>" \
+                   + "<p>" + _(I18n.BRIDGEDB_TEXT[1]) \
+                   + "<pre id=\"bridges\">" \
+                   + "%s" \
+                   + "</pre></p>" \
+                   + "<p>" + _(I18n.BRIDGEDB_TEXT[2]) + "</p>" \
+                   + "<p>" + _(I18n.BRIDGEDB_TEXT[3]) + "</p>" \
+                   + "<p>" + _(I18n.BRIDGEDB_TEXT[4]) + "</p>" \
+                   + "</body></html>"
 
-        return template_string
+        return html_msg
 
 
 def addWebServer(cfg, dist, sched):
@@ -241,14 +240,10 @@ def getMailResponse(lines, ctx):
     else:
         answer = "(no bridges currently available)"
 
-    EMAIL_MESSAGE_TEMPLATE =   _(EMAIL_1) \
-                             + _(EMAIL_2) \
+    EMAIL_MESSAGE_TEMPLATE = "".join("%s\n" % _(I18n.BRIDGEDB_TEXT[5:2])) \
                              + "%s" \
-                             + _(EMAIL_3) \
-                             + _(EMAIL_4) \
-                             + _(EMAIL_5) \
-                             + _(EMAIL_6)
-    body.write(_(EMAIL_MESSAGE_TEMPLATE) % answer)
+                             + "".join("%s\n" % _(I18n.BRIDGEDB_TEXT[7:4]))
+    body.write(EMAIL_MESSAGE_TEMPLATE % answer)
 
     f.seek(0)
     logging.info("Email looks good; we should send an answer.")
-- 
1.7.1




More information about the tor-commits mailing list