Placeholders: Support for macOS #6290

Fix test: QTemporaryDir::filePath was added in Qt 5.9

LocalDiscoveryTracker: Separate from Folder and move to libsync

To allow relevant code to be closer together and for testing in
unittests without having to get a gui Folder.

See #6120

Logger: Guard zlib usage by ZLIB_FOUND

Similar to the use for the checksum.
I know that zlib is required in principle, but i don't have it
in one of my test building environment, and it is easier to just
disable it.

Get TestDownload to pass

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

Download: Remove useless code and add a test

From issue #7015, the code is wrong because the path is the file system path and
not the path on the DB.
But since this is a conflict, this means the reconcile will still want to download
the file from the server next sync, so we need not to worry about this case

Download: Use the <s:message> from the reply in the error message if any

Issue: #6459

Add a test for resuming downloads

Doc: Add showExperimentalOptions to conffile.rst

Wizards: Show placeholder option only if showExperimentalOptions is set

This config file option will also control other features in the future.

Wizards: Add placeholder option and warning to account wizard

Also add the warning dialog to the option in the folder wizard.

NSIS: Register placeholder extension

Also change the placeholder suffix config option to not include the dot,
the dotless form is needed in the nsis script.

Placeholders: Fix migration issues

Some edgecases weren't covered and didn't have tests yet.

SyncOptions: Add default placeholder suffix

Otherwise each test has to set this up anew.

Placeholders: Ignore placeholder files in older clients

To do this, we add the placeholder extension to the user exclude file
automatically. However, newer clients shouldn't use that exclude
pattern: so we also add version directives that allow making exclude
patterns dependent on the client version.

Placeholders: Download from shell integration

Placeholder: The extension is now a branding option

Placeholders: Fixup clicking on placeholder after previous change

Now that the name in the db is the name of the placeholder file,
we need to adjust the call to downloadPlaceholder

Placeholders: Safe migration to older client versions

Now the db entries for placeholders will have the full placeholder
paths. That way older clients will, on remote discovery, delete the
placeholders and download the real files.

Placeholders: Deal with conflicts when a placeholder exists

So "foo.owncloud" exists but the user adds a new "foo".

Placeholders: install mimetype on linux

Placeholders: Add an checkbox in the FolderWizard to enable the placeholder feature

Placeholders: Move the placeholder option from the account to the folder

PlaceHolders: Trigger a download of the placeholder and open it

On-demand downloading: Placeholder-file based prototype

- Controled by an option.
- New remote files start out as ItemTypePlaceholder, are created with a
  .owncloud extension.
- When their db entry is set to ItemTypePlaceholderDownload the next
  sync run will download them.
- Files that aren't in the placeholder state sync as usual.
- See test cases in testsyncplaceholders.

Missing:
- User ui for triggering placeholder file download
- Maybe: Going back from file to placeholder?

Merge pull request #2714 from nextcloud/ensure_sync_after_conflict_resolution

Ensure a sync happens after a conflict resolution

Ensure a sync happens after a conflict resolution

You'd expect that after a conflict resolution the file watcher would
pick up the change and trigger a sync. For some reason it doesn't seem
to happen on at least some Ubuntu systems. In such cases the user would
then still have a stale conflict entry in the activity list and wouldn't
be able to do anything with it.

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

Merge pull request #2713 from nextcloud/usermodel_properly_check_out_of_bound_index

UserModel: check indices are inside the user list

UserModel: check indices are inside the user list

The model was just checking for the user list being empty or not which
is overly optimistic. Indeed there might be cases where the id is
actually outside the boundaries so properly check for this.

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

[tx-robot] updated from transifex

Merge pull request #2708 from nextcloud/fix_new_account_wizard_open

Have the new account wizard open again

Have the new account wizard open again

UserModel can't be connected to AccountSettings if the settings dialog
doesn't exist. This is the case now since we delay the creation of that
dialog and free it after use.

Instead it should be properly channeled through the Systray object all
the way up to OwncloudGui which knows how to handle this properly.

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

Merge pull request #2692 from nextcloud/fix-issue-1029

Handle the warning message when unchecking folders for syncing.

Handle the warning message when unchecking folders for syncing.

Split widgets and slot to handle the refreshing of the view:
- refreshSelectiveSyncStatus is connected to signal dirtyChanged
and will handle big folder warning.
- slotSelectiveSyncChanged  which is connected to dataChanged signal
and will handle the selective sync warning. It fixes #1029 because
it looks for the checkbox state before showing the warning.

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

