Switch to for range loop

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

Let Theme tell us if we're branded or not

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

Add OpacityMask to fix mouse hover on round corner.

- Fix for #2173.
- Removed the workaround for the left round corner which was also
fixed with the OpacityMask.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Let context menu open wizard if there are no accounts

The context menu offers to open the main dialog and the settings
even if no accounts are configured.  In this case, the main dialog
is useless and the settings are probably confusing.  Hence, this
commit replaces these actions in the context menu by an action
to open the wizard (which also opens on left click, so this is
the most natural thing to do).

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

[tx-robot] updated from transifex

Merge pull request #2224 from nextcloud/qt-5.12.9-update

Update CI for Qt 5.12.9

Update CI for Qt 5.12.9

Also remove the jq package installation from the AppImage build script because
the package is included in the new Docker image (see nextcloud/docker-ci#231).

Signed-off-by: Michael Schuster <michael@schuster.ms>

[tx-robot] updated from transifex

Merge pull request #2209 from nextcloud/fix-pause-sync-states

Fix pause sync states

sigh, too

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Get a const auto for the FolderMap to iterate over

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Replaced Qt foreach by C++11 ranged for loop

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Properly set syncPause state on Systray creation when account init is done

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Fixed copy/paste bug, resulting in a wrong signal ::pauseSync used instead of ::resumeSync

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Fix Windows installer graphics files for 2.7

For some reason the exported bitmaps in #2211 were not properly saved in the right
format for display by the NSIS installer system.

Signed-off-by: Michael Schuster <michael@schuster.ms>

[tx-robot] updated from transifex

Drone Upload: Fix if condition for master, in addition to #2213

Don't try to comment on master (old condition was from a previous version of the script).

Signed-off-by: Michael Schuster <michael@schuster.ms>

Merge pull request #2213 from nextcloud/enh/ci-upload

Drone: Upload AppImage to GitHub using nextcloud-desktop-bot

Drone: Upload AppImage to GitHub using nextcloud-desktop-bot

Uploading AppImage's to transfer.sh often fail and cause the CI to wait for
the 15 minutes timeout.

This new approach uses a bot user to create releases in a dedicated ci-builds
repository. Each PR (or master) will get a dedicated release with the AppImage
supplied as a release asset. Older assets will be deleted prior uploading the
recent build.

The bot will then (in case of a PR) create a comment in the PR, containing the
download link.

Home of the CI releases: https://github.com/nextcloud-desktop-bot/ci-builds/releases

Inspired by: https://github.com/nextcloud/android/blob/master/scripts/uploadArtifact.sh

Signed-off-by: Michael Schuster <michael@schuster.ms>

Add -no-strip cmd line arg to macdeployqt if when building with Debug profile

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

[tx-robot] updated from transifex

Merge pull request #2215 from jneplokh/pr/ownCloud-reference-fix

Change ownCloud reference to Nextcloud

Change ownCloud reference to Nextcloud

Signed-off-by: Jacob Neplokh <me@jacobneplokh.com>

[tx-robot] updated from transifex

Merge pull request #2211 from nextcloud/logo-2.7

Logo update for 2.7

Update Windows installer graphics with 2.7 logo

Signed-off-by: Michael Schuster <michael@schuster.ms>

Update logo for 2.7

Based on: https://github.com/nextcloud/promo/blob/c876a02584705835adb91e2b932b5b63bce2c59b/nextcloud-icon.svg

Signed-off-by: Michael Schuster <michael@schuster.ms>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Update our E2E API requirement

Now that we adjusted our protocol to follow the slightly updated server
API, let's make sure we don't try to talk to a server with an older API.

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

Merge pull request #2201 from nextcloud/disable_parallelism_for_e2e_remote_mkdir_jobs

Disable parallelism for e2e remote mkdir jobs

Disable parallelism if we think the parent is e2e

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

Allow to override the parallelism

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

Move PropagateRemoteMkdir ctor to the cpp file

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

Make sure the tray icon point is in screen coordinates

Previously we were using QCursor::pos() in the Linux case, which is a
fair heuristic except it will always be relative to the primary screen
and not the current screen. This explains why we had to adjust with the
virtual geometry.

In the Windows and Mac case we got the position out of QSystemTrayIcon
and that one was already relative to the current screen.

So now we use QCursor::pos(currentScreen()) which ensures we give the
coordinates relatively to the current screen also in the Linux case.

Since all platforms are now having that point in the same coordinate
system we don't need to mess around with the virtual geometry to
compensate.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2188 from nextcloud/upgrade-syncjournaldb-to27

Move existing syncjournal db files from client ver <2.7

Fix typo, use compound assignment op

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Only attempt to move if file exist

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Move existing syncjournal db files from client ver <2.7

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

[tx-robot] updated from transifex

Fetch tags for Debian build

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2183 from nextcloud/fix-delete-base-folders-instr

Prevent empty FileMap in local csync to be processed if folder not empty

Merge branch 'fix-delete-base-folders-instr' of https://github.com/Nextcloud/desktop into fix-delete-base-folders-instr

Remove debug variable

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Merge branch 'master' into fix-delete-base-folders-instr

Prevent empty FileMap in local csync to be processed if folder not empty

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Properly register folder as encrypted during sync

Turns out that when we added a new e2e folder during sync, we were
passing the wrong path to the e2e object. We have several path
convention in the sync code, just happened to be the wrong one.

I still long for the day when we'll use the type system to deal with
paths. All those strings are error-prone.

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

Recover from missing metadata on uploads

Other clients seem to allow creating empty encrypted folders with no
metadata associated to them, so let's get ready to handle it. In case of
a 404 to get the metadata, we create an empty metadata payload and pass
it further down.

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

Change the separator in the private key

It used to be a base64 encoded '|', now it is still a '|' but not
encoded, let's adjust accordingly.

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

Prevent parallelism when deleting encrypted files

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

Unlock the folder only after we actually deleted the file

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

Allow to specify the e2e-token on DeleteJob

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

Stop unlocking from within PropagateRemoteDeleteEncrypted

We need to finish deleting the file before we can actually unlock. Also
the token will be necessary for the delete to succeed.

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

Compare the unencrypted filename with the originalName

Otherwise obviously we'd never find the files in the metadata.

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

Use the encrypted path for collection listing

If we use the local path we'll get a 404.

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

Pass e2e-token to PropagateUpload subjobs

The headers() method is used to pass extra headers to the PUT jobs for
instance, definitely needed for uploads now.

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

Pass the e2e-token for encrypted MKCOL

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

Allow to pass extraHeaders to a path based MkColJob

We could specify the extra headers with a URL but not a simple path.

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

E2EE RFC has been updated to use e2e-token instead of token

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

Merge pull request #2177 from nextcloud/fix-issue-2175

Share link display the correct permissions for files and folders.

Fix text in the permissions menu for share link.

Changed word capitalization and replace && for andi to match the server.

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

Share link display the correct permissions for files and folders.

- Fix #2175:
- Shows the correct default permissions
- Shows the permissions the user set

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

Merge pull request #2168 from nextcloud/display-default-permissions-share

Display the correct default permissions when creating a share.

Display the correct default permissions when creating a share.

- The default displayed was not in sync with the server because
the client was setting permissions when creating a share while it
should get the default permissions from the server to display it to the
user first.

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

Use braced initializer list for returns in Systray::taskbarGeometry()

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Make csync_file_stat_s::isE2eEncrypted a bitfield

This is in line with other bools of that struct and gets rid
of clang-tidy's modernize-use-default-member-init warning.
(The trivial alternative to get rid of that warning would be
to use default member init for that bool variable, but I think
it is better to have it as a bitfield.)

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Remove redundant initialization of ElidedLabel::_elideMode

The value gets already initialized by default member initialization,
so there is no need to set it again in the constructor.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Remove redundant initialization of HttpCredentials::_keychainMigration

The value gets already initialized by default member initialization,
so there is no need to set it again.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Remove redundant initialization of SyncFileStatus::_shared attribute

The value gets already initialized by default member initialization,
so there is no need to set it again in the copy constructor.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Use default member init for PKeyCtx::_ctx and PKey::_pkey

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Use more auto in folderstatusdelegate.cpp

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Filter clang-tidy output on Drone for errors

The run-clang-tidy is very verbose and prints a lot of unneccessary
information.  This commit greps for errors in the output.
Moreover, we add the -quiet switch.

As a desired "side" effect, this commit repairs the issue of
run-clang-tidy-6.0 that it does not return a usable exit code.
Hence clang-tidy errors will lead to CI failures now.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Simplify clang-tidy invocation for Drone

Symlinking and running bash is not necessary.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Adjust to makeDbName returning an absolute path

PR #1451 changed the behavior of makeDbName which now returns an
absolute path. This obviously then failed in nextcloudcmd which
prepended to it the path to the local folder to sync. We just stop doing
this to have the journal db end up at the right place in that case as
well.

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

[tx-robot] updated from transifex

Handle properly null avatar

If we try to produce the avatar before we managed to connect we'd end up
trying to paint in a null image. Just return early, which will allow the
caller to do something else instead.

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

Merge pull request #2167 from nextcloud/fix_url_decoding_for_folder_encryption_status

Fix url decoding for folder encryption status

Decode the href from the XML payload

The XML we get as reply has href entities properly percent encoded.
Since we didn't go through QUrl next we didn't get a properly decoded
version. Make sure we decode it before storage.

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

No need to compute base at each loop run

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

Set maximum height of account and app menu

The menus may become too large if they have too many items
(the account menu has this in case of many accounts, the apps
menu in case of many apps).  This commit limits the maximum
height such that the menus do not hide the header.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

[tx-robot] updated from transifex

Merge pull request #2159 from nextcloud/fix_appsmenu_binding_loop

Reach out to the contentItem inside of the ListView to find the width

Reach out to the contentItem inside of the ListView to find the width

Turns out that the ListView embedded in the Menu (reachable via
contentItem) would create a binding loop if we're using its
childrenRect. But really we're interested in the total width for the
instantiated delegate *inside* the ListView. That's why we go one level
deeper and get the childrenRect of the inner contentItem instead.

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

Merge pull request #2127 from nextcloud/fix/proxy-password-storage

ConfigFile security: Migrate Proxy password to keychain

Cleanup auto pointers and qobject casts, refactor KeychainChunk

Signed-off-by: Michael Schuster <michael@schuster.ms>

ProxyAuthHandler: Add template member function execAwait to avoid code duplication

Signed-off-by: Michael Schuster <michael@schuster.ms>

Keychain: Use auto deletion in WebFlowCredentials and ConfigFile

- Also make use of the new KeychainChunk::DeleteJob

Signed-off-by: Michael Schuster <michael@schuster.ms>

Refactor KeychainChunk to use QEventLoop and add DeleteJob class

- Use QEventLoop for synchronous exec()
- Rename startAwait() to exec()
- Add code for auto deletion
- Add new DeleteJob class
- Cleanup, tweaks

Signed-off-by: Michael Schuster <michael@schuster.ms>

Refactor ProxyAuthHandler to use QEventLoop

Signed-off-by: Michael Schuster <michael@schuster.ms>

ConfigFile security: Migrate Proxy password to keychain

When specified in the config file, the Proxy password will be migrated
to the keychain, for backward compatibility and to allow admins to
overwrite an existing password by rolling out updated config files.

Once migrated to the keychain, the password will be removed from the
config file.

Signed-off-by: Michael Schuster <michael@schuster.ms>