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

Fix ExtDeprecationWarning #3889

Merged
merged 1 commit into from
Oct 30, 2017
Merged

Fix ExtDeprecationWarning #3889

merged 1 commit into from
Oct 30, 2017

Conversation

davidread
Copy link
Contributor

This warning is seen when running paster commands when you've installed dev-requirements.txt

(default)vagrant@vagrant-ubuntu-trusty-64:/vagrant/src/ckan$ paster --plugin=ckan jobs -c $CKAN_INI worker
/usr/lib/ckan/default/local/lib/python2.7/site-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.sqlalchemy is deprecated, use flask_sqlalchemy instead.
  .format(x=modname), ExtDeprecationWarning

It's raised at this moment:

  File "/vagrant/src/ckan/ckan/lib/cli.py", line 2647, in command
    self._load_config()
  File "/vagrant/src/ckan/ckan/lib/cli.py", line 321, in _load_config
    self.site_user = load_config(self.options.config, load_site_user)
  File "/vagrant/src/ckan/ckan/lib/cli.py", line 235, in load_config
    flask_app = _get_test_app().flask_app
  File "/vagrant/src/ckan/ckan/tests/helpers.py", line 171, in _get_test_app
    app = ckan.config.middleware.make_app(config['global_conf'], **config)
  File "/vagrant/src/ckan/ckan/config/middleware/__init__.py", line 59, in make_app
    flask_app = make_flask_stack(conf, **app_conf)
  File "/vagrant/src/ckan/ckan/config/middleware/flask_app.py", line 76, in make_flask_stack
    from flask_debugtoolbar import DebugToolbarExtension

Upgrading the version of the Flask debug toolbar fixes it. This is a simple point release, and is the latest currently on pypi - see changelog: https://pypi.python.org/pypi/Flask-DebugToolbar

@amercader amercader merged commit c5a1be8 into master Oct 30, 2017
@amercader
Copy link
Member

Ace, thanks @davidread

@amercader amercader deleted the fix-ExtDeprecationWarning branch October 30, 2017 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants