docs: Update or correct default enabledPreviewProviders lists #41294
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
See this for the current default list:
https://github.com/owncloud/core/blob/v10.15.0/lib/private/PreviewManager.php#L266-L281
and the relevant comments / examples have been ordered the same way.
OC\Preview\PDF
was dropped from the config.sample.php because it is not in the default enabled list.Question:
What about these in https://doc.owncloud.com/server/10.14/admin_manual/configuration/files/previews_configuration.html#preview-format-requirements not mentioned anywhere in the updated files:
Are the docs even correct. few lines in the code we can see this:
https://github.com/owncloud/core/blob/v10.15.0/lib/private/PreviewManager.php#L334-L337
Related Issue
N/A
Motivation and Context
Better documentation
How Has This Been Tested?
No testing, only documentation changes
Screenshots (if appropriate):
N/A
Types of changes
Not sure what to tick here
Checklist:
Not sure what to tick here