[ckan-changes] commit/ckan: John Glover: [logic] Bug fix: task_status_show should take entity_id, not entity_type

[Bitbucket]

1 new commit in ckan:


https://bitbucket.org/okfn/ckan/changeset/dff6bc3fd527/
changeset:   dff6bc3fd527
branch:      feature-1371-task-status-logic-layer
user:        John Glover
date:        2011-11-03 18:23:03
summary:     [logic] Bug fix: task_status_show should take entity_id, not entity_type
affected #:  2 files

diff -r 8dbd4a0adf67298316587b9e200c46440a7ef283 -r dff6bc3fd527c51c7496099bf0f719f9efbfc838 ckan/logic/action/get.py
--- a/ckan/logic/action/get.py
+++ b/ckan/logic/action/get.py
@@ -797,7 +797,7 @@
     else:
         query = model.Session.query(model.TaskStatus)\
             .filter(and_(
-                model.TaskStatus.entity_type == data_dict['entity_type'],
+                model.TaskStatus.entity_id == data_dict['entity_id'],
                 model.TaskStatus.task_type == data_dict['task_type'],
                 model.TaskStatus.key == data_dict['key']
             ))


diff -r 8dbd4a0adf67298316587b9e200c46440a7ef283 -r dff6bc3fd527c51c7496099bf0f719f9efbfc838 ckan/tests/functional/api/test_action.py
--- a/ckan/tests/functional/api/test_action.py
+++ b/ckan/tests/functional/api/test_action.py
@@ -681,7 +681,7 @@
 
         # make sure show works when giving a (entity_id, task_type, key) tuple
         postparams = '%s=1' % json.dumps({
-            'entity_type': task_status['entity_type'],
+            'entity_id': task_status['entity_id'],
             'task_type': task_status['task_type'],
             'key': task_status['key']
         })

Repository URL: https://bitbucket.org/okfn/ckan/

--

This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.