Merge pull request #2948 from nextcloud/bugfix/conflict-dialog-issue

Fix crash when opening conflict dialog

Iteration

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

Fix crashes with conflict dialog

When the client runs and a conflict gets detected, the sync engine runs
two times.

On the first run, the sync engine detects the conflict, marks the
file as a conflict and propagates that to the GUI. This leads to an
error notification with the original filename in the main dialog.

The sync engine runs then a second time. On this second run, the file
that originally caused the conflict is not anymore a conflict
file. Instead, the sync engine detects the conflicted copy and
propagates that file as a conflict to the GUI.

When opening the conflict dialog with the original file name (not the
conflicted copy) a crash happens. Usually, the two sync runs are really
fast, so the user does not notice the first notification. However, a
problem can occur if a conflict gets created while the client is not
running. Since then, the client does not do two sync runs. It does only
run once.

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

[tx-robot] updated from transifex

Merge pull request #2958 from nextcloud/bugfix/vfs-win-download-large-files

VFS Windows: Fix download of large files

VFS Windows: Fix download of large files

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2931 from elsiehupp/build-instructions

Improve OSX build Instructions

Update OSX build instructions

Signed-off-by: Elsie Hupp <9206310+elsiehupp@users.noreply.github.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2951 from nextcloud/enhancement/better_logging_of_cfpapi_error

Log error message from CFAPI

Log error message from CFAPI

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2949 from nextcloud/staleAction

Move from stale bot to stale action

Move from stale bot to stale action

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2935 from nextcloud/do-not-auto-start-debug-build

Do not add debug build to startup programs.

Do not add debug build to startup programs.

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

Merge pull request #2937 from nextcloud/staleSecondTry

Stale: issue must have bug and must not have approved label

Stale: issue must have bug and must not have approved label

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2925 from nextcloud/fix-incorrect-logger-output-for-non-latin-strings

Fix incorrect logger output for non-Latin strings.

Fix incorrect logger output for non-latin strings.

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 #2928 from nextcloud/add_missing_override_specifier

Add missing override specifier

Add missing override specifer to WebFlowCredentials

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

[tx-robot] updated from transifex

Merge pull request #2914 from marcohald/master

Configuration Options for Windows Registry

Merge branch 'master' into master

Configuration Options for Windows Registry

Added the configuration  options
        confirmExternalStorage
        crashReporter
        newBigFolderSizeLimit
        useNewBigFolderSizeLimit
    to the Windows registry

Signed-off-by: Marco Hald <marcohald@users.noreply.github.com>

Merge pull request #2919 from nextcloud/validate_ssl_providers_certificate

Validate the providers ssl certificate

Validate the providers ssl certificate

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2906 from nextcloud/fix-url-scheme-security-issue

Validate sensitive URLs to onle allow http(s) schemes.

Validate sensitive URLs to onle allow http(s) schemes.

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

Default parameter nullptr widget for openBrowser.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2904 from ivaradi/ui-namespace

Move FolderCreationDialog into the OCC namespace

Put FolderCreationDialog into the OCC namespace.

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

[tx-robot] updated from transifex

Merge pull request #2892 from nextcloud/repair_e2ee_on_non_remote_root_sync_folder

Repair E2EE on sync folders which don't point to the root of the server on the remote end

Make sure we pass a full remote path to LSCOL jobs

Indeed the path we have is supposedly not fully qualified in case of a
sync folder which doesn't point to / on the remote end. But LSCOL works
with absolute paths on the server so make sure this is what we give it
out.

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

The e2eMangledName was relative to the remote folder, repair it

This got broken during the huge discovery refactoring. I wrongly passed
the mangled name as is out of discovery, but coming from listing jobs it
was fully qualified while the jobs at propagation time and the db expect
those paths to be relative to the remote folder.

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

Merge pull request #2897 from nextcloud/allow-creation-of-new-folders-from-the-settings-dialog

Allow creation of new folders from the Settings Dialog.

Allow creation of new folders from the Settings Dialog.

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

[tx-robot] updated from transifex

Merge pull request #2901 from nextcloud/win-tests-compilation-hot-fix

Win tests compilation fix.

Win tests compilation fix. For Windows, the build was failing because of not being able to locate #include <sqlite3.h> in tests.

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

[tx-robot] updated from transifex

Merge pull request #2900 from nextcloud/make_sure_theme_resources_are_loaded

Load the theme resource explictly

Load the theme resource explictly

Otherwise they won't be loaded automatically at startup leading to lots
of missing icons. Since they're now in a static library they need that
explicit loading.

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

Merge pull request #2834 from nextcloud/compile_nextcloud_as_lib

Compile nextcoud gui as separate library

Add automoc property to updater

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

Add test utils library

The test utils library collects all dependencies that are needed for
tests and gets automatically linked with every test.

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

Compile nextcoud gui as separate library

Compiling nextcoud gui as a separate library. This is needed to more
easily write tests. The whole nextcloud application can now be linked
against the tests.

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

Merge pull request #2894 from nextcloud/fix-docs

Documentation: break line in config example.

Documentation: break line in config example.

Also make sure the following note is not displayed as one block with the
config example.

Minor change: fix typo.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2874 from nextcloud/more_active_stalebot

Reduce exempt labels list to give a chance to stale bot to do something

Reduce exempt labels list to give a chance to stale bot to do something

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

Merge pull request #2877 from nextcloud/revert_2875

Revert "Specify PUBLIC for the link targets to make them visible to w…

Revert "Specify PUBLIC for the link targets to make them visible to windeployqt"

This reverts commit 002abdc38b31d98211eb329046319430768d0037.

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

Merge pull request #2872 from nextcloud/use-push-notifications-for-activities-in-tray

Use push notifications for Tray activities/notifications fetch trigger.

Use push notifications for Tray activities/notifications fetch trigger.

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

Merge pull request #2875 from nextcloud/make_qtwebsockets_visible_to_windeployqt

Specify PUBLIC for the link targets to make them visible to windeployqt

Specify PUBLIC for the link targets to make them visible to windeployqt

I went full steam on making it all public which is not really required,
it's only really QtWebSockets we're after. Could always be fine tuned
later on if this works as expected.

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

Merge pull request #2867 from nextcloud/fix_file_resolve_dialog

Start file conflict resolve dialog asynchronous

Start file conflict resolve dialog asynchronous

Fixes #2804

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

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

Replace dots with ellipses and add spaces.

Replace dots with ellipses and add spaces.

[tx-robot] updated from transifex

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