[#2788] Use return_id_only instead of return_package_dict

Suggested by @tobes, as it is more backwards compatible

ckan/logic/action/create.py

@@ -98,9 +98,8 @@ def package_create(context, data_dict):
         which groups exist call ``group_list()``
     :type groups: list of dictionaries
 
-    :returns: the newly created dataset (if 'return_package_dict' is True in the
-              context, which is the default. Otherwise returns just the dataset
-              id)
+    :returns: the newly created dataset (unless 'return_id_only' is set to True
+              in the context, in which case just the dataset id will be returned)
     :rtype: dictionary
 
     '''
@@ -165,10 +164,10 @@ def package_create(context, data_dict):
     context["id"] = pkg.id
     log.debug('Created object %s' % str(pkg.name))
 
-    return_package_dict = context.get('return_package_dict',True)
+    return_id_only = context.get('return_id_only', False)
 
-    output = _get_action('package_show')(context, {'id':context['id']}) \
-            if return_package_dict else context['id']
+    output = context['id'] if return_id_only \
+            else _get_action('package_show')(context, {'id':context['id']})
 
     return output
 

ckan/logic/action/update.py

@@ -276,10 +276,10 @@ def package_update(context, data_dict):
 
     log.debug('Updated object %s' % str(pkg.name))
 
-    return_package_dict = context.get('return_package_dict',True)
+    return_id_only = context.get('return_id_only', False)
 
-    output = _get_action('package_show')(context, {'id': data_dict['id']}) \
-            if return_package_dict else data_dict['id']
+    output = data_dict['id'] if return_id_only \
+            else _get_action('package_show')(context, {'id': data_dict['id']})
 
     return output