[ckan-changes] [okfn/ckan] f78101: [#1705] Add optional vocab argument to Tag.get() a...

GitHub noreply at github.com
Wed Feb 1 11:32:20 UTC 2012


  Branch: refs/heads/feature-1698-tag-taxonomies
  Home:   https://github.com/okfn/ckan
  Commit: f7810165b73adb3db3ed60d771bc5b4c37d77751
      https://github.com/okfn/ckan/commit/f7810165b73adb3db3ed60d771bc5b4c37d77751
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2012-01-31 (Tue, 31 Jan 2012)

  Changed paths:
    M ckan/model/tag.py
  M ckan/model/vocabulary.py

  Log Message:
  -----------
  [#1705] Add optional vocab argument to Tag.get() and Tag.by_name()

With the default value for the new vocab argument (None) Tag.get() and
Tag.by_name() will return only free tags (tags that don't belong to any
vocabulary). If a vocab argument is given they'll only return tags from
that vocab.

All the tests still pass, but new tests will need to be added for this.


  Commit: fa89fe606ad38f995e367d18b43b6e4282c43a08
      https://github.com/okfn/ckan/commit/fa89fe606ad38f995e367d18b43b6e4282c43a08
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2012-02-01 (Wed, 01 Feb 2012)

  Changed paths:
    M ckan/model/tag.py

  Log Message:
  -----------
  [#1705] Add vocab_id_or_name=None args to some model/tag.py methods.

Change the behaviour of the methods so that they should return only free
tags (tags without a vocabulary) when called with no vocab_id_or_name
argument. With a vocab_id_or_name they return only tags belonging to
that vocab.

The existing tests still work but new tests for the new argument need to
be added.

Also added docstrings to these methods because it wasn't too obvious
what they did.


  Commit: bdb2cc19d40add8bb7c496fef535dabd0b76d0cb
      https://github.com/okfn/ckan/commit/bdb2cc19d40add8bb7c496fef535dabd0b76d0cb
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2012-02-01 (Wed, 01 Feb 2012)

  Changed paths:
    M ckan/logic/converters.py
  M ckan/logic/validators.py
  A ckan/tests/logic/test_converters.py
  A ckan/tests/logic/test_validators.py

  Log Message:
  -----------
  Merge branch 'feature-1698-tag-taxonomies' of github.com:okfn/ckan into feature-1698-tag-taxonomies


Compare: https://github.com/okfn/ckan/compare/a411b14...bdb2cc1


More information about the ckan-changes mailing list