[ckan-changes] [ckan/ckan] 3282b1: Extend IDomainObjectModification

GitHub noreply at github.com
Tue Sep 16 12:10:49 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 3282b12e3f6b694693799e832fe8d4f731fa5b57
      https://github.com/ckan/ckan/commit/3282b12e3f6b694693799e832fe8d4f731fa5b57
  Author: Tom Mortimer-Jones <tom at morty.co.uk>
  Date:   2014-07-21 (Mon, 21 Jul 2014)

  Changed paths:
    M ckan/model/modification.py
    M ckan/plugins/interfaces.py

  Log Message:
  -----------
  Extend IDomainObjectModification

Currently the notify signal is sent before the database commit has occurred. It
is not possible to listen for modification events and know that the changes
have made it to the database. This can lead to a race condition if you try to
get a record from the database.

For an example see:

https://github.com/ckan/ckanext-archiver/issues/6

This extension of the interface would allow notification after the database
commit has occurred.


  Commit: f847c329a0b288cbef6995c848bf2ee7ccdcf281
      https://github.com/ckan/ckan/commit/f847c329a0b288cbef6995c848bf2ee7ccdcf281
  Author: joetsoi <joe.yeung.tsoi at gmail.com>
  Date:   2014-09-16 (Tue, 16 Sep 2014)

  Changed paths:
    M ckan/model/modification.py
    M ckan/plugins/interfaces.py

  Log Message:
  -----------
  Merge branch 'extend-idomainobjectmodification-interface'


Compare: https://github.com/ckan/ckan/compare/431636c1a116...f847c329a0b2


More information about the ckan-changes mailing list