[ckan-changes] [ckan/ckan] 7096d6: [#1572] Improvements

Fri Sep 11 16:29:29 UTC 2015

  Branch: refs/heads/1572-dataset-purge
  Home:   https://github.com/ckan/ckan
  Commit: 7096d6e979acb20b0a0ec025c6d8c7ac5f329246
      https://github.com/ckan/ckan/commit/7096d6e979acb20b0a0ec025c6d8c7ac5f329246
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2015-09-11 (Fri, 11 Sep 2015)

  Changed paths:
    M ckan/logic/action/delete.py
    A ckan/migration/versions/080_continuity_id_indexes.py
    M ckan/model/domain_object.py
    M ckan/tests/logic/action/test_delete.py
    M test-core.ini

  Log Message:
  -----------
  [#1572] Improvements
* no revision is needed when purging the package. The only thing being
  created is a Activity, which is not revisioned.
* no need to explicitly delete an object's revision objects. Despite the
  very old comment, it is clear that this does happen naturally through
  cascades. This is verified in the test and by reading the SQL
  produced.
* Added a resource to the test for fullness.
* Added helpful comments to the test-core.ini to show people how to use
  it to see generated SQL commands.