Merge pull request #2698 from nextcloud/themed-icons-v2

Use the icon color property to color the icons in the tray window header.

Use the icon color property to color the icons in the tray window header.

The icon color is set with APPLICATION_WIZARD_HEADER_TITLE_COLOR in the
theme.

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

Merge pull request #2697 from nextcloud/v3.1.5

Bump master version to 3.1.50

Bump master version to 3.1.5

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

Bump master version to 3.1.5

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

Merge pull request #2696 from nextcloud/v.3.1.0

Bump master version to 3.1.0

Bump master version to 3.1.0

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2685 from nextcloud/fix-slow-sharee-search-in-dialog

Fixed slow sharee search in the share dialog

Fixed slow sharee search in the share dialog

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

Merge pull request #2686 from nextcloud/add-email-sharing

Added sharing via email.

Added sharing via email.

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

[tx-robot] updated from transifex

Merge pull request #2690 from nextcloud/update-documentation

Update documentation on how to generate debug logs.

Update documentation on how to generate debug logs.

Minor change: bypasses => bypass.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2682 from nextcloud/debug-skip

Add debug log to check which folders are being skipped during syncing.

Add debug log to check which folders are being skipped during syncing.

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

[tx-robot] updated from transifex

Merge pull request #2666 from nextcloud/notification_actions_bring_back_from_version_2

Notification action buttons and context menu

Implemented notification action buttons and context menu for confirm/dismiss and other actions.

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

Merge pull request #2677 from briankendall/fix-tray-window-spaces-bug

Fix macOS bug where tray window causes spaces to switch

Fix macOS bug where tray window causes spaces to switch

Signed-off-by: Brian Kendall <brian@briankendall.net>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2652 from ivaradi/debian-version-discrepancy

Handle the case when the release version differs from that in VERSION.cmake

Handle the case when the release version differs from that in VERSION.cmake

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

Merge pull request #2675 from nextcloud/fix-appimage-build

fix AppImage build

fix AppImage build by using v0.10.0 for qtkeychain

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 #2667 from nextcloud/version-bump

Master is now 3.0.82.

Master is now 3.0.82.

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

Merge pull request #2540 from nextcloud/fix-issue-1458

Add last / to exclude list file path.

Check if there are patterns associated with a sync exclude file.

The previous check didn't take into the account that .sync-exclude.lst
might be empty which would crash at Q_ASSERT(_allExcludes.contains(basePath))
in the prepare function. It also takes into account that
_allExcludes[basePath] was creating new items in the list.

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

Add last / to exclude list file path.

This Q_ASSERT(this->endsWith('/')) in BasePathByteArray was the reason
for bug report #1458.

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

[tx-robot] updated from transifex

Merge pull request #2665 from nextcloud/sparkle_build_fixes

Sparkle build fixes

Simplify Sparkle handling in CMake

Fix build without Sparkle.framework

(PR #6567)

Settings: Add warning when switching update channel

[tx-robot] updated from transifex

Merge pull request #2647 from nextcloud/cherry_pick_updater_fixes_and_improvements

Cherry pick updater fixes and improvements

Please the clang-tidy overlord

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

Don't scare users with critical parser log messages

Include architecture in updater query

Fixes: #8074

Cleanup stirngs

Updater: Fix crash when there is an XML error

The problem was accessing the lines with an off by one error, while printing
the log.

Other problem included the fact that QDebug added spaces, quotes, and other
things which made the error lot look right in the console.

Issue #7545

Updater: Fix Version numer not shown in the user visible string

There was one argument too many in the string.
Bug was introduced in commit 21ac396b

Issue: #7288

Updater: Improved logging output #7388

NSISUpdater: Improve logging for version comparison

Updater: Log parsing issues in autoupdatLog parsing issues in autoupdatee

Autoupdater: Log the URL #3475

Updater: Preserve target version string

For #7217

Updater: Make "Update manually" not also "skip this version"

See #7217

WinUpdater: More useful options on update failure #7217

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

Updater: show the human readable version in the settings

The xml looks like this:

   <version>2.5.0.328</version>
   <versionstring>ownCloud Client 2.5.0 alpha1 (build 238)</versionstring>

And we should show the version string

Issue #6602