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

cms/sources.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 19 matching lines @@
     parsed = urlparse.urlparse(url)
     page = parsed.path
     if parsed.scheme != "" or page.startswith("/") or page.startswith("."):
       # Not a page link
       return None, None
 
     if page == "" and url != "":
       # Page-relative link
       return None, None
 
-    def has_locale(locale, page):
-      try:
-        page = config.get("locale_overrides", page)

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

So locale overrides are now only repsected when generating the pages but no
longer when running the development server? :(

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

No, this logic is actually built into the default has_locale function. The block
here was unnecessarily duplicating it - I removed it to make sure it doesn't
falsify testing.

-      except ConfigParser.Error:
-        pass
-      return self.has_locale(locale, page)
-
     config = self.read_config()
     default_locale = config.get("general", "defaultlocale")
     default_page = config.get("general", "defaultpage")
     alternative_page = "/".join([page, default_page]).lstrip("/")
 
     if self.has_localizable_file(default_locale, page):
       if not self.has_localizable_file(locale, page):
         locale = default_locale
     elif self.has_page(page):
-      if not has_locale(locale, page):
+      if not self.has_locale(locale, page):
         locale = default_locale
     elif self.has_page(alternative_page):
-      if not has_locale(locale, alternative_page):
+      if not self.has_locale(locale, alternative_page):
         locale = default_locale
     else:
       logging.warning("Link to %s cannot be resolved", page)
 
     parts = page.split("/")
     if parts[-1] == default_page:
       page = "/".join(parts[:-1])
 
     path = "/%s/%s" % (locale, page)
     return locale, urlparse.urlunparse(parsed[0:2] + (path,) + parsed[3:])
@@ skipping 80 matching lines @@
   def list_locales(self):
     result = set()
     for filename in self.list_files("locales"):
       if "/" in filename:
         locale, path = filename.split("/", 1)
         result.add(locale)
     return result
 
   def has_locale(self, locale, page):
     config = self.read_config()
-    if config.has_option("locale_overrides", page):
+    try:
       page = config.get("locale_overrides", page)
+    except ConfigParser.Error:
+      pass
     return self.has_file(self.locale_filename(locale, page))
 
   def read_locale(self, locale, page):
     default_locale = self.read_config().get("general", "defaultlocale")
     if locale == default_locale:
       result = {}
     else:
       result = dict(self.read_locale(default_locale, page))
 
     if self.has_locale(locale, page):
@@ skipping 110 matching lines @@
         path = os.path.join(dir, filename)
         if os.path.isfile(path):
           result.append(relpath + filename)
         elif os.path.isdir(path):
           do_list(path, relpath + filename + "/")
     do_list(self.get_path(subdir), "")
     return result
 
   def get_cache_dir(self):
     return os.path.join(self._dir, "cache")