[ckan-changes] [okfn/ckan] 5fe687: [#445] Only lookup strings in term translation tab...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/okfn/ckan
  Commit: 5fe687381cc5fcf9cc340e98b921d7af38215a35
      https://github.com/okfn/ckan/commit/5fe687381cc5fcf9cc340e98b921d7af38215a35
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2013-02-22 (Fri, 22 Feb 2013)

  Changed paths:
    M ckanext/multilingual/plugin.py

  Log Message:
  -----------
  [#445] Only lookup strings in term translation table

This fixes the multilingual tests that are currently failing on master.

It seems that when the multilingual extension was written, package dicts
never contained ints as values. Now they do. But postgres cannot compare
an int to a string so it crashes. This commit changes the multilingual
plugin so that it only tries to lookup strings in the term translation
table, other types are just skipped.

Fixes #445.


  Commit: 1e8a1a2b7f7e4af85a85736f7d28030441df6694
      https://github.com/okfn/ckan/commit/1e8a1a2b7f7e4af85a85736f7d28030441df6694
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2013-02-25 (Mon, 25 Feb 2013)

  Changed paths:
    M ckanext/multilingual/plugin.py

  Log Message:
  -----------
  [#445] Refactor a loop in multilingual plugin

This just makes a small loop in ckanext/multilingual clearer, the logic
should be the same.


  Commit: e1b5fc95df3a6aa7ff1f4157762e1e81d5d7e88a
      https://github.com/okfn/ckan/commit/e1b5fc95df3a6aa7ff1f4157762e1e81d5d7e88a
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-25 (Mon, 25 Feb 2013)

  Changed paths:
    M ckanext/multilingual/plugin.py

  Log Message:
  -----------
  Merge pull request #457 from okfn/445-bug-multilingual-tests-failing-2

[#445] Only lookup strings in term translation table


Compare: https://github.com/okfn/ckan/compare/ec768320d042...e1b5fc95df3a