Issue 2625 - [cms] Crowdin synchronisation script

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,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
 #
 # You should have received a copy of the GNU General Public License
 # along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
 
 import codecs
+import collections
 import ConfigParser
 import json
 import os
 from StringIO import StringIO
 import subprocess
 import urlparse
 import zipfile
 import logging
 
 class Source:
@@ skipping 133 matching lines @@
     config = self.read_config()
     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 = {}
+      result = collections.OrderedDict()

[Sebastian Noack, 2015/07/08 13:03:20]

Nit: This could be simplified a little:

result = collections.OrderedDict()
if locale != default_locale:
  result.update(self.read_locale(default_locale, page))

[kzar, 2015/07/11 19:21:18]

I think we should leave this unrelated change out of this patchset.

[Sebastian Noack, 2015/07/14 11:31:04]

It's not unrelated if you change the code anyway. And in particular your change
is introducing the duplication here.

[kzar, 2015/07/14 12:54:27]

Done.

     else:
-      result = dict(self.read_locale(default_locale, page))
+      result = collections.OrderedDict(self.read_locale(default_locale, page))
 
     if self.has_locale(locale, page):
       filedata = self.read_file(self.locale_filename(locale, page))
       localedata = json.loads(filedata)
       for key, value in localedata.iteritems():
         result[key] = value["message"]
 
     return result
 
   #
@@ skipping 102 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")