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

Clarify comment

Simplify file comparison

Include auth type in http log

Log the final http request

Merge pull request #2778 from nextcloud/windows_cfapi_backend_for_vfs

Windows cfapi backend for vfs

Ensure we properly cancel hydration on server errors

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

Don't look at virtual files data for the mimetype matches

Otherwise this would attempt to download the file everytime we check the
direct editor for a given file which wouldn't be adequate. Would also
lead to a deadlock in our case since that would happen in the main
thread and implicit hydration is driven from there as well.

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

Implement callback between wrapper and vfs object for hydration requests

This comes with a test simulating an open request coming from another
process (although in our case it's really just a thread). The actual
hydration works as expected by cfapi, handling of encrypted files is for
now missing.

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

Improve test readability with a bunch of macros

Since we often checked the same file states to see if they were virtual,
non virtual or gone, let's factor this out. This way intent in tests is
clearer.

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

Port the tests to the CfApiWrapper

This reduces code duplication and opportunities of mistakes between the
test and the implementation.

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

Split the CfAPI lower level code in a wrapper

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

Add initial version of the CfApi backend

For now this implements only the logic necessary to get a test suite
equivalent to the TestSyncVirtualFiles one to pass. It doesn't (yet)
honor request to fetch files from the system.

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

One byte placeholders assumption only holds for the suffix backend

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

Don't consider CfApi backend as experimental

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2748 from nextcloud/rakekniven-patch-1

Removed translation for "etag" and changed spelling

Changed spelling of "etag".

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

Removed translation for "etag" and changed spelling

Shouldn't be translated.

Just checked https://en.wikipedia.org/wiki/HTTP_ETag and all available localized pages.

Signed-off-by: rakekniven <mark.ziegler@rakekniven.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

Merge pull request #2759 from ivaradi/ubuntu-hirsute

Add support for Hirsute

Add support for Hirsute

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

[tx-robot] updated from transifex

Merge pull request #2739 from hrittich/master

Fixed the autostart for AppImages. See #2504.

Fixed the autostart for AppImages. See #2504.

Signed-off-by: Hannah Rittich <hrittich@users.noreply.github.com>

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

Changed triple dot to ellipsis

Changed triple dot to ellipsis

Reported at Transifex.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2726 from nextcloud/settings-ui

UI improvements: Settings dialog

Update the settings dialog window title.

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

General settings: move buttons to the bottom left of their group boxes.

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

[tx-robot] updated from transifex

Merge pull request #2725 from nextcloud/default-mono-icons

Make monochrome icons the default on Linux and Mac OS.

Make monochrome icons the default on Linux and Mac OS.

Unless it is branded with other icons.

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

[tx-robot] updated from transifex

Merge pull request #2700 from nextcloud/major_engine_update

Major engine update

Adjust the appimage script to the buildsystem changes

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

Workaround bug in older GCC used for the AppImage toolchain

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

Get the excluded files test to pass again on Windows

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

Please the clang-tidy overlord

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

Don't warn if everything is fine

Remove now unneeded cmake test and define

Replace custome date parser QDateTime::fromString(value, Qt::RFC2822Date)

Fix unit tests by setting Date header

Use time the request was send,..

not when it was processed by the client, to determine the quality of the connection.

Speedup test build by compile the fake server just once

Owncloud - virtual files smaller <1KB - problems with syncing

The issue was caused by gziped responses not providing a
content lenght header.

Fixes: #8248

Pause sync when remove all dialog is displayed

Fixes: #8263

Use enum instead of int

Fix test on systems where C:\ is not writeable

Don't recurse the whole tree looking for a git dir, use PROJECT_SOURCE_DIR

Fix usePlaceholders migration

Remove junk files

Fix vfs experimental text

Fix folder permission check on NTFS

Fixes: #8187

Fix condition

Append .exe to crashreporter path so it is better located

Ensure the callback is triggered

Don`t block main thread when displaying all files removed dialog

Fixes: #8170

Don't use exec() on dialogs

Update libcrashreporter-qt

Draw active progressbar

Pass a widget to the progressbar in the folder delegate to draw the correct active state

Remove the plugin loader mechanism for VFS backends

We will have all the code in public anyway so it can just be compiled
in. Thus no need to go through the plugin loading dance. Replaced the
loading with factory functions. Kept mostly the same structure
otherwise.

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

Allow to control availability of folders in the settings dialog

It felt odd to be able to control the encryption status in the settings
dialog but not the availability. Also availability was controllable on
the root, so let's make it widely available.

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

Resurrect the display of subfolders for VFS sync folders

This got removed from the settings since in that case selective sync
isn't supported. Unfortunately that's also necessary to display them to
allow encrypting folders via the context menu.

So we display the subfolders again but in the case of VFS we disable the
ability to (un)check them. They just have an icon, a name and a context
menu.

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

Update the FolderStatusModel when a folder encryption ends

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

Set the account state after E2EE is setup in the GUI

Otherwise we would not display the E2EE message if the account was
connected before the creation of the settings dialog. This was likely
caused by the delayed creation of the settings dialog.

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

Remove empty action in the settings toolbar

This was causing an unwanted line in the middle of the bar.

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

Fix typo and style

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

Get rid of now unused GetFolderEncryptStatusJob

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

Remove the encryption flag check from encrypted propagation code

If we use those encrypted propagation code paths, we already know from
the discovery phase (and thus the journal db) that the folders are
encrypted so no need to check again.

This will remove another expensive round trip with the server.

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

Remove encrypt status job fetching from ClientSideEncryption

This is now unused.

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

No need to fetch encryption data separately before discovery

This step isn't necessary anymore, no one looks at ClientSideEncryption
for that information anyway.

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

Remove unused methods on ClientSideEncryption

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

Stop using e2e()->isFolderEncrypted() in the folder wizard

We used the same approach than for the FolderStatusModel by getting the
is-encrypted property straight from the LSCOL job.

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

FolderStatusModel now uses the LSCOL job result for encryption status

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

FolderStatusModel LSCOL job now gets the is-encrypted property

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

Stop using e2e()->isFolderEncrypted() in the jobs

Thanks to the new discovery algorithm, we got all the freshest E2EE
information straight from the database so reuse it instead of going
through an in memory copy.

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

Set the encrypted flag in the database when EncryptFolderJob succeeds

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

Change EncryptFolderJob path convention

It had a different path convention than all the other jobs, most likely
for legacy reasons because of the tight coupling it had to the settings
dialog.

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

Have the folder token inside the EncryptFolderJob

No need to look for a token on the outside we can just work properly by
keeping all the state encapsulated in the job.

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

Don't use getFileRecordByE2eMangledName in jobs anymore

Thanks to the new discovery algorithm we got both mangled and original
file names in the item so no need to go through the database for
nothing.

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

Repair E2EE deletions

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

Make sure the encrypted flag gets all the way to the journal db

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