Adrià Mercader
Adrià Mercader
Following on #7160
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