Merge pull request #2868 from nextcloud/vfs_cfapi_dont_mistake_unpinned_directory_for_files_to_dehydrate

Don't treat new folders in unpinned folders as files to dehydrate

Don't treat new folders in unpinned folders as files to dehydrate

This would only happen if the parent of the newly created folder would
be explicitly set to online only, hence why it went under the radar
previously.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #2733 from nextcloud/Valdnet-patch-3

Move space in "owncloudadvancedsetuppage.cpp"

Move space in "owncloudadvancedsetuppage.cpp"

Merge pull request #2840 from nextcloud/avoid_keeping_account_alive_from_subobject

Avoid keeping Account alive via a shared ptr in ClientSideEncryption

Remove unused method

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Avoid keeping Account alive via a shared ptr in ClientSideEncryption

This account object was really only used during the initialization phase
or for forgetting the sensitive data. So let's receive it as parameter
there and pass it on from job to job as needed.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #2865 from nextcloud/vfs_cfapi_dont_get_stuck

VFS CfAPI implementation shouldn't get stuck

Don't crash when the same request is filled twice

The OS might request the same file again if we take too much time to
fulfill a request. So in case it's queueing the same one again instead
of bailing out just fail the second one and let the first one finish
properly.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Be a bit more resilient with errors

In case we'd been to slow to fullfill or we're still processing a
cancelled request better not just crash. We still log the issue though.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

[tx-robot] updated from transifex

Merge pull request #2734 from nextcloud/Valdnet-patch-4

Move space in "folderwizard.cpp"

Move space in "folderwizard.cpp"

Merge pull request #2735 from nextcloud/Valdnet-patch-5

Change the quotation marks

Change the quotation marks

Merge pull request #2814 from nextcloud/files_push_notifications

Add push notifications for file changes

Add push notifications for file changes

Resolves #2802

Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>

Merge pull request #2864 from nextcloud/fixes_for_wizard_advanced_setup

Fixes for wizard advanced setup

Ensure the right radio is selected when selective sync is picked

