[ckan-changes] [ckan/ckan] 64f632: [#1852] Fix resource view auhtorization functions

GitHub noreply at github.com
Wed Nov 12 13:16:10 UTC 2014


  Branch: refs/heads/1852-default-resource-views
  Home:   https://github.com/ckan/ckan
  Commit: 64f632f53069f92115a217228244b89b3c9a551d
      https://github.com/ckan/ckan/commit/64f632f53069f92115a217228244b89b3c9a551d
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-12 (Wed, 12 Nov 2014)

  Changed paths:
    M ckan/logic/auth/create.py
    M ckan/logic/auth/delete.py
    M ckan/logic/auth/update.py
    M ckan/new_tests/logic/auth/test_create.py
    M ckan/new_tests/logic/auth/test_delete.py
    M ckan/new_tests/logic/auth/test_update.py

  Log Message:
  -----------
  [#1852] Fix resource view auhtorization functions

The auth functions correctly forward to the relevant `resource_*` ones,
but they were passing the data_dict as is, so `id` referred to the
resource view, not the resource.

Added new auth tests. There is a failing one that depends on #2037


  Commit: 41ad38d7c88f476b256688a306c8723380ff7971
      https://github.com/ckan/ckan/commit/41ad38d7c88f476b256688a306c8723380ff7971
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-12 (Wed, 12 Nov 2014)

  Changed paths:
    M ckan/new_tests/logic/action/test_delete.py
    M ckan/new_tests/logic/action/test_get.py
    M ckan/new_tests/logic/action/test_update.py
    M ckan/tests/logic/test_action.py

  Log Message:
  -----------
  [#1852] Move rest of resource views tests from old to new ones


Compare: https://github.com/ckan/ckan/compare/3f2fe927764a...41ad38d7c88f


More information about the ckan-changes mailing list