Issue 2340 - Don`t generate pages if less than 30% have been translated

cms/utils.py

 # coding: utf-8
 
 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2015 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
@@ skipping 41 matching lines @@
     raise Exception("Page %s uses unknown format %s" % (page, format))
 
   # Note: The converter might change some parameters so we can only read in
   # template data here.
   params["templatedata"] = source.read_template(params["template"])
 
   defaultlocale = params["config"].get("general", "defaultlocale")
   params["defaultlocale"] = defaultlocale
 
   locales = [
-    locale
-    for locale in source.list_locales()
+    l

[Sebastian Noack, 2015/05/06 16:15:30]

Why didn't you keep the more descriptive variable name, as you wrap the lines
anyway? Not that this change seems to be related anyway.

[Wladimir Palant, 2015/05/06 17:21:59]

Please see
http://codereview.adblockplus.org/5242593268989952/diff/5629499534213120/cms/...

+    for l in source.list_locales()
     if source.has_locale(locale, localefile)
   ]
   if defaultlocale not in locales:
     locales.append(defaultlocale)
   locales.sort()
   params["available_locales"] = locales
 
   params["head"], params["body"] = converter()
+  if converter.total_translations > 0:
+    params["translation_ratio"] = (1 -
+        float(converter.missing_translations) / converter.total_translations)
+  else:
+    params["translation_ratio"] = 1
+
   return params
 
-def process_page(source, locale, page, format, site_url_override=None):
+def process_page(source, locale, page, format=None, site_url_override=None):
   return TemplateConverter(
     get_page_params(source, locale, page, format, site_url_override),
     key="templatedata"
   )()