[ckan-changes] [ckan/ckan] c97cd2: [#1420] Fix group_show by taking schema out of the...

Fri Aug 1 15:54:19 UTC 2014

  Branch: refs/heads/release-v2.2.1
  Home:   https://github.com/ckan/ckan
  Commit: c97cd2241e7846d0cb292fe906b31643465dd633
      https://github.com/ckan/ckan/commit/c97cd2241e7846d0cb292fe906b31643465dd633
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2014-07-29 (Tue, 29 Jul 2014)

  Changed paths:
    M ckan/controllers/group.py
    M ckan/lib/dictization/model_dictize.py

  Log Message:
  -----------
  [#1420] Fix group_show by taking schema out of the context when making sub-calls.

  Commit: 8de84e83057764a7f6045c044b286243558bddab
      https://github.com/ckan/ckan/commit/8de84e83057764a7f6045c044b286243558bddab
  Author: Mikko Koho <mikko.koho at csc.fi>
  Date:   2014-08-01 (Fri, 01 Aug 2014)

  Changed paths:
    M ckan/lib/search/query.py
    M ckan/logic/action/update.py

  Log Message:
  -----------
  Add quotes to package ID and name in Solr queries to prevent Solr errors with custom dataset identifiers.

  Commit: bfe55711a9ef067287f6f88d75d29db6bc3a777d
      https://github.com/ckan/ckan/commit/bfe55711a9ef067287f6f88d75d29db6bc3a777d
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2014-08-01 (Fri, 01 Aug 2014)

  Changed paths:
    M ckanext/datastore/db.py
    M ckanext/datastore/tests/test_upsert.py

  Log Message:
  -----------
  [#1776] Fix bug when upserting [] on JSON column in the datastore

We were testing that the new value wasn't `None` using `if value`. As `[]` is a
falsy value, we never went through that `if` and ended up trying to run an
invalid SQL query.

This commit fixes that by explictly testing `if value is not None`.

This was first discovered in
http://stackoverflow.com/questions/24207065/inserting-empty-arrays-in-json-type-fields-in-datastore

Compare: https://github.com/ckan/ckan/compare/ebd7a0554252...bfe55711a9ef