[ckan-changes] [ckan/ckan] 1a3906: [#3004] Upgrade Babel requirement, fix issues with...

[GitHub]

  Branch: refs/heads/3004-upgrade-babel
  Home:   https://github.com/ckan/ckan
  Commit: 1a39062c3465b2529a44875e19786083e7fdc72e
      https://github.com/ckan/ckan/commit/1a39062c3465b2529a44875e19786083e7fdc72e
  Author: amercader <amercadero at gmail.com>
  Date:   2016-05-09 (Mon, 09 May 2016)

  Changed paths:
    M ckan/lib/helpers.py
    M ckan/lib/i18n.py
    M ckan/templates/snippets/language_selector.html
    M ckan/tests/controllers/test_home.py
    M requirements.in
    M requirements.txt

  Log Message:
  -----------
  [#3004] Upgrade Babel requirement, fix issues with language selector

The language selector did not work with Babel 2.3.4 because the URLs on
the option tags didn't contain the language code. This was caused
because we are passing the Babel `Locale` classes to the `url_for`
helper, which compares them against a list of string codes (eg `ca`).
On older Babel versions comparing eg `Locale('ca') == 'ca'` used to
work.

This includes the necessary changes both on the selector snippet and the
helper functions so we are comparing apples to apples.


  Commit: 3adb55195c85df56aa365392fe61e3bc64c46ef2
      https://github.com/ckan/ckan/commit/3adb55195c85df56aa365392fe61e3bc64c46ef2
  Author: amercader <amercadero at gmail.com>
  Date:   2016-05-09 (Mon, 09 May 2016)

  Changed paths:
    M ckan/lib/i18n.py
    M ckan/tests/legacy/test_coding_standards.py

  Log Message:
  -----------
  [#3004] PEP8 lib/i18n.py


Compare: https://github.com/ckan/ckan/compare/1a39062c3465^...3adb55195c85