| Index: cms/utils.py |
| =================================================================== |
| --- a/cms/utils.py |
| +++ b/cms/utils.py |
| @@ -54,25 +54,31 @@ def get_page_params(source, locale, page |
| # 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
Wladimir Palant
2015/05/06 17:21:59
Please see http://codereview.adblockplus.org/52425
|
| + 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" |
| )() |