Share dialog: use the original name and not the virtual file name

When sharing a virtual file, we should actually use the original file name
not the virtual file name

Issue: #6461

Virtual files: Only remove virtual file once on download

With thanks to @ogoffart for spotting the problem.

Settings migration: Preserve future settings where possible

See discussion in #6506

Config: Add version flags to accounts and folders

Also, if there is too-new configuration, backup the file, show a
warning message asking the user whether it's ok to discard the
configuration from the future.

See #6504

Rename Placeholders to Virtual Files in code #6531

SyncRunFileLog: Add the requestid

https://github.com/owncloud/client/pull/6427#issuecomment-383879509

Blacklist: remember the X-Request-ID

Issue #6420
Store the X-Request-ID in the SyncFileItem and also in the blacklist.
Note that for consistency reason, the X-Request-ID is also in the
SyncFileItem if the request succeeds.

Currently there is no UI to access it, but it can be queried with sql
commands

Wizard: Add a "(recommended)" to the main option #6470

Rename "placeholder" feature

Placeholder: Stop adding ignore pattern

Because we can't make older clients preserve the version directive that
was attached to it.

See #6504 and #6498

Placeholder: Don't contain "stub"

Placeholders: Save to key that's invisible to <2.5 clients #6504

FolderWizard: fix crash when experimental feature are disabled

The _placeholderCheckBox only exists if the experimental features are enabled

Found via the crash reporter
https://sentry.io/owncloud/desktop-win-and-mac/issues/556407777/

Correct incorrect reference to OSX Menu bar

This fixes https://github.com/owncloud/documentation/issues/4109.

Share Dialog: Disable workaround for macOS too #6185

CMake: Add a default placeholder_suffix

To make themes work that don't define it explicitly.

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