"Fix for AttributeError: 'SessionMaker' object has no attribute '_model_changes' when using fixture to load fixtures" #89

Closed
wants to merge 1 commit into
from

Projects

None yet
@jpanganiban

Hi,

I encountered the same error as this (scenario is also stated in the link): http://flask.pocoo.org/mailinglist/archive/2011/3/6/flask-sqlalchemy-and-fixtures/.

Thanks

@techniq
techniq commented Aug 22, 2012

I was having this issue as well and the workaround in the Commit worked for me. Hopefully this can get merged into the next release. Thanks.

@JohnBrodie

Any chance of this getting merged upstream? It'd be great not to have to patch to get my tests to run.

@adurieux

Any chances this will be merged sometime ?

@mitsuhiko mitsuhiko closed this in #135 Jul 20, 2013
@ShaneHudson

I'm still having this from in version 1.0, any update on fixing it?

@mitsuhiko
Owner

I will push out a new release soon (tm). Can't promise today but definitely tomorrow.

@ShaneHudson

Ok brilliant, thanks :)

On Mon, Jul 29, 2013 at 12:37 PM, Armin Ronacher
notifications@github.comwrote:

I will push out a new release soon (tm). Can't promise today but
definitely tomorrow.


Reply to this email directly or view it on GitHubhttps://github.com/mitsuhiko/flask-sqlalchemy/pull/89#issuecomment-21714069
.


Shane Hudson (Website Developer -
www.ShaneHudson.nethttp://www.shanehudson.net/
)

07794746595

@ShaneHudson https://twitter.com/#!/ShaneHudson / +Shane
Hudsonhttps://plus.google.com/u/0/110111510059204475260

@glyphobet

Any progress on this? The fix in the diff does not appear to be present in 1.0.

@Archarachne

I got flask_sqlalchemy installed in Virtualenv and the files I've got are init.py && _compat.py. (installed with pip install Flask-SQLAlchemy)
The place that I could put the fix in is http://i.imgur.com/D7SeSYE.png, but not exacly as you have done it, as the code is different.
What should I do then?

Besides I wonder why that happens because I only get the error on the server that I upload my virtualenv app. Locally works ok.

@mitsuhiko
Owner

Any progress on this? The fix in the diff does not appear to be present in 1.0.

It's in the currently pending 2.0. Unfortunately there are some blockers right now that make me not release it. I'm going to re-investigate how to release something new over the weekend.

@Archarachne

This time I'll be the one to ask: is there any progress on this? :( I know that that some things are hard to fix, but this issue is stopping my masters degree paper/project from being finished.
Can you maybe provide us the date that this might be done?

@martijnvermaat
Contributor

You can already get it if you don't mind working with an unreleased version:

pip install -e git+https://github.com/mitsuhiko/flask-sqlalchemy#egg=Flask-SQLAlchemy
@Archarachne

It worked! OMG thank you very much for that. 👍

@pudo
pudo commented Dec 6, 2013

Hm, is there a new release of this coming up? This effectively prevents using flask-sqlalchemy and celery working together.

@john-osullivan

This same bug is keeping me from deploying on Heroku and finishing a project -- hope the solution gets added into the main code soon!

@jbmartin

Had the same problem. @martijnvermaat's suggestion worked for me.

@mvexel mvexel added a commit to mvexel/maproulette that referenced this pull request Feb 11, 2014
@mvexel mvexel have to get the current HEAD of the flask-sqlalchemy to work around a…
…n issue

preventing the loader script to work:
mitsuhiko/flask-sqlalchemy#89
5f07fd0
@mvexel mvexel added a commit to mvexel/maproulette that referenced this pull request Feb 11, 2014
@mvexel mvexel Merge branch 'master' of github.com:osmlab/maproulette
* 'master' of github.com:osmlab/maproulette:
  Adding apt-add-repository dependency to documentation
  stuff is fixed in the head of oauthlib so we can fall back to that
  More logging
  add challenge query fallback logic
  Adding a logging statement to challenge_tasks
  adding fork of oauthlib to requirements, fixes #80 together with previous commits
  have to get the current HEAD of the flask-sqlalchemy to work around an issue preventing the loader script to work: mitsuhiko/flask-sqlalchemy#89
  added simplejson to requirements (only required for loader script)
  oauth streamlining - making it work required patching oauthlib client.py
  restarting oauth fix effort
  completed change from challenge id to slug as foreign key. fixed tiny bug in challenge endpoint
  changed challenge getter from id to slug throughout, fixed arg parser in challenge endpoint
  fix in index template JS balance
fe1dbf4
@jontayesp

Would love to see this released soon as well.

@swaniris

I want to see this released too.

@jontayesp

@mitsuhiko status?

@kristenkam kristenkam pushed a commit to kristenkam/maproulette that referenced this pull request Apr 28, 2014
@mvexel mvexel have to get the current HEAD of the flask-sqlalchemy to work around a…
…n issue

preventing the loader script to work:
mitsuhiko/flask-sqlalchemy#89
a222c69
@kristenkam kristenkam pushed a commit to kristenkam/maproulette that referenced this pull request Apr 28, 2014
@mvexel mvexel Merge branch 'master' of github.com:osmlab/maproulette
* 'master' of github.com:osmlab/maproulette:
  Adding apt-add-repository dependency to documentation
  stuff is fixed in the head of oauthlib so we can fall back to that
  More logging
  add challenge query fallback logic
  Adding a logging statement to challenge_tasks
  adding fork of oauthlib to requirements, fixes #80 together with previous commits
  have to get the current HEAD of the flask-sqlalchemy to work around an issue preventing the loader script to work: mitsuhiko/flask-sqlalchemy#89
  added simplejson to requirements (only required for loader script)
  oauth streamlining - making it work required patching oauthlib client.py
  restarting oauth fix effort
  completed change from challenge id to slug as foreign key. fixed tiny bug in challenge endpoint
  changed challenge getter from id to slug throughout, fixed arg parser in challenge endpoint
  fix in index template JS balance
5de4506
@xcompass xcompass added a commit to ubc/compair that referenced this pull request May 24, 2014
@xcompass xcompass Update requirements.txt to use dev flask-sqlalchemy
The current stable version 1.0 has issue: AttributeError: 'SessionMaker'
object has no attribute '_model_changes'. See details here:
mitsuhiko/flask-sqlalchemy#89. The issue is
fixed in current master. It seems the author is not going to release any
new version soon. So using master instead of stable release for now.
e72b3fc
@harobed
harobed commented Jun 25, 2014

Up

@jquacinella

+1 for new release

@davidism davidism added this to the 2.1 milestone Jan 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment