Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix for SqlBypass session store #2040

Merged
merged 1 commit into from Jul 12, 2011

Conversation

Projects
None yet
2 participants
Contributor

jhtwong commented Jul 12, 2011

Fix for SqlBypass session store - split off from #2016

Two issues fixed:

  1. connection_pool is not defined - needed by SessionStore#drop_table!
    and create_table! since c94651f

  2. initialization of connection to the default of AR::Base.connection
    only occurred at the singleton level - the instance level method defined
    by cattr_accessor did not have this logic

@jhtwong jhtwong Fix for SqlBypass session store
Two issues fixed:
1) connection_pool is not defined - needed by SessionStore#drop_table!
and create_table! since c94651f

2) initialization of connection to the default of AR::Base.connection
only occurred at the singleton level - the instance level method defined
by cattr_accessor did not have this logic
b87f8d8

@spastorino spastorino added a commit that referenced this pull request Jul 12, 2011

@spastorino spastorino Merge pull request #2040 from SAP-Oxygen/3-1-stable-sqlbypass-patch
Fix for SqlBypass session store
a91afeb

@spastorino spastorino merged commit a91afeb into rails:3-1-stable Jul 12, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment