[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 #5007 from owncloud/rebuildWinShellIntegration

Rebuild win shell integration

Fix a QAction leak (#5008)

Attach the accout menu QAction to the menu rather than the
ownCloudGui since it should be destroyed when the context
menu is cleared.

Fix windows HiDPI #4994 (#4998)

This was removed in 0194ebb2229adc33f5b9790231a3384b46474bde
because it breaks on Linux. However, it looks like it is correct
for Windows. In the meantime the surrounding ifdef has changed
from !Q_OS_MAC to Q_OS_WIN, so reverting it makes sense.

Fix a deadlock when shutting down during discovery (#4993)

Since the SyncEngine now quits and waits for the discovery thread,
the main thread can enter a deadlock where the discovery thread waits
for its directory result.

Add a 2 seconds timer to the discovery thread wait condition
to limit the deadlock time.

binaries: Update to build 42 owncloud/client#4977

shell_integration/windows: Set VS output directory in the binary submodule

This allows creating a new build simply by hitting F7 in visual studio
and then committing the changed files in the binary submodules without
having to know which files to move where.

shell_integration/windows: Add a manually-increased build number

Since this isn't yet built together with the rest, we can't reliably
use the client version without having to rebuilt the extension manually
on each release. But it would still be nice to be able to tell if a user
has the right version or not loaded, so instead just tag each update of
the DLLs with a separate release version.

This means that every time we update those binaries we'll have to manually
increment the number in the header, but worst case we'll forget from time
to time but we'll be able to know if we didn't, and know if we can rely on
the number or not.

SocketAPI: Use different pipe name #4977 (#4997)

Named pipe names that contain backslashes sometimes lead to problems.

Share: Permissions for federated shares on servers >=9.1 #4996 (#5001)

This disables the workaround 487e1fdca5ee04fc98c1ed77898df70d740967c8
for servers that are new enough to support fine grained permissions
on federated shares.

The consequence is that the 'reshare' permission is now granted by
default and that users can edit permissions on the usual fine-grained
level again.

The way the client deals with servers <9.1 is unchanged.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Tray: Add minimal mode as workaround and testing tool #4985 #4990

[tx-robot] updated from transifex

Fix Qt4 build

owncloudcmd: Fix --exclude regression

It's ok if the system exclude file is missing when --exclude is
specified explicitly.

[tx-robot] updated from transifex

csync: Use the full file stat destructors (#4992)

* For csync_file_stat_t and csync_vio_file_stat_t
* Add original_name to the vio file stat destructor

Fix the Qt4 build

QSharedPointer::reset doesn't exist in Qt4.

Merge pull request #4979 from jturcotte/2.2

Memory improvements during propagation

Setup csync logging earlier (#4991)

We were missing some csync related log output during startup.

Discovered in #4967

[tx-robot] updated from transifex

This branch is headed towards 2.2.3 now

excludes: Only log if the pattern was really logged. (#4989)

Performance: Don't redundantly add the same exclude files #4967 (#4988)

Excludes: Don't redundantly add the same exclude files #4967, Never accept duplicate exclude patterns #4967

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Reduce the memory usage of the StopWatch during propagation

Use a QMap to avoid using a full hashtable for only a few entries, and
clear the QMap once we're done with the measuring. This saves a few
hundred bytes per job during propagation that would otherwise only be
freed at the end of the sync.

Delete the ComputeChecksum object when the job is finished

Each object takes almost 1k and we don't need it once the propagation is
done.

Prevent the sync protocol widget from over-using memory

During propagation, we create a line for each file, taking memory, but
we delete all lines passed 2000 right at the beginning of the next sync.
Since the user has little chances of being able to read past those 2000
lines in the log, we might as well keep it capped at 2000 also during
propagation to prevent it from eating memory.

Fix a crash on if shutting down during propagation

The SyncRunFileLog owned by the Folder must be destroyed after the
SyncEngine since the SyncEngine will abort during destruction, resulting
in all jobs being aborted.

It's possible that this crash only happens with a debug build.

VERSION.cmake: This is 2.2.2

[tx-robot] updated from transifex

SocketAPI: Push new status of dirty files regardless when not synced (#4970)

The FolderWatcher inserts files to be marked as SYNC and we
currently assume that all file statuses will be updated by the
following sync. It's however possible that the FolderWatcher
notify us of a change that csync won't consider necessary to
propagate, in which case a new status wouldn't be pushed and
the file manager would continue showing this file as syncing.

Re-push the file status when emptying the dirty files list
before propagating to avoid this issue, most likely the OK
status.

Checksum speedup. (#4968)

No need to allocate (and initialize to 0) a 10 MiB buffer for each files, even
when most files are much smaller than that.
So make sure the buffer that we allocate is not bigger than the file size.
And Also 10 MiB is a bit big for a buffer. 500 KiB should be more than enough.
(Too big allocations can cause problem because of memory fragmentation and such)

SyncEngine: make abort and closing more robust

We first need to set the abort flag to csync and then aborting the discovery
job, otherwise, the discovery thread could start a new job in the mean time.

We also need to make sure that the thread has existed before we destroy the
exclude list.

[tx-robot] updated from transifex

Merge pull request #4971 from owncloud/fixAccountHandleSslErrorsCrash

Attempt to fix the Account::slotHandleSslErrors crash

Attempt to fix the Account::slotHandleSslErrors crash

Events from the crash reporter suggest that the QNAM and its
child replies might get deleted before returning from this method
and the only possible cause we can see is that the inner event
loop has something to do with it.

Try keeping a ref on the QNAM while in this method to make sure
that it won't get deleted by the inner event loop.

Account: Use smart pointers to hold its credentials and QNAM objects

Also make sure that both use QObject::deleteLater as their delete
function.

PropagateUpload: fix crash in PUTFileJob::slotTimeout

Same fix as in commit 60c101d9

From the crash reporter:

Crash
EXCEPTION_ACCESS_VIOLATION_READ at 0x4
qnetworkreply.cpp in QNetworkReply::request at line 476
propagateupload.cpp in OCC::PUTFileJob::slotTimeout at line 100
moc_abstractnetworkjob.cpp in OCC::AbstractNetworkJob::qt_static_metacall at line 98
qobject.cpp in QMetaObject::activate at line 3716
moc_qtimer.cpp in QTimer::timeout at line 192
qtimer.cpp in QTimer::timerEvent at line 247
qobject.cpp in QObject::event at line 1267
qapplication.cpp in QApplicationPrivate::notify_helper at line 3722
qapplication.cpp in QApplication::notify at line 3505
qcoreapplication.cpp in QCoreApplication::notifyInternal at line 932

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Fix docker build instruction for windows

[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 #4949 from owncloud/jnweiger-tray-patch-patch-patch

Update accountsettings.cpp fixing the tray-patch

Update accountsettings.cpp

Workaround for Ubuntu 16.04 tray issue #4944 (#4947)

[tx-robot] updated from transifex

[tx-robot] updated from transifex

fix win install instructions: build path and docker image name (#4901)

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Fix compilation on windows

Folder: fix removing folder with escaped characters (#4943)

FolderDefinition::save and load escapes the alias. We also need to escape
it when we remove it.
New folder can't be created with alias that needs escaping, but old folder
from old config may still exist, and we must allow user to delete them.

Folder: Ignore change from the watcher if the file has not changed its mtime or size (#4942)

For issue #4927:
On Windows 10, we get a notification after the sync is finished for file that were
just downloaded. The guard we have against our "own changes" are only working when
the sync is running and the OwncloudPropagator still alive.

[tx-robot] updated from transifex

Merge pull request #4939 from owncloud/osx_filewatcher_fix

OS X filewatcher fix

OS X: White-list the folder watcher enums

I got syncs triggered for even opening a file by double clicking it in Finder
and having launched Preview.app (e.g. for JPEG files).

Merge pull request #4938 from owncloud/fix_4840

GUI: don't force the environment variable for high DPI on linux.

GUI: don't force the environment variable for high DPI on linux.

Setting the Environment variable only for owncloud makes in inconsistant with
other Qt application running at the same time.

The users can still set it themself for the whole desktop if they wish.

Addresses #4840

Merge pull request #4935 from owncloud/fix_updater_memleak

Fix memleak in updater

Fix memleak in updater

Fix link to the binary packages in README.md

[tx-robot] updated from transifex

[tx-robot] updated from transifex

QNAM: Try to workaround missing Qt patches #4720 #3888 #4051

We try this on all platforms now so we have more consistent behaviour.

Search results are filtered by server. Don't filter a second time.

Also allows searching on attributes other than displayname in ldap
configurations. Search results match web and mobile apps.

Connectivity: Reduce ETag job timeout to 60 sec #4275

Before it used the default job timeout

[tx-robot] updated from transifex

Connectivity: Delete job on timeout #4275

This was not done if there was no reply

Update changelog for 2.2.1

  Remove stray file

Merge pull request #4923 from RobinGeuze/2.2

Fix Qt4 build

Fix Qt4 build

Merge pull request #4922 from owncloud/fix_4840

Don't try to determine pixel ratio with Qt < 5.6.0

[tx-robot] updated from transifex

Don't try to determine pixel ratio with Qt < 5.6.0

This seems to be broken. Worst case users can still
set it manually

Addresses #4840

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #4913 from owncloud/normalize_name_in_filestatus

Undo regression caused by 727e73d

Undo regression caused by 727e73d

normalization to NFC is still required. Mac OS API will not take
care of that by default.

Resolves #4884

Certs: Re-ask for different cert after rejection #4898 (#4911)

Previously rejecting any kind of certificate meant that the user
was never asked again, even if the certificate changed.

Now we keep track of which certificates were rejected and ask again
if the ones mentioned in the ssl errors change.

mitmproxy is excellent for testing this.

Progress: Don't count items without propagation jobs #4856 (#4910)

* Progress: Don't count dirs without propagation jobs #4856

These directory SyncFileItems are necessary for bookkeeping
but should not influence the progress display at all.

* Progress: Skip ignored files #4856

[tx-robot] updated from transifex

Utility: Better fix for the translation of minutes, seconds, ...

The plural was not translated because of QTBUG-3945. Work around it.
Issue #4855

[tx-robot] updated from transifex

SyncEngine: cleanup setting isDirectory