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

Exclude MSOffice mime types association if OnlyOffice app is installed #2301

Merged

Conversation

proxyconcept
Copy link
Contributor

Signed-off-by: Grégory Marigot gmarigot@teicee.com

Summary

The Nextcloud Office app configure the association with all possible office documents mime types.
But if we also have the OnlyOffice app, it's not possible to open any document with it...

Collabora and OnlyOffice are both great and it's a common case to want to install both: Collabora for OpenOffice documents and OnlyOffice for MSOffice documents.

This patch is a workaround specific for this case (and because it's not possible to wait for nextcloud/server#8105) : the Nextcloud Office app check if the OnlyOffice app is not installed before adding all the MSOffice mime types.

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Documentation (manuals or wiki) has been updated or is not required

@Raudius
Copy link
Contributor

Raudius commented Aug 9, 2022

If OnlyOffice is installed we should also merge MIMETYPES_MSOFFICE with MIMETYPES_OPTIONAL for the mimetypesNoDefaultOpen capability.

This will allow these mimetypes to be opened with collabora via the context menu.

lib/Capabilities.php Outdated Show resolved Hide resolved
@juliushaertl juliushaertl added enhancement New feature or request 2. developing Work in progress labels Oct 17, 2022
@juliushaertl
Copy link
Member

@proxyconcept Any news on addressing the review comments?

@juliushaertl
Copy link
Member

I took over and implemented the suggestion and rebased.

@juliushaertl juliushaertl added 4. to release Ready to release, can be waiting for the tests to finish and removed 2. developing Work in progress labels Sep 1, 2023
Signed-off-by: Grégory Marigot <gmarigot@teicee.com>
@juliushaertl juliushaertl merged commit 04b6e0a into nextcloud:main Sep 8, 2023
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to release, can be waiting for the tests to finish enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6.1.0 no longer plays nice with OnlyOffice
3 participants