This got broken previously when trying to reconcile during a
cherry-pick. This also repairs the case when the theme tells us the
default is to not select anything (not that we're using this right now
but in case we'd like to brand this).

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

It's not really recommended anymore at least not on Windows

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #2853 from nextcloud/e2ee-fix-rename-root-folder-issue

E2ee fix rename root folder issue & Fix upload stuck issue due to incorrect Lock/Unlock sequence

E2EE Fix incorrect name of a nested encrypted item in the Settings dialog when the root folder is non-encrypted and it is renamed.

Signed-off-by: allexzander <blackslayer4@gmail.com>

Fix folder upload issue due to wrong Lock/Unlock order

Signed-off-by: allexzander <blackslayer4@gmail.com>

Fix E2EE root folder rename issue.

Signed-off-by: allexzander <blackslayer4@gmail.com>

Merge pull request #2859 from nicolasfella/qt5153

Fix build against Qt 5.15.3

Fix build against Qt 5.15.3

Use QLatin1Char to prevent build error due to stricter QT_NO_CAST_FROM_ASCII

Signed-off-by: Nicolas Fella <nicolas.fella@gmx.de>

Merge pull request #2860 from nextcloud/rakekniven-patch-2

Removed blank before colon

Removed blank before colon

Signed-off-by: rakekniven <mark.ziegler@rakekniven.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2845 from nextcloud/default_to_colored_systray_on_linux

Default to colored systray on linux

Fix the branding detection for monochrome icons on Mac

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Revert "Make monochrome icons the default on Linux and Mac OS."

This reverts commit 016410dbaad9cfa85209d6424dc2df9362a80a17.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

[tx-robot] updated from transifex

Merge pull request #2844 from nextcloud/handle_svg_branded_clients

Handle SVG branded clients

Use APPLICATION_ICON_SET for themed icons

We now favor APPLICATION_ICON_SET to isBranded() regarding the decision
to use PNG or SVG for icons.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Use the SVG wizard logo when APPLICATION_ICON_SET says so

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Add the APPLICATION_ICON_SET option

Also mention the svg possibly used in the wizard

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Reference the wizard_logo.svg file from the resources

Somehow this was missing

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2827 from nextcloud/apply_version_suffix_to_msi_filename

Integrate the version suffix to the MSI filename

Integrate the version suffix to the MSI filename

This should be enough to reinstate the "-daily" marker in the filename
for daily builds.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #2826 from nextcloud/reference_coding_style_wiki_page

Reference coding style wiki page from CONTRIBIUTING.md

Merge branch 'master' into reference_coding_style_wiki_page

Reference coding style wiki page from CONTRIBIUTING.md

Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>

Merge pull request #2824 from nextcloud/e2ee-can-not-move-folder

Fix e2ee folder move issue

Fix e2ee folder move issue

Signed-off-by: allexzander <blackslayer4@gmail.com>

Merge pull request #2793 from nextcloud/xattr_backend_for_vfs

XAttr backend for VFS

Add the XAttr backend prototype with its test suite

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Don't detect file changes during hydration for virtual files

Indeed, that file size will almost always change between the 1 byte
placeholder and the hydrated file. Only when using the CfAPI on Windows
this won't be the case since because it will expose the original size
even for placeholders.

Also worth noting: the suffix backend didn't hit that case since the
filename changes (with suffix for placeholders, without for hydrated
files).

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Provide the path to the current folder to statTypeVirtualFile on unix

Otherwise backends can't get to the actual file which will be needed for
the XAttr backend.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Add the plumbing for a new extended attributes backend

Ideally this will end up being the backend we use for both Linux and
macOS but that will require work with desktop environments on the Linux
side and to reverse engineering at least on xattr value on macOS.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #2821 from nextcloud/e2ee-fix-parallel-uploads-issue

E2EE fix upload parallelism issue.

E2EE fix upload parallelism issue.

Signed-off-by: allexzander <blackslayer4@gmail.com>

[tx-robot] updated from transifex

Merge pull request #2823 from nextcloud/repair_chunked_uploads

Repair chunked uploads

Repair chunked uploads

There's been a confusion between the chunk number and the chunk
offset leading to corruptions... Let's pass the proper offset to
the UploadDevice again.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

[tx-robot] updated from transifex

Merge pull request #2815 from nextcloud/update-docs

Update Github issue template and documentation with the new 3.1 option to "Create Debug Archive".

Update documentation with the new 3.1 option to "Create Debug Archive".

Signed-off-by: Camila <hello@camila.codes>

Fix documentation path in instruction of doc/README.

Signed-off-by: Camila <hello@camila.codes>

Update Github issue template with the new 3.1 option to "Create Debug Archive".

Also:
- cleaned up some of the text: nobody read that and it was mostly
from a time when this team was only one person.
- remove references to e2ee known issues. Not relevant anymore.

Signed-off-by: Camila <hello@camila.codes>

[tx-robot] updated from transifex

Merge pull request #2799 from nextcloud/e2ee-root-folder-remove-issue

E2ee root folder remove issue

Implement nested items removal when removing root encrypted folder.

Signed-off-by: allexzander <blackslayer4@gmail.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2801 from nextcloud/fix_cfapi_win32_build

Fix CfAPI Win32 build

Second attempt at fixing CfAPI wrapper build in Win32 mode

MSVC having so useless error messages it didn't quite point to the root
cause of the issue... it turns out that through the maze of macros
defined in the windows API, there's one which impacted the function
pointer definition of CfCloseHandle which would then not convert to
FileHandle::Deleter as expected. So I end up wrapping it in a lambda to
help... luckily this kind of lambdas decay into a simple function
pointer so there's likely no overhead it's just to coerce the compiler
into doing the right thing.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Revert "Fix CfAPI wrapper build in Win32 mode"

This reverts commit 3b3864296a2bc09540989241325e938ed5a3275e.

[tx-robot] updated from transifex

Merge pull request #2800 from nextcloud/fix_cfapi_win32_build

Fix CfAPI wrapper build in Win32 mode

Fix CfAPI wrapper build in Win32 mode

For some reason MSVC manages to deduce the right constructor in Win64
mode but not in Win32 mode. So let's be more explicit about what we
return.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

[tx-robot] updated from transifex

Merge pull request #2794 from FlexW/master

Remove NO_SHIBBOLETH flag and dead code

Remove NO_SHIBBOLETH flag and dead code

Signed-off-by: Felix Weilbach <felix.weilbach@t-online.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2781 from nextcloud/cherry_pick_2020_w53

Cherry pick 2020 w53

Fix a possible crash with the remove all files dialog

Use Q_REQUIRED_RESULT directly like in other places

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Mark vfs functions as OC_REQUIRED_RESULT

Handle errors in convertToPlaceholder

Update windows launch on start binary location

Fixes: #7672

Fix testVersionOfInstalledBinary for brandings

Use owncloudcmd in testVersionOfInstalledBinary

... as it works without X in CI.

Fix style

Add todo for Qt 5.15

Cleanup pathtoUNC and its test