Merge pull request #2648 from nextcloud/debian_packages_should_not_block_ci

Issue a warning for Debian pipeline failures but don't fail the CI

Issue a warning for Debian pipeline failures but don't fail the CI

Since this fails every time we bump the version number and takes time to
catch up while failing every following PRs, let's prevent this pipeline
from failing the CI.

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

Merge pull request #2630 from nextcloud/bump_to_version_3.0.81

Master is now 3.0.81

Master is now 3.0.81

This feels a bit like moving back in that case, but that's due to the
change of convention in the release process. We used to bump early but
now we're moving to a point when we bump at the last minute.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2616 from nextcloud/move_journaldb_files_back_to_sync_folders

Move journaldb files back to sync folders

Migrate journal dbs files back to the sync folder

In case of past collisions during the 3.0 times... well one will resync
from scratch unfortunately. But if that happened there are likely other
problems which occurred.

Also this might fix some of the bugs with people loosing settings from
the database. Indeed the -wal and -shm concatenations were wrong. Using
append was in fact changing the folderDefinition member which (I guess)
would potentially lead to funny ".db-wal-shm-wal-shm" names.

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

Revert "moved default db sync file to Qt standard path AppDataLocation"

This reverts commit fc6b895f524cfadd7edff83098c70d7cc44248a9.

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

Partial Revert "Adapted for cmd version and added migration step for config file"

This partially reverts commit 67ed54671b5f3be9520089f85d9e2e7aebc037f6.

The migration code was left untouched.

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

Revert "Adjust to makeDbName returning an absolute path"

This reverts commit 2492e241ac1b22cfbdbcb97739e9365fba69ac22.

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

Merge pull request #2531 from nextcloud/bring_back_systray_menu_pause_sync_action

Bring back the "Pause sync" action in the systray context menu

Bring back the pause/resume action in the systray menu

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

Move pause/unpause helpers to Systray

We had signals just to call those backs in ownCloudGui, they were
otherwise unused. So let's move them inside of Systray since it's
specific to it anyway.

Also fix the dangerous call to sender(). We can call this function
without going through a signal/slot connection and also it's never
connected to an AccountState.

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

Merge pull request #2620 from nextcloud/fix_crash_when_displaying_share_dialog

Make sure the settings dialog exist before hiding it

Make sure the settings dialog exist before hiding it

Because of PR #2580 the settings dialog doesn't always exist. We need to
check for it first before placing calls to it.

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

Merge pull request #2621 from nextcloud/wiswedel/documentation/chunkedUpload

[documentation] upload chunks config

[documentation] upload chunks config

documenting https://github.com/nextcloud/desktop/commit/e86499d9907f3bab5eac6e371eef772f8befd2e5

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2615 from nextcloud/fetch_apps_on_connectivity_changes_only

Fetch apps when we get connected only

Fetch apps when we get connected only

This was right now on the check connectivity beat which is too much when
you got many users. Be more conservative there and only update the list
of apps when the account gets connected.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2603 from nextcloud/remove_findqt5keychain_cmake_module

Get rid of FindQt5Keychain.cmake

Adjust AppImage build of QtKeychain to be found by cmake

Now that things are done in a more standard way, let's adjust the
AppImage build so that QtKeychain is picked up properly now that our
FindQt5Keychain.cmake file is gone.

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

Get rid of FindQt5Keychain.cmake

QtKeychain provides Qt5KeychainConfig.cmake and friends nowadays, so no
need to have a less reliable and outdated find module on our end.

Also this shows that we were including keychain.h in the wrong way and
were not using the link target, so both got fixed as well.

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

Merge pull request #2589 from sryze/windows7-fixes

Misc fixes for Windows 7

Misc fixes for Windows 7

Signed-off-by: Sergey Zolotarev <sryze@protonmail.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2591 from nextcloud/fix_conflict_dialog_wording

Remove last left over of the "remote" wording

Remove last left over of the "remote" wording

I switched from "remote" to "server" but apparently left one behind by
mistake...

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

Merge pull request #2586 from nextcloud/Valdnet-patch-2

l10n: Change the triple dot to ellipsis

l10n: Change the ellipsis

l10n: Add a space in generalsettings.ui

Merge pull request #2582 from nextcloud/macos_add_more_information_on_skipped_file_watcher_event

Also output the event flags in the debug logs

Also output the event flags in the debug logs

When we skip a watcher event on mac, we log the file path but not the
event flags which came with it. Let's add it, it should help figure out
what's going on with #2578.

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

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

Triple dot to ellipsis

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 #2581 from nextcloud/ensure_systray_window_doesnt_prevent_windows_session_close

Ensure we quickly show/hide the systray window on startup

Ensure we quickly show/hide the systray window on startup

This is quick enough that no flickering should appear in practice.

We end up doing this because for some reason on Windows (I dug up deep
into the Windows QPA without nailing it down) not showing that systray
window at least once prevents the app object to return from exec() when
the session ends.

It's as if that window would be in some limbo state (neither opened nor
closed) which would prevent quitting. Clearly what we're doing here is a
workaround...

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

