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

Only use the user session if ownCloud is already installed #23488

Merged
merged 1 commit into from
Mar 22, 2016

Conversation

LukasReschke
Copy link
Member

When installing ownCloud with autotest and MySQL some log entries may be created which will invoke the logging class. IUserSession has a dependency on the database which will make the installation fail => 💣

cc @DeepDiver1975 @MorrisJobke

When installing ownCloud with autotest and MySQL some log entries may be created which will invoke the logging class. IUserSession has a dependency on the database which will make the installation fail => 💣
@LukasReschke LukasReschke added this to the 9.1-current milestone Mar 22, 2016
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @bartv2, @MorrisJobke and @MTGap to be potential reviewers

@LukasReschke
Copy link
Member Author

Yay. MySQL is happy 🚀

@rullzer
Copy link
Contributor

rullzer commented Mar 22, 2016

👍

DeepDiver1975 added a commit that referenced this pull request Mar 22, 2016
…lled

Only use the user session if ownCloud is already installed
@DeepDiver1975 DeepDiver1975 merged commit b1e5adf into master Mar 22, 2016
@DeepDiver1975 DeepDiver1975 deleted the only-use-usersession-if-installed branch March 22, 2016 20:25
@lock
Copy link

lock bot commented Aug 6, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants