-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Data API" button disappears on resource page after empty update #3012
Comments
@NicolaiMogensen this is indeed a bug. The "Data API" button relies on the PR incoming |
As datastore_active is stored as an extra, if it's not provided in the incoming data_dict on resource_update it will get dropped. This happens most noticeably when udpating via the frontend, causing the green Data API button to disappear.
Fix in #3020 |
@amercader That was fast! Thanks for correcting this, as it broke the buttons on all resources when doing |
…lost [#3012] Persist datastore_active on resource updates
As datastore_active is stored as an extra, if it's not provided in the incoming data_dict on resource_update it will get dropped. This happens most noticeably when udpating via the frontend, causing the green Data API button to disappear.
As datastore_active is stored as an extra, if it's not provided in the incoming data_dict on resource_update it will get dropped. This happens most noticeably when udpating via the frontend, causing the green Data API button to disappear.
ckan_version: 2.5.2
Please describe the expected behaviour
When updating a resource without changing any fields, url or description, the green "Data API" button should still be there, because the data is still in the datastore (And API calls of this type confirms that it still is). Not sure if it is expected of the datapusher to trigger its update or not.
Please describe the actual behaviour
The green "Data API" button disappears, as if data is now removed from the datastore for the particular resource. It would seem like CKAN thinks this triggers the datapusher hook, even though it does not (logs confirm it nothing happens).
What steps can be taken to reproduce the issue?
Push .CSV file to the datastore using the web interface, then update the resource using either the
resource_update
API call or the web interface, without changing anything.I'm curious to see if i'm the only one with this issue, i can reproduce over several CKAN installations running 2.5.2. If i need to post this in the datapusher github issue tracker instead, close this and i will post it there instead.
The text was updated successfully, but these errors were encountered: