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

[GitHub]

  Branch: refs/heads/master
  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.


  Commit: ed5fe6e86e520800edcf84d37e5f0c1d8a6ef485
      https://github.com/ckan/ckan/commit/ed5fe6e86e520800edcf84d37e5f0c1d8a6ef485
  Author: Ross Jones <ross at mailbolt.com>
  Date:   2015-09-14 (Mon, 14 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:
  -----------
  Merge pull request #2637 from ckan/1572-dataset-purge

dataset_purge improvements


Compare: https://github.com/ckan/ckan/compare/4ca2e6158c40...ed5fe6e86e52