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

The QDbBackedSessionHandler SessionOpen method is not static #88

Closed
olegabr opened this issue Jan 23, 2013 · 5 comments
Closed

The QDbBackedSessionHandler SessionOpen method is not static #88

olegabr opened this issue Jan 23, 2013 · 5 comments
Milestone

Comments

@olegabr
Copy link
Member

olegabr commented Jan 23, 2013

It leads to an error:
'Non-static method QDbBackedSessionHandler::SessionOpen() should not be called statically'

olegabr added a commit to olegabr/qcubed that referenced this issue Jan 23, 2013
@vaibhav-kaushal
Copy link
Member

Yeah, I too saw that now. I am using the handler in production. I guess I
forgot to update it here.

On Wed, Jan 23, 2013 at 9:53 PM, Oleg Abrosimov notifications@github.comwrote:

It leads to an error:
'Non-static method QDbBackedSessionHandler::SessionOpen() should not be
called statically'


Reply to this email directly or view it on GitHubhttps://github.com//issues/88.

Regards
Vaibhav Kaushal http://about.me/vaibhav.kaushal

@vaibhav-kaushal
Copy link
Member

Going to edit the file online directly because it is a small change. Please do not merge the pull request.

@vaibhav-kaushal
Copy link
Member

Did it (master branch). Should be fixed now.

@olegabr
Copy link
Member Author

olegabr commented Jan 23, 2013

@vaibhav-kaushal, I'm not sure that it is conformant with the Workflow proposed: Workflow

@vaibhav-kaushal
Copy link
Member

Oops! OKay, this was the last time.

On Wed, Jan 23, 2013 at 10:07 PM, Oleg Abrosimov
notifications@github.comwrote:

@vaibhav-kaushal https://github.com/vaibhav-kaushal, I'm not shure that
it is conformant with the Workflow proposed: Workflowhttps://github.com/qcubed/framework/wiki/Workflow


Reply to this email directly or view it on GitHubhttps://github.com//issues/88#issuecomment-12604354.

Regards
Vaibhav Kaushal http://about.me/vaibhav.kaushal

scottux added a commit that referenced this issue Jan 24, 2013
Fixes the issue #88. 'static' keyword.
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