added owncloudcmd bandwidth limit parameter (#5707)

Merge remote-tracking branch 'origin/avatar_pics'

Also fix compilation because of 22370fdbdadd06f0cacd249a8d7a32f0d3c1374e

Pull request #5482

Merge remote-tracking branch 'origin/2.3'

 Conflicts:
VERSION.cmake

AccountSettings: Sync with clean discovery on Ctrl-F6 #5666

AccountSettings: Easier access to selected folder alias

owncloudcmd: return code depend on sync result

Issue #3936

[tx-robot] updated from transifex

Avatar: macOS support

[tx-robot] updated from transifex

ShareDialog: Reenable user input textedit on error #5694

[tx-robot] updated from transifex

Wizard: Improve folder creation (2)

[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

Wizard: Improve folder creation

[tx-robot] updated from transifex

Doc: Update to new sync journal db name #5662

Transfers: Show a decimal for single digit MB rates #4428

ShareDialog: Hide the detailed permissions if there is only one #5655

ShareDialog: Margin and spacing refinements #5627

ShareDialog: Fix thumbnail size #5654

owncloudcmd docs: Mention required sync-exclude.lst #5679

Improve http error messages; cleanup

By default QNetworkReply::errorString() often produces messages like
   "Error downloading <url> - server replied: <reason>"
but the "downloading" part invariably confuses people since the
error might very well have been produced by a PUT request.

This commit produces clearer error messages for HTTP errors.

Additionally:
* Remove some unnecessary null checks from slots connected to
  network job signals and document that these signals never send
  null replies.
* There was a bug where AbstractNetworkJob::_timedout wasn't
  set when derived classes overrode slotTimeout. We now ensure
  it's always set by disallowing overrides of slotTimeout.
  Instead it now calls onTimedOut, which allows custom handling.
* Several subclasses declared errorString, isTimedOut. Move
  these to AbstractNetworkJob.
* Unify handling of OC-ErrorString (via the new, general
  Job::errorString)
* Add documentation in various places.

[tx-robot] updated from transifex

Fixing a duplicate in the build instructions

- The duplicate and some typos were introduced in 42aed56a5a92a32b86e2bdd1c6e9d75e27728c8b
- Also fixes some minor format issues. There's a lot of notes in this docs, we should refactor some.

[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

Chunking NG: Also send OC-Total-Length on MOVE

For https://github.com/owncloud/core/issues/26988

complete url when using themed wizardUrlPostfix

Before we would only open the part of the url that the users entered.
Now if the wizardUrlPostfix is used this is should be appended to that
when opening the browser on the result page.

Removed the coverity badge

Following the discussion on https://github.com/owncloud/client/commit/da45d99b8fed9a37df8031433e11eb574c28a8df
Also updated the link to the current building instructions.

[tx-robot] updated from transifex

PropagateDownload: fix possible crash

Backtrace looks like this:
Crash: EXCEPTION_ACCESS_VIOLATION_READ at 0x0
  File "propagatedownload.cpp", line 234, in OCC::GETFileJob::slotReadyRead
  File "moc_propagatedownl_CA5CFSHZDTX34X.cpp", line 86, in OCC::GETFileJob::qt_static_metacall
  File "qobject.cpp", line 495, in QMetaCallEvent::placeMetaCall
  File "qobject.cpp", line 1256, in QObject::event
  File "qapplication.cpp", line 3804, in QApplicationPrivate::notify_helper

GETFileJob::slotReadyRead can be called with a QueuedConnection when the
bendwith manager is involved. In that case, if the QNAM was reset
in between, the reply might have been destroyed.
(This is only speculation based on the backtrace)

AccountState: Attempt to fix a crash

Backtrace from the crash reporter:

Crash: EXCEPTION_ACCESS_VIOLATION_READ at 0x21
  File "qcoreapplication.cpp", line 1281, in QCoreApplication::postEvent
  File "qobject.cpp", line 2125, in QObject::deleteLater
  File "connectionvalidator.cpp", line 240, in OCC::ConnectionValidator::reportResult
  File "connectionvalidator.cpp", line 206, in OCC::ConnectionValidator::slotAuthFailed
  File "moc_connectionvalidator.cpp", line 127, in OCC::ConnectionValidator::qt_static_metacall
  File "qobject.cpp", line 3716, in QMetaObject::activate
  File "moc_networkjobs.cpp", line 653, in OCC::PropfindJob::finishedWithError
  File "networkjobs.cpp", line 570, in OCC::PropfindJob::finished

I believe the problem is caused because 'this' was deleted in ConnectionValidator::reportResult
as the signal connectionResult gets emited. The AccountState::slotConnectionValidatorResult
slot does indeed call slotInvalidCredentials which might call {Shibboleth,Http}Credentials::fetchFromKeychain
which might emit fetched directly, which will call AccountState::slotCredentialsFetched
which deletes the _connectionValidator

So use deleteLater when deleting the _connectionValidator, hoping this helps

Merge branch 'dynamic_chunking'

dynamic chunking: cleanup, fixes, improvements

* make target duration a client option instead of a capability
* simplify algorithm for determining chunk size significantly
* preserve chunk size for the whole propagation, not just per upload
* move options to SyncOptions to avoid depending on ConfigFile
  in the propagator
* move chunk-size adjustment to after a chunk finishes, not when
  a new chunk starts

[tx-robot] updated from transifex

dynamic chunking prototype

[tx-robot] updated from transifex

VERSION: 2.3.1 was release, this branch is now for 2.3.2

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Update CMakeLists.txt

remove doc-man target in doc/CMakeLists.txt https://github.com/owncloud/client/issues/2153

Documented the usage of environment variables.

In particular the `OWNCLOUD_CHUNK_SIZE` as requested per #2449.
I looked through the code and it seems the env vars override the conf file, but I'm not 100 % sure.
The `OWNCLOUD_CRITICAL_FREE_SPACE_BYTES` and `OWNCLOUD_FREE_SPACE_BYTES` are now mentioned in two places.

ShareDialog: Make "can edit" partially checked sometimes #5642

Previously the check box was checked if *any* of its sub-permissions
were granted. This can hide the fact that only a limited subset of them
are actually granted.

The new behavior is to display as "partially checked" if only some
of the sub-permissions are available. Clicking the check box itself
still toggles between granting all or none of them.

[tx-robot] updated from transifex

Update README.md

Fix Jenkins badges

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Wizards: Never propose an existing folder for syncing #5597

Wizard: Remove unused oldLocalFolder property

owncloudsetuppage.ui: Remove that file, it is unused

[tx-robot] updated from transifex

Fixing a typo on an issue reference
(cherry picked from commit ed03cdf45a696161c4a4f94eaa9eda80559b36d2)

Fix incorrect documentation for the ownCloud command-line client

This fixes #5609.

Including some info on the proxy config. in the docs.

This was a request of @michaelstingl to document how to configure the proxy manually via the .cfg file for config provisioning purposes.
- Restructured the bullet points into 3 different tables; I believe it's a cleaner approach; maybe this is also applicable to other sections.
- Would be logical to include also settings on proxy auth?

Folder::showSyncResultPopup: Fix undefined behavior when there is no errors

When there is no errors  _syncResult.firstItemError() is NULL, and accessing
it's _file member is an undefined behavior. (Thankfully, createGuiLog did not
use the string when the count was 0, but we are not supposed to create
null references.

Found with the UB sanitizer:

src/gui/folder.cpp:348:49: runtime error: member access within null pointer of type 'OCC::SyncFileItem'
src/gui/folder.cpp:348:19: runtime error: reference binding to null pointer of type 'const QString'

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge remote-tracking branch 'origin/2.3'

 Conflicts:
src/libsync/owncloudpropagator.h

ActivityListModel: attempt to fix a crash.

The backtrace seems to indicate that the account is invalid.
I don't know how this can happen, maybe the account's display
name was changed while the app is running?

Backtrace:
Crash: EXC_BAD_ACCESS / KERN_INVALID_ADDRESS at 0x18
  Module "owncloud", in OCC::AccountState::account
  Module "owncloud", in OCC::ActivityListModel::data
  Module "owncloud", in OCC::ActivityItemDelegate::paint
  Module "QtWidgets", in QListView::paintEvent
  Module "QtWidgets", in QWidget::event
  Module "QtWidgets", in QFrame::event
  Module "QtWidgets", in QAbstractScrollArea::viewportEvent
  Module "QtWidgets", in QAbstractItemView::viewportEvent
  Module "QtWidgets", in QAbstractScrollAreaFilter::eventFilter
  Module "QtCore", in QCoreApplicationPrivate::sendThroughObjectEventFilters

Propagator: change order of destruction

The destructor of the PropagateItemJob will access the propagator's
_activeJobList. So the _rootJob needs to be destroyed before it.
Order of destruction is the reverse of the order of the members in
the class. So put it at the end so it can be destroyed first.

(This made TestSyncEngine::testDirDownloadWithError crash sometimes
in the master branch)

ChangeLog for 2.3.1

Merge remote-tracking branch 'origin/2.3'

 Conflicts:
src/libsync/owncloudpropagator.cpp

About: Show the runtime versions of Qt and OpenSSL #5592

I didn't want to touch this visible translated string in a
patch release so I just switched the data.

For Qt versions before 5, we continue to show the build-time version
of OpenSSL but it's clearly marked as such.

SyncEngine: fix test regression in TestChunkingNG::testCreateConflictWhileSyncing

In 8ef11a38c99c6a33b175f1ad792ec02b52f88d25, we started blacklisting
SoftError for 0 seconds.  But if the two sync happen with less than
1s interval, we would still prevent them to happen.
So make sure we expire if 0 seconds have expired

[tx-robot] updated from transifex

Attempt to fix a crash in FolderStatusModel::slotUpdateDirectories

The backtrace looks like:

  File "atomic_base.h", line 396, in QString::~QString
  File "qlist.h", line 442, in OCC::FolderStatusModel::slotUpdateDirectories

This is the only QList operation, and it may crash if the list is empty.
It can be empty if the propfind returned empty results.
I'm not sure how this can be possible to have an empty list there since
the server is always supposed to return at least one entry, for the directory
itself. But it can happen if a directory was transformed in a file, or
if there is a bug on the server.

HttpCreds: Update app passwords url #5605

See also owncloud/core#27360

Account server version: Helper to create versions

Hex literals don't work well with version 10: 0x100000 doesn't do
the right thing.

Ignore list editor: adjust the size

The previous patch on this file made it a bit too big, so make it a bit smaller
For issue #5600

Using different window flags on the connection method dialog (#5614)

This is a possible fix for #3850, applying custom window flags (http://doc.qt.io/qt-5.8/qt.html#WindowType-enum) on the connection method dialog. These avoid resizing the dialog and displaying a maximize button on OS X and context help button on Windows.

[tx-robot] updated from transifex

Add more versions to --version #5592

* Add git revision, if available
* Add SSL runtime version
* Add compile-time Qt version

Windows/NTFS: Do not attempt to upload inaccessible files #5544

It is possible to create files with filenames that differ
only by case in NTFS, but most operations such as stat and
open only target one of these by default.

When that happens, we want to avoid uploading incorrect data
and give up on the file.

Typically this situation should never occurr during normal use
of Windows. It can happen, however, when a NTFS partition is
mounted in another OS.

[tx-robot] updated from transifex

Ignore list editor: adjust the size of the columns in the table

So the title of the column is fully visible.
Adjust the default size and the size policy of the label so it behave
correctly as the window get resized

Issue #5600

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Blacklist: Escalate repeated SoftError to NormalError #5500

Merge pull request #5585 from owncloud/add-replace-sync-directory-to-faq

Add explanation of how to replace local sync folder to FAQ

Revise the point about starting a clean sync

On review, after feedback from @phil-davis, he's right that users need
to be fully briefed as to the implications of choosing a clean sync —
especially when they have a large amount of data in their account.

[tx-robot] updated from transifex

Avatar: Use QImage instead of QPixmap to avoid dep on QApplication.

That fixes the test suite.

Propagator: Attempt to fix a crash in OwncloudPropagator::scheduleNextJob

The crash reporter shows many crashes in OwncloudPropagator::scheduleNextJob.
We don't really know what could be the cause, but it's probably because
the _activeJobList contains dangling pointer.
So this patch makes sure to remove all the jobs from this list as they get
destroyed.

Fix TestSyncEngine::testDirDownloadWithError

On master, the scheduling is different and the tasks abort in another
order

Merge remote-tracking branch 'origin/2.3'

 Conflicts:
src/libsync/owncloudpropagator.cpp

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Add explanation of how to replace local sync folder to FAQ

This fixes owncloud/documentation/issues/2923.