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

Check_ckan_version() throwing error due to missing argument #410

Open
aleeexgreeen opened this issue Apr 25, 2024 · 2 comments
Open

Check_ckan_version() throwing error due to missing argument #410

aleeexgreeen opened this issue Apr 25, 2024 · 2 comments

Comments

@aleeexgreeen
Copy link

ckan version 2.9.7

I am getting this error when navigating to a scheming page after installing CKAN + ckanext-scheming on my machine for the first time

ckanext-scheming/ckanext/scheming/templates/scheming/form_snippets/slug.html", line 3, in <module>
    {% set read_endpoint = '.read' if h.check_ckan_version(min_version="2.9") else '_read' %}
TypeError: check_ckan_version() missing 1 required positional argument: 'max_version'

Going into /form_snippets/slug.html and adding a max_version fixes the issue, but it is a temporary solution

@wardi
Copy link
Contributor

wardi commented Apr 25, 2024

h.check_ckan_version in 2.9 doesn't require its max_version argument. https://github.com/ckan/ckan/blob/f2d47c8f87bc5f224ebff93b23d0f6fda8fb81b4/ckan/plugins/toolkit.py#L470

Do you have any other plugins installed that might have overridden this helper?

@aleeexgreeen
Copy link
Author

These were the other plugins at the time of install fluent stats text_view image_view recline_view datastore xloader.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants