Skip to content
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

Incorrect 'download resource' hyperlink when a resource is unable to upload to datastore #2873

Closed
Mbrownshoes opened this issue Feb 10, 2016 · 6 comments

Comments

@Mbrownshoes
Copy link

@Mbrownshoes Mbrownshoes commented Feb 10, 2016

When a dataset cannot be previewed in the Data Explorer - e.g. when the dataset it too large, or has not been loaded to the DataStore (e.g. http://demo.ckan.org/dataset/bad-hyperlink-example/resource/36a112c0-7a4e-40da-aa0d-8e0c67e64957) the Data Explorer is providing an incorrectly addressed "Download resource" hyperlinked button
image

The hyperlink URL http://demo.ckan.org/dataset/bad-hyperlink-example/resource/36a112c0-7a4e-40da-aa0d-8e0c67e64957 is missing the terminating text /download/

@amercader
Copy link
Member

@amercader amercader commented Mar 16, 2016

@smotornyuk @Engerrs can either of you have a look at this one? It should be easy to track where the URL is coming from. The template used is this one:

https://github.com/ckan/ckan/blob/master/ckan/templates/package/snippets/resource_view.html

Thanks!

@smotornyuk
Copy link
Member

@smotornyuk smotornyuk commented Mar 17, 2016

I will take that one

@amercader amercader assigned smotornyuk and unassigned amercader Mar 17, 2016
@amercader
Copy link
Member

@amercader amercader commented Mar 17, 2016

Thanks @smotornyuk!

@wardi
Copy link
Contributor

@wardi wardi commented Mar 17, 2016

Was this not fixed by #2599

smotornyuk added a commit to smotornyuk/ckan that referenced this issue Mar 24, 2016
`resource_view` template uses raw_resouce_url in order to create
download link if resource can't be displayed, but this variable is not
passed into snippet. To make that simpler, use resource.url property
for download button.
@smotornyuk
Copy link
Member

@smotornyuk smotornyuk commented Mar 24, 2016

No, problem here in variable that not passed into template

@amercader
Copy link
Member

@amercader amercader commented Mar 29, 2016

Fixed by #2930

@amercader amercader closed this Mar 29, 2016
amercader added a commit that referenced this issue Mar 29, 2016
`resource_view` template uses raw_resouce_url in order to create
download link if resource can't be displayed, but this variable is not
passed into snippet. To make that simpler, use resource.url property
for download button.

Conflicts:
	ckan/templates/package/snippets/resource_view.html
amercader added a commit that referenced this issue Mar 29, 2016
`resource_view` template uses raw_resouce_url in order to create
download link if resource can't be displayed, but this variable is not
passed into snippet. To make that simpler, use resource.url property
for download button.
amercader added a commit that referenced this issue Mar 29, 2016
`resource_view` template uses raw_resouce_url in order to create
download link if resource can't be displayed, but this variable is not
passed into snippet. To make that simpler, use resource.url property
for download button.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants