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

Rename file logging from "ownCloud" to "file" #490

Closed
MorrisJobke opened this issue Jul 21, 2016 · 0 comments
Closed

Rename file logging from "ownCloud" to "file" #490

MorrisJobke opened this issue Jul 21, 2016 · 0 comments
Labels
good first issue Small tasks with clear documentation about how and in which place you need to fix things in.

Comments

@MorrisJobke
Copy link
Member

Config option log_type in config.php allows to set owncloud as type, which is actually logging to a file. This should be renamed. Beside that to allow backwards compatibility if the option is set to owncloud the new file logging should be used.

@MorrisJobke MorrisJobke added the good first issue Small tasks with clear documentation about how and in which place you need to fix things in. label Jul 21, 2016
Faldon pushed a commit to Faldon/server that referenced this issue Jul 22, 2016
- Updated the config sample to point to log_type='file'
- Renamed the Class for logfile logging to File in namespace 'OC\Log\'.
  Changed the occurrences of 'OC\Log\Owncloud' to 'OC\Log\File'.
- Renamed the Class for log:file command to File in namespace 'OC\Core\Command\Log\File'.
  Changed registration of the command to use 'OC\Core\Command\Log\File'.
- Changed default Syslog tag to Nextcloud
- Retained backwards compatibility for configs with 'logtype' => 'owncloud'

- Adjusted tests for the new file log.

Closes nextcloud#490.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Small tasks with clear documentation about how and in which place you need to fix things in.
Projects
None yet
Development

No branches or pull requests

1 participant