Merge pull request #2527 from nextcloud/improve_conflict_handling_gui

Improve conflict handling gui

Use the ConflictDialog directly from the activity list

No need to go to the file manager first to then have the user go through
the context menu, just popup the dialog directly.

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

Move the activity action behavior on the C++ side

We better do this before this business logic grows in the QML side and
gets out of control. We'll need finer grained information due to the
conflict handling anyway.

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

Repair SettingsDialog::showIssuesList

This was completely disabled code and the connect was gone... So let's
introduce the connect back and have it trigger the main window while
selecting the right user. This way the new activity list is displayed
and since the conflicts are shown at the top this is hopefully obvious
again.

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

Add a function to know the user id matching a given account

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

In the case of conflicts, have the socket api propose the config dialog

This should be safe in the case of conflicts in folders on which the
user can write. For other cases we still use the older actions.

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

Add a conflict dialog to help the user solve them

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

Add KeepBothVersions as a possible solution to ConflictSolver

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

Move some of the SocketApi conflict handling in a ConflictSolver class

The socket api move and delete commands are not strictly about conflicts
since they also deal with files which couldn't be uploaded for some
other reason. Still the new ConflictSolver could be used in those cases.

This opens the door at reusing that logic in other places.

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

Repair the canAddToDir logic

This could only work at the root of the sync folder where the record for
the parent folder would be invalid. Otherwise the negation would be
wrong... assuming you can add a file only if the permission is not
there.

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

SocketAPI: Introduce conflict resolution actions #6252

For conflicts generally as well as new files in read-only directories
the context menu will now present delete and move options.

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

FileSystem: make removeRecursively() reusable

We want to use it for deleting directory conflicts.

FileSystem: Fix bad indentation

Merge pull request #2580 from nextcloud/delay-settingsdialog-init

Delay initialization of SettingsDialog

Fix crash in SettingsDialog with delayed initialization

setWindowFlags triggered changeEvent, thus causing a crash in customizeStyle.

This fix should be kept even if we decide against delayed init in the future.

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

Delay initialization of SettingsDialog

ownCloudGui::slotShowSettings already got what it takes to create it only when we try to show it for the first time.

This however has some implications:

Pros:
- Only created when needed, while testing saved ca. 20 MB of RAM and got freed again after closing the dialog.
- Since we defaulted to the new Tray UI from 3.0, this is an added bonus for users don't opening the settings.

Cons:
- Resources like the avatar image have to be refetched everytime the dialog is recreated.
  This may be desired as well, because it ensures displaying no outdated info (e.g. on connection issues).

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

[tx-robot] updated from transifex

Merge pull request #2575 from nextcloud/expose_branding_values_to_qtquick

Expose branding values to qtquick

Use the Theme colors for the header part of the window

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

Expose Theme on the QML side

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

Expose the branding related members of Theme as properties

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

Don't hardcode the color of text in the header

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

Get rid of ncBlueHover

Use a similar trick of a semi-transparent rectangle on top when the
mouse area is hovered. This way it will always work whatever is the
background color.

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

Merge pull request #2572 from nextcloud/avoid_depth_infinity_propfind_for_e2ee

Avoid depth infinity propfind for e2ee

Don't use depth infinity anymore to get the folders e2ee status

This way we avoid the expensive SQL query on the server at the price of
more round-trips since we're doing the recursive traversal by hand now.

Also it turns out this depth was used for all the other propfind calls
during sync when we want fresher information regarding a folder. This
was very inefficient in all cases and won't happen anymore.

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

Make sure jobs don't outlive ClientSideEncryption

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

Factor out the logic to start a new e2ee status job

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

Mention the folder we listed in the debug logs

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

Change the way we store the job result

If there's more than one job we need to unite the maps not simply overwrite
them.

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

Start managing a list of GetFolderEncryptStatusJob

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

Add a way to know which folder was listed for encrypted status

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

[tx-robot] updated from transifex

Merge pull request #2568 from nextcloud/fix-macos-crash-403-folder-view

Fix crash when clicking on folder with status 403 in the main dialog.

Fix crash when clicking on folder with status 403 in the main dialog.

LsColJob was still being used after delete was called.

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

[tx-robot] updated from transifex

Merge pull request #2551 from nextcloud/fix_share_dialog_animation_for_enforced_password_policy

Fix share dialog animation for enforced password policy

Stop the animation if we got an error message

Otherwise it would spin forever while we know we're not doing any work
anymore since we got a message from the server.

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

Inform _emptyShareLinkWidget of linkShareRequiresPassword signals

All the other ShareLinkWidgets process that signal (which allows to
display error messages for instance) but not that one for some reason.
That being said it might need to deal with an enforced password
situation.

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

Emit last to avoid use after delete crash

It turns out the shareDeleted() signal is connected to a function
cleaning up the ShareLinkWidget holding the last shared pointer to the
Share object. Since we use member variables for calling updateFolder()
this would lead to using deleted objects.

Just swap the call and the signal to have everything back in order.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex