[ckan-changes] [ckan/ckan] 9adf61: Allow specifying user when creating a group using ...

GitHub noreply at github.com
Sat Feb 15 00:13:13 UTC 2014


  Branch: refs/heads/1505-invite-to-organization-causes-error
  Home:   https://github.com/ckan/ckan
  Commit: 9adf61603237191feafd58ffd991a639307d7871
      https://github.com/ckan/ckan/commit/9adf61603237191feafd58ffd991a639307d7871
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2014-02-14 (Fri, 14 Feb 2014)

  Changed paths:
    M ckan/new_tests/factories.py

  Log Message:
  -----------
  Allow specifying user when creating a group using the factory

With this, we're able to create a group owned by a specific user.


  Commit: 21f4873f318b0f727c94c0d6a833073d33563c29
      https://github.com/ckan/ckan/commit/21f4873f318b0f727c94c0d6a833073d33563c29
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2014-02-14 (Fri, 14 Feb 2014)

  Changed paths:
    M ckan/new_tests/helpers.py

  Log Message:
  -----------
  new_tests.helpers.call_auth now work for any auth type (update, create, ...)


  Commit: 6eac66ad7c0d2760e7be1cb1d984f1ce0cffd260
      https://github.com/ckan/ckan/commit/6eac66ad7c0d2760e7be1cb1d984f1ce0cffd260
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2014-02-14 (Fri, 14 Feb 2014)

  Changed paths:
    M ckan/logic/auth/create.py
    A ckan/new_tests/logic/action/test_create.py
    A ckan/new_tests/logic/auth/test_create.py
    M ckan/tests/logic/test_action.py
    M ckan/tests/logic/test_auth.py

  Log Message:
  -----------
  [#1505] Fix bug when non-sysadmins invited users to organizations

The problem was with the .user_invite() auth function. Instead of adding 'id'
to the data_dict, I was adding it to the context. Fixed now.

I've also changed it to require a data_dict, and a data_dict['group_id'] (as
they're required anyway).

While I'm at it, I rewrote the tests into the new testing style.


Compare: https://github.com/ckan/ckan/compare/9adf61603237^...6eac66ad7c0d


More information about the ckan-changes mailing list