[ckan-changes] [okfn/ckan] 16924d: [#1178] Remove backward migrations, as we don't su...

[GitHub]

  Branch: refs/heads/1178-users-invitations
  Home:   https://github.com/okfn/ckan
  Commit: 16924d937af4207a585fd34eaf2be083bcc2d6d2
      https://github.com/okfn/ckan/commit/16924d937af4207a585fd34eaf2be083bcc2d6d2
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-10-11 (Fri, 11 Oct 2013)

  Changed paths:
    M ckan/migration/versions/071_add_state_column_to_user_table.py

  Log Message:
  -----------
  [#1178] Remove backward migrations, as we don't support it


  Commit: 0bd163e75eacc65fee0546fa356f195f1e109713
      https://github.com/okfn/ckan/commit/0bd163e75eacc65fee0546fa356f195f1e109713
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-10-11 (Fri, 11 Oct 2013)

  Changed paths:
    M ckan/tests/lib/test_authenticator.py

  Log Message:
  -----------
  [#1178] Fix repeated test name


  Commit: d073ce7f6b56361cf8c5452c2f969e4d45356437
      https://github.com/okfn/ckan/commit/d073ce7f6b56361cf8c5452c2f969e4d45356437
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-10-11 (Fri, 11 Oct 2013)

  Changed paths:
    M ckan/logic/action/create.py

  Log Message:
  -----------
  [#1178] Use the data returned by validate, instead of the one passed as param


  Commit: 1da817c40b59d7dfa866abd9e68ed844442f34d4
      https://github.com/okfn/ckan/commit/1da817c40b59d7dfa866abd9e68ed844442f34d4
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-10-11 (Fri, 11 Oct 2013)

  Changed paths:
    M ckan/logic/action/create.py
    M ckan/logic/schema.py

  Log Message:
  -----------
  [#1178] Move user_invite_schema to ckan.logic.schema


  Commit: c2e4f767eed6485383a7ef2efd40a8e990d0a623
      https://github.com/okfn/ckan/commit/c2e4f767eed6485383a7ef2efd40a8e990d0a623
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-10-11 (Fri, 11 Oct 2013)

  Changed paths:
    M ckan/logic/action/create.py
    M ckan/tests/logic/test_action.py

  Log Message:
  -----------
  [#1178] Refactor user name creation

I've isolated the loop where it needs to be. This makes the code clearer, but
makes it susceptible to errors. For example, after the random name was
generated but before the user is created, another user with the same name might
be created, breaking our code.

This should be rare enough to bother us.


Compare: https://github.com/okfn/ckan/compare/c33916283669...c2e4f767eed6