IssuesWidget: Fix insertion of sync errors before item errors

SyncEngineTest: _size now correct for uploads #5855

Merge remote-tracking branch 'origin/2.3'

IssuesWidget: Add button to retry 507 errors #5537

Since these errors are blacklisted, it can take up to 24h to retry items
that had a 507 error for a while. This way users can intervene and cause
an upload attempt immediately.

IssuesWidget: Hide folder column when possible

Add ElidedLabel

A label that adjusts its text based on Qt::TextElideMode.

PropagateUpload: Better messaging for 507 #5537

It now produces a summary error message indicating the problem.

Adjust blacklist database table to contain 'errorCategory'. This is
useful for two things:
  - Reestablishing summary messages based on blacklisted errors. For
    example if we don't retry a 507ed file, we still want to show the
    message about space on the server
  - Selectively wiping the blacklist: When we have ui for something like
    "I deleted some files, please retry all files now!", we want to
    delete all blacklist entries of a specific category only.

PropagateUpload: Put upload error handling in one function

ProgressInfo: Carry a sync status

* A bunch of code was determining sync status by ad-hoc comparing some
  progress info fields. It can now just check the status, making it
  easier to comprehend.
* There's a clear indication for "a new sync is starting", which helps
  wiping the issues tab at the right time.

SyncEngine: Introduce overall errors that are not tied to a file #5746

For now we use them for:
* csync errors: This allows them to appear in the sync issues tab
* insufficient local disk space, as a summary of individual file errors

Insufficient remote space will use them too, as might other issues that
are bigger than a single sync item.

[tx-robot] updated from transifex

More refs. to corresponding issue/PR on 2.3.3 changelog

ChangeLog: 2.3.3

OAuth: Redirects to the server in case of sucessfull login

Requires https://github.com/owncloud/oauth2/pull/45

This commit moves the reply after we got the token reply from
the server, that allows to reply with an error to the browser
if the login does not work.

[tx-robot] updated from transifex

IssuesTab: Hide filters if they wouldn't be useful

As suggested by @dragotin

FolderMan: Add missing folderListChanged emits

She signal of a folder list change was not emitted when folders were
deleted from the list.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Account/Credentials: Have identical lifetimes

The QNAM may continue to outlive both.

Rename Credentials::getQNAM() to createQNAM() while we're at it - it's
used to make a new QNAM that will subsequently be owned by the Account
object.

See d01065b9a12e69ca493a232f3a8e8f3d416fed52 for rationale.

Relates to
d40c56eda561e3a541bf1b23f70fa8d659d3037e
147cf798a6f13c9b53a9f1fb2db1ef26c8c63273

[tx-robot] updated from transifex

shell/windows Build 46: Private links context menu

Update building.rst

Link share: Remove direct download if unavailable #5837

