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

Update comments

Everyone uses their private mail, so ensure people can get in touch.

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

[tx-robot] updated from transifex

Merge pull request #2534 from nextcloud/connection_wizard_improvements

Connection wizard improvements

Ensure we use the app name in the GUI

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

Fix the secure lock label aspect

Make sure we got a fixed size for that label. Also ensure that the
pixmap we display there is properly scaled to fit while maintaining the
aspect ratio.

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

Hide the slideshow when going to the login page

This is indeed very distracting otherwise.

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

Add a label explaining the server address field

This is as discussed in #1160

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

Improve wording on the wizard buttons

This is as discussed on #1158

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

[tx-robot] updated from transifex

Merge pull request #2543 from renyhp/patch-2

Update bookmarks location

Update bookmarks location

Signed-off-by: renyhp <renyhp@disroot.org>

Merge pull request #2542 from nicolasfella/puzzel

Avoid string translation puzzle

Avoid string translation puzzle

Combining translated strings like this makes them hard to translate since the order of words is different between languages.
Use proper placeholder strings instead.

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

Merge pull request #2541 from nicolasfella/work/nullptrflags

Don't use nullptr for QFlags

Don't use nullptr for QFlags

This fixes some deprecation warnings

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

Merge pull request #2221 from nicolasfella/declarative

Make QML code more declarative by using properties

Make QML code more declarative by using properties

By using properties and property bindings the QML code gets more declarative rather than imperative, which is considered better.

This patch:
- Introduces a currentUserId property in UserModel that replaces the equivalent Q_INVOKABLE call
- Introduces an avatar property in User that contains the avatar's image provider url without any fallback
- Introduces new image provider urls for fallback images
- Moves the fallback image selection to QML since we want different fallbacks according to where it is used
- Wires up the necessary signals to propagate a changing avatar

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2530 from nextcloud/repair_windows_build

Repair the Windows build

Export the WordList functions

Now that they are used from gui/ they need to be properly exported so
that linking doesn't fail when visibility is activated (only on our
Windows build it seems).

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

Merge pull request #2521 from nextcloud/use_friendly_user_agent_during_authentication

Use friendly user agent during authentication

Use setHeader instead of setRawHeader for the user agent

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

Use friendly user agent during webflow2 auth

This way the server displays a less scary string while granting access
from the browser. Also this same string will be used in the "Devices and
sessions" section of the server settings.

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

Add a function for a more friendly user agent string

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

Don't force user agent if one is already set on the request

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

[tx-robot] updated from transifex

Merge pull request #2514 from nextcloud/make_it_easier_for_user_to_provide_debug_information

Make it easier for user to provide debug information

Also add build infos in the archive

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

Also add the command line arguments in the archive

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

Add a button to create a debug archive

This will harvest everything we might need for debugging purposes:
 * config file
 * sync journal dbs
 * log files

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

Add dependency on QtGuiPrivate

I'm not a huge fan of using private APIs but QZip is really the API with
the least hassles for our debug archive need. No external dependency and
we know it is generally available and stable despite the lack of
stability promise.

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

Expose more of the logger state

It is better to rely on the Logger state to know exactly where we're
logging. Indeed due to the the various ways to impact its state the
config alone might not now where we're logging.

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

Make sure --logfile has precedence on config

Since we changed the default in the config file and since log dir had
precedence on log file, the --logfile command line option wasn't doing
anything anymore.

We make sure it has an effect again overriding --logdir or the logDir
config entry.

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

Change the defaults for logging

Turn on the logging by default for everyone. Let's use a log dir within
the config directory of the application and have debug logs expiring
after a day.

This obviously means we'll generate quite some logs but with the
automated compression it shouldn't be too horrible. Obviously that
scales with the amount of files and syncs occurring. In our tests with a
large setup we're around 100 MB for a day worth of logs, this shouldn't
be too much of an issue on today's average desktop/laptop.

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

Merge pull request #2520 from nextcloud/handle_askForOptionalPassword_capability

Handle ask for optional password capability

If askForOptionalPassword is enabled preset a random password

This is the same approach used on the server side. Turns out I quite
like it, this avoids popping up a dialog to the user and since she won't
know the password she'll have to set a new one anyway or disable it.

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

Since we must ask for a password this can't be a default public link

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

Add sharePublicLinkAskOptionalPassword to Capabilities

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2512 from nextcloud/fix_updater_with_github_downloads

Handle redirects when downloading updates

Handle redirects when downloading updates

This is necessary for downloads coming from Github for instance. They
are systematically redirected and we'd just fail the download.

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

Merge pull request #2500 from nextcloud/docs-msi

Windows MSI: Update Docs & add SKIPAUTOUPDATE property

Docs: Be slightly more modern and use 64-bit examples

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

Windows MSI: Add SKIPAUTOUPDATE property for the skipUpdateCheck setting

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

Docs: Use our app name in the MSI docs

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

Windows MSI: Stay compatible with the upstream "StartMenuShortcuts" feature

Even though we only create a program shortcut in the Start Menu, try to make administrators lives easier by not diverging feature and option naming.

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

Docs: Adopt latest MSI doc enhancements from upstream master

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

Docs: Adopt upstream commit fe04300 for MSI installation

See: https://github.com/owncloud/client/commit/fe043006c8f4a43b033243a85c294d21c7450238

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

Docs: Update macOS version info

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2497 from nextcloud/fix/mac-folderwatcher-memory-leak

macOS: Fix memory leak in FolderWatcherPrivate::startWatching

Fix source file access modes

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

macOS: Fix memory leak in FolderWatcherPrivate::startWatching

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

[tx-robot] updated from transifex

Merge pull request #2478 from Aldaris1985/patch-1

Update propagateupload.cpp

Update propagateupload.cpp

Typo corrected.

Merge pull request #2493 from nextcloud/kesselb-patch-1

Delete FUNDING.yml

Delete FUNDING.yml

[tx-robot] updated from transifex

Merge pull request #2487 from nextcloud/add-talk-icon

Add a clickable icon for Talk notifications.