Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: cms/sources.py

Issue 29317015: Issue 2625 - [cms] Crowdin synchronisation script (Closed)
Patch Set: Created June 15, 2015, 2:12 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cms/sources.py
diff --git a/cms/sources.py b/cms/sources.py
index 51770027059033470ef3652bb0756ee44370f217..9abcbc90598b0150a5cfe3b20952a1b115ff3082 100644
--- a/cms/sources.py
+++ b/cms/sources.py
@@ -16,6 +16,7 @@
# along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
import codecs
+from collections import OrderedDict
import ConfigParser
import json
import os
@@ -169,9 +170,9 @@ class Source:
def read_locale(self, locale, page):
default_locale = self.read_config().get("general", "defaultlocale")
if locale == default_locale:
- result = {}
+ result = OrderedDict()
kzar 2015/06/15 14:24:28 This is required to keep the correct order of page
Wladimir Palant 2015/06/29 19:05:38 Crowdin isn't very good at keeping that order but
kzar 2015/07/02 12:33:13 I found that it does help in practice for our webs
else:
- result = dict(self.read_locale(default_locale, page))
+ result = OrderedDict(self.read_locale(default_locale, page))
if self.has_locale(locale, page):
filedata = self.read_file(self.locale_filename(locale, page))

Powered by Google App Engine
This is Rietveld