It would have been much nicer to keep the menu assigned to the
QToolButton, but if one switches away from InstantPopup (to adjust the
entries before they're displayed), the button always gets a menu
indicator that can't be removed.

Link shares: Add "show file listing" option #5837

shell integrations: Change share menu title to "Share..."

Since the appname is in the context submenu title already.

Also drop APPNAME string.

SocketAPI: Change strings to a consistent _MENU_TITLE postfix

shell/macos: Remove legacy (< 10.10) overlay icons

Having to update the plugins to add a submenu in the context menu,
it's more cost-effective to remove support for legacy macOS versions
than implementing and testing this setup.

shell/macos: Use a submenu to include private link actions

shell/windows: Use a submenu to include private link actions

Refactor things a bit to be able to reuse some code and
clean things up.

SocketApi: Add GET_STRINGS:BEGIN and END

The Windows shell extension relied on the response of
SHARE_MENU_TITLE to advance its state machine, but in order
to use the new GET_STRINGS instead, we need to know when the
last string was received. Also add BEGIN for consistency.

SocketApi: Adjust the format of the mailto: URL

Minor tweaks that weren't actually an issue, but just in case.

SocketApi: Improve logging

Introduce private link sharing #5023

* SocketAPI has COPL_LOCAL_LINK / EMAIL_LOCAL_LINK commands
* The nautilus and dolphing shell integrations show a submenu from which
  one can share as well as access the private link.
* The SocketAPI provides a new GET_STRINGS command to access localized
  strings.
* The private link can also be accessed from the user/group sharing
  dialog.
* The numeric file id is extracted from the full id to create the
  private link url.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Fix crash on account deletion

Calling forgetSensitiveData() on account deletion leads to a timer for
clearQNAMCache() being queued. Then the Account object is deleted. The
Credentials object stays alive for now because it has a deleteLater
deleter.

If the timer calls into a slot on the Credentials object, the _account
pointer will be invalid at this time.

As a workaround, move the target slot to Account - that way it will not
be called as the account object is already destroyed.

However since Account and Credentials are mutually dependent, it would
be much preferable if their lifetimes were linked, avoiding this
category of bugs.

The current behavior was introduced in
d40c56eda561e3a541bf1b23f70fa8d659d3037e and I currently don't
understand why - maybe there's another way of dealing with the problem
that existed then.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Add a more functional error view #5516 (#5861)

* Add a more functional error view #5516

* Allow filtering of ignores and warnings to see only important bits.
* Navigate from the folder view to the error view by clicking on the
  error list with the red background.
* Move the error list into its own ui file to allow easier extension.
* Fix issue around tab id handling in ActivitySettings.
* Rename "Action" column to "Issue".
* Change mouse cursor to hand over button and new error list area

Several OSX fixes provided by guruz.

AccountState: Add a 1-5min reconnection delay #5872

This only applies when the server was explicitly in maintenance mode or
when it was 503-unavailable.

Blacklist: Don't let errors become warnings #5516

Before, blacklisted errors were set to FileIgnored status and hence
displayed as warnings. Now, they have their own BlacklistedError
category which allows them to appear as errors in the issues list and in
the shell integration icons.

main.cpp: Fix a compiler warning

 src/gui/main.cpp:112:9: warning: bool literal returned from 'main' [-Wmain]

Used 1 to keep previous behaviour. I supposed the code was meant
to return success (0), but it does not really matter anyway.

SyncEngine: Keep local size in SyncItem #5855

SyncEngineTest: Fix test reliability

There was a rounding issue in the mtimes which sometimes resulted in an
off-by-one error. Caused by storing a full QDateTime in the FileInfo but
the mtime saved to the disk being truncated to seconds.

[tx-robot] updated from transifex

Fix check_csync_update

Commit 816096311027bac0f391ea97d0b61b4175e9b8f9 changed the
SQL requests from csync to read from the checksumtype table.
So we need to add this table in the fake DB as well

Fix check_csync_exclude test

Commit 4697f0274f282cafd28cbad07066efb397dbf21b remove .htaccess from
the sync-exclude.lst, so use another known excluded pattern

Classify chunked items correctly. Issue #5850

TestSyncJournalDB: Fix concurrent usage on CI

csync tests: remove stray 'print'

SyncEngine: Add unittest for SyncFileItem properties #5855

Checks instruction, direction, size, modtime for three common cases.

csync tests: Fix for new owncloudcmd return codes

These were introduced in 4af45394f9f61fbc68c9f0a4d2e5387d14052e3b

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Update central.owncloud.org link

Desktop client category was renamed, old link was 404 - see: https://central.owncloud.org/t/new-central-categories-you-decide

Update central.owncloud.org link

Desktop client category was renamed, old link was 404 - see: https://central.owncloud.org/t/new-central-categories-you-decide

[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

[tx-robot] updated from transifex

Jenkinsfile: Some comments

CSyncSqliteTest: Fix test

test_journal.db: Add checksumtype table

Merge remote-tracking branch 'origin/2.3'

[tx-robot] updated from transifex

Jenkinsfile: Compile in debug because of all the asserts

Jenkinsfile: Compile in debug because of all the asserts

Jenkinsfile: Run verbose tests

Jenkinsfile: Run full testing

Jenkinsfile: Run full testing

Jenkinsfile: 2.3.3 will ship with Qt 5.

This should fix the current CI run too

owncloudcmd: Explain --debug parameter

[tx-robot] updated from transifex

Ignore ui: Add the sync journal patterns #5844

The ignore patterns for the journal files are hardcoded. Add them to
the UI to make them discoverable.

Journal: Don't use a ._ path if it won't work #5633

When synchronizing a folder on a samba share, creating files that begin
with ._ is often forbidden. This prevented the client from creating
its ._sync_abcdef.db file.

Now, it'll check whether the preferred filename is creatable, and if
it isn't it'll use .sync_abcdef.db instead.

The disadvantage is that this alternative path won't be ignored by
older clients - that was the reason for the ._ prefix.

[tx-robot] updated from transifex

shell/Windows: Update binaries to build 45

shell_integration/windows: Create _newQueries as a non-signaled auto-reset event.

Since ResetEvent() is never called, the call to WaitForMultipleObjects()
can always return immediately once the event has been signaled.

Initialize stop variable.

Avoid stopping the worker thread immediately if the stop variable happens to be true.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Remove debug messages by default from sync.engine

Other categories were already set properly.

Compare the hash of files with identical mtime/size #5589

* For conflicts where mtime and size are identical:

  a) If there's no remote checksum, skip (unchanged)
  b) If there's a remote checksum that's a useful hash, create a
     PropagateDownload job and compute the local hash. If the hashes
     are identical, don't download the file and just update metadata.

* Avoid exposing the existence of checksumTypeId beyond the database
  layer. This makes handling checksums easier in general because they
  can usually be treated as a single blob.

  This change was prompted by the difficulty of producing file_stat_t
  entries uniformly from PROPFINDs and the database.

SyncEngineTest: Add network override

This is useful for monitoring what kind of network requests are
sent to the fake server. Such as "did this sync cause an upload?"
and "was there a propfind for this path?". It can also inject
custom replies.

SyncEngineTest: Send 'checksums' in FakePropfind

SyncEngineTest: Fix date locale related bug

The client is very picky about date strings it accepts. If dates are
formatted with a non-C locale (such as localized weekday names), it
fails to parse it and tests fail in subtle ways.

shell/Windows: Update binaries to build 45

shell_integration/windows: Create _newQueries as a non-signaled auto-reset event.

Since ResetEvent() is never called, the call to WaitForMultipleObjects()
can always return immediately once the event has been signaled.

Initialize stop variable.

Avoid stopping the worker thread immediately if the stop variable happens to be true.

AccountSettings: Triggering log in re-ask about previously rejected certificates #5819

Issue #5819

[tx-robot] updated from transifex

OAuth: cleanup debug messages

- Add category to the all messages (they did not have it was merged right after
the patch to add category everywhere, but this code did not have it.)

- Make sure there is no warnings in the normal flow. (The wizard does a request
without authentication to determine the auth type)

Don't link dirrectly against openssl

All our crypto code is handled by qt nodaways.
No need to carry this dependency.

Especially since it causes warnings on system where there are
twp openssl version installed:
/usr/bin/ld: warning: libcrypto.so.1.0.0, needed by /usr/lib/libQt5Network.so.5.9.0, may conflict with libcrypto.so.1.1