Adrià Mercader

Results 292 comments of Adrià Mercader
trafficstars

This is desired behaviour but we should mention the bulk_ functions in package_update and package_delete auth functions so users are aware of them and can override them too if necessary.

@EricSoroos You need this to use a request context in your test: ```diff diff --git a/ckan/tests/lib/test_helpers.py b/ckan/tests/lib/test_helpers.py index 43f247bb5..f45d0eaf3 100644 --- a/ckan/tests/lib/test_helpers.py +++ b/ckan/tests/lib/test_helpers.py @@ -887,6 +887,7 @@ def test_decode_view_request_filters(test_request_context):...

@EricSoroos If you don't have a local source install I'd try this: ``` # Clone this docker repo git clone https://github.com/okfn/docker-ckan # Replace the `build:` section in `docker-compose.yml` with `image:...

Summary from our discussion in the dev meeting: * If not allowed and the field value is always overridden the API call should fail with an error as suggested *...

Fixed in dda6bb8f4, this will be in 2.9.6

Let's search usage of CKAN_CURRENT_URL on popular extensions and mention this on the release notes. And patch 2.9 and master as well

@EricSoroos do you want to have a look at this?

@wardi yes, that's https://github.com/ckan/ckan/pull/6926 , right?

@EricSoroos This needs to be fixed in master as well: * https://github.com/ckan/ckan/blob/830a65d3623b1be8c26f96637d03f1878923a510/ckan/views/__init__.py#L194 * https://github.com/ckan/ckan/blob/830a65d3623b1be8c26f96637d03f1878923a510/ckan/lib/helpers.py#L658 Can you send a PR against master with these fixes? Thanks