[ckan-changes] [ckan/ckan] 4486d7: Deal with a bad "sort" param better - dont log as ...

Fri Feb 19 12:50:53 UTC 2016

  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 4486d7649a76059e9ac3d79703b40faf58e238e7
      https://github.com/ckan/ckan/commit/4486d7649a76059e9ac3d79703b40faf58e238e7
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2016-02-11 (Thu, 11 Feb 2016)

  Changed paths:
    M ckan/controllers/package.py
    M ckan/lib/search/query.py
    M ckan/templates/snippets/search_form.html
    M ckan/tests/controllers/test_package.py

  Log Message:
  -----------
  Deal with a bad "sort" param better - dont log as error and disturb admins.

  Commit: 3d3ccfb565fa0402a17af02b920f14da2bd5a2e7
      https://github.com/ckan/ckan/commit/3d3ccfb565fa0402a17af02b920f14da2bd5a2e7
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2016-02-11 (Thu, 11 Feb 2016)

  Changed paths:
    M bin/travis-run-tests
    M ckan/lib/search/query.py

  Log Message:
  -----------
  Logging to help debug travis

  Commit: e7676c113f6ff1c25482059a8c9a251cdc79b22e
      https://github.com/ckan/ckan/commit/e7676c113f6ff1c25482059a8c9a251cdc79b22e
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2016-02-11 (Thu, 11 Feb 2016)

  Changed paths:
    M bin/travis-run-tests
    M ckan/lib/search/query.py

  Log Message:
  -----------
  Alternative SOLR error message seen on Travis. Both are solr-jetty 1.4.1, so not sure why we get different ones.

  Commit: 42637b0cbcb0f9668126852c987aa02430736a3d
      https://github.com/ckan/ckan/commit/42637b0cbcb0f9668126852c987aa02430736a3d
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2016-02-11 (Thu, 11 Feb 2016)

  Changed paths:
    M ckan/lib/search/query.py

  Log Message:
  -----------
  Fix syntax

  Commit: b39e0949f072f3f59226a2bd8266d1d6b6c078a0
      https://github.com/ckan/ckan/commit/b39e0949f072f3f59226a2bd8266d1d6b6c078a0
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2016-02-11 (Thu, 11 Feb 2016)

  Changed paths:
    M bin/travis-run-tests
    M ckan/lib/search/query.py

  Log Message:
  -----------
  Logging to help debug travis

  Commit: 77127bcd3d5462263f9b6f0e611cfdea5c8c09d8
      https://github.com/ckan/ckan/commit/77127bcd3d5462263f9b6f0e611cfdea5c8c09d8
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2016-02-12 (Fri, 12 Feb 2016)

  Changed paths:
    M bin/travis-run-tests
    M ckan/lib/search/query.py

  Log Message:
  -----------
  Deal with SOLR error coming back as HTML on travis.

  Commit: 99a1c2f6a5c953f969c4d9e97399a875857d9e20
      https://github.com/ckan/ckan/commit/99a1c2f6a5c953f969c4d9e97399a875857d9e20
  Author: Ross Jones <ross at servercode.co.uk>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M ckan/lib/search/query.py
    M ckan/tests/controllers/test_package.py

  Log Message:
  -----------
  Add yet another potential error message.

Because we support so many 'versions' of Solr, we're encountering
different error messages as the only way of determining what went wrong.
In some cases the response is HTML :(

To fix this we explicitly check in the test if it succeeded (we get a
200 when we get an error response) and inform the dev where to go to add
their newly discovered error message.

  Commit: 592b65f2538ae88fbc3b0c3155cf9fcd77a0458c
      https://github.com/ckan/ckan/commit/592b65f2538ae88fbc3b0c3155cf9fcd77a0458c
  Author: Ross Jones <ross at mailbolt.com>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M bin/travis-run-tests
    M ckan/controllers/package.py
    M ckan/lib/search/query.py
    M ckan/templates/snippets/search_form.html
    M ckan/tests/controllers/test_package.py

  Log Message:
  -----------
  Merge pull request #2875 from ckan/solr-errors

Deal with a bad "sort" param better

Compare: https://github.com/ckan/ckan/compare/70790836026c...592b65f2538a