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.

Update README.md

Fix Jenkins badges

Wizards: Never propose an existing folder for syncing #5597

Wizard: Remove unused oldLocalFolder property

owncloudsetuppage.ui: Remove that file, it is unused

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

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.

Remove unreachable code #5582

AbstractNetworkJob: Improve redirect handling #5555

* For requests:
  - reuse the original QNetworkRequest, so headers and attributes
    are the same as in the original request
  - determine the original http method from the reply and the request
    attributes
  - keep the original request body around such that it can be sent
    again in case the request is redirected

* Simplify the interface that is used for creating new requests in
  AbstractNetworkJob.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

ActivityListModel: fix possible crash in slotActivitiesReceived

Backtrace from the crash reporter:

Crash: EXCEPTION_ACCESS_VIOLATION_READ at 0x401
  File "moc_activitylistmo_M5OEXJ7XGJYTWT.cpp", line 92, in OCC::ActivityListModel::qt_static_metacall
  File "qobject.cpp", line 3730, in QMetaObject::activate
  File "moc_networkjobs_7AMNCW4BBANVRK.cpp", line 1342, in OCC::JsonApiJob::jsonReceived
  File "networkjobs.cpp", line 714, in OCC::JsonApiJob::finished
  File "abstractnetworkjob.cpp", line 207, in OCC::AbstractNetworkJob::slotFinished
  File "moc_abstractnetwor_PFI2TXGQHRE33H.cpp", line 98, in OCC::AbstractNetworkJob::qt_static_metacall
  File "qobject.cpp", line 3730, in QMetaObject::activate
  File "moc_qnetworkreply.cpp", line 367, in QNetworkReply::finished
  File "qnetworkreplyhttpimpl.cpp", line 2100, in QNetworkReplyHttpImplPrivate::finished
  File "qnetworkreplyhttpimpl.cpp", line 279, in QNetworkReplyHttpImpl::abort

My theory is that the AccountState stored in a property of the job was destroyed.
before the job timed out.
Therefore, the qobject_cast within the qvariant_cast would call the metaObject()
virtual function on a dangling pointer.

Fix it by storing a QPointer instead which will track the deletion.

Propagator: Fix finished signal of directory being emited twice and causing crash

When there is a FatalError, we ended up emiting the finished signal for
the directory job several times, which would lead to crashes

Issue #5578

NSIS: Don't display link on finish page

[tx-robot] updated from transifex

[tx-robot] updated from transifex

VERSION: master branch is 2.4

Merge branch '2.3'

 Conflicts:
src/libsync/owncloudpropagator.cpp

2.3.0 was released: Update version

SslButton: Remove additional ')'

Regression since 7bfe0613827aa7cee93a77ccbedab9b07ea9ca78
Issue: #5573

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Update client version from 2.2 to 2.3 (#5568)

As the latest version of 2.3, the version information needs to be
updated as well.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Potential fix for crash in PropagateDirectory::abort (#5564)

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Don't abort propagation job abortions synchronously from finished

This leads to crashes since we changed the connection to the parent
jobs not to be queued anymore.
We don't really need to bubble up the finished state through
parents in that case, and it would also mean that we'd recurse
all the way through leaves as we go up to each parent. So just call
abort directly on the OwncloudPropagator and make sure the abortion
call is posted to the event loop.

Reduce the connection data used by PropagateDirectory objects

Avoid using connections to report up the job tree for signals
that we can directly communicate to the OwncloudPropagator.
This slightly reduces the memory usage and avoid passing those calls
through the whole parent chain.

Create PropagateItemJobs only before starting them

Split running jobs into a separate vector

Split the subjob logic out of PropagateDirectory

Replace WaitForFinishedInParentDirectory with WaitForFinished for directory moves

In preparation for the PropagateDirectory refactoring, simplify things
by removing WaitForFinishedInParentDirectory, which is currently
implemented as a one-level check.

This value is important for directory items, but is however never
used since a directory CSYNC_INSTRUCTION_RENAME item will always be in
PropagateDirectory::_firstJob, which will have to pass through its own
PropagateDirectory job's parallelism() before reaching the parent's
_subJobs optimization.
Since PropagateDirectory::parallelism can only return WaitForFinished
or FullParallelism, that value is lost. So this commit doesn't
change the behavior for directories, and allow file renames to be
scheduled in parallel across directories (which isn't a problem).

Fix TestSyncEngine::abortAfterFailedMkdir (again)

It could be possible that _firstJob is marked as finished if
aborted before its parent PropagateDirectory was marked as finished,
allowing a posted scheduleNextJob call to schedule the child job
in-between.

Verify that all strings are properly escaped (#5558)

- I checked every occurence of a '%2' and make correct use of the
QString::arg overload that takes several argument instead of chaining
them, because the first argument can contains a '%1'

 - I tried to look for every label that they either use plain text or richtext
and escape the user provided strings in there.

[tx-robot] updated from transifex

Wizard: Handle PROPFIND redirects #5553

By default, followRedirects is true for all requests, to transparently
handle redirections. In the wizard, we have special redirect-handling
code though and that was being skipped.

Setting the flag to false allows the wizard to be aware of redirects
and to handle them in the correct way. Tested with the server described
in
https://github.com/owncloud/administration/tree/master/redirectServer

There's a second bug here, where followRedirects always converts
redirected requests to the GET verb. That means redirected PROPFINDs
will never have worked. This change un-breaks them for the wizard only.
There should be no case that previously worked that stops working now.

[tx-robot] updated from transifex

Merge pull request #5549 from owncloud/fix-documentation-build-process

Fix documentation build process

Merge branch 'master' into fix-documentation-build-process

Re-add the Makefile for controlling the build process

I'm not sure, why the file was removed, it was there in 2013, but it was
removed. This commit adds it back so that the build process can be
automated in a UNIX environment.

Update ChangeLog

Fix the Sphinx-doc build configuration

I'm not sure what the configuration/setup has been like to date. But as
it stood, it didn't build successfully. Given that, I've created a copy
of doc/conf.py, included in this commit, from the default doc/conf.py.in
and set the appropriate variables as necessary.

NOTE: The build didn't work when the version, release, and latex_logo
values weren't set, either directly or with interpolatable values.

Merge pull request #5547 from owncloud/update-docs-build-configuration

Create a build configuration file

Merge pull request #5545 from owncloud/grammar-updates

Fix grammar and spelling errors in the documentation

[tx-robot] updated from transifex

Message Boxes: Force to be more on top #5503 (#5536)

Create a build configuration file

This is no different from the default file, conf.py.in, except for the
specification of the logo file.

Fixed grammar and spelling errors in the documentation

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Propagator: Fix t8.pl

The test sets OWNCLOUD_MAX_PARALLEL to 1 to disable parallelism.
But since the max amount of parallelism is twice as much, that does not
work.
So change the way we compute the hardMaximumActiveJob: Use the value of
OWNCLOUD_MAX_PARALLEL to maximize this amount and base the maximum amount
of transfer jobs on it instead of the other way.

A result of this change is that, in case of bandwidth limit, we keep the
default of 6 non-transfer jobs in parallel. I believe that's fine since
the short jobs do not really use bandwidth, so we can still keep the same
amount of small jobs.

Revert "Don't blacklist 507 Insufficent Storage #5346 (#5424)"

This reverts commit e1f5a49c21826590811710f73e37b7b72e256960.

Retrying uploads with insufficent storage errors frequently leads to
high server traffic. See #5537 for links and a sketch of a correct
solution.

Revert "Fix TestSyncEngine::abortAfterFailedMkdir"

This reverts commit 1cec2ca13dc35aba6aaf79f42bde9a6dd00143a7
but keeps the reset() -> take() part that is probably the cause
of crashes we've seen in the crash reporter.

[tx-robot] updated from transifex

AccessManager: don't set an invalid network configuration on linux (#5538)

If we call
 setConfiguration(QNetworkConfiguration());
This sets an invalid configuration on the QNAM.
But later, when we really go online because interfaces are discovered,
QNetworkAccessManagerPrivate::_q_onlineStateChanged is called (with isOnline=true).
And this will set the state to disconnected because customNetworkConfiguration is
true, and the networkConfiguration state is disabled.

The workaround we to fix another bug on Windows in which the default network
configuration was not behaving properly.

The issue on linux is hard to reproduce and only happen in some condition,
but it was reproduced on smashbox when they run two owncloudcmd at the same time.

Issues: #4720 , #3600

Recent Changes: Fix duplicates #5256

The transmissionProgress emission when the whole sync finishes
shouldn't look exactly like the one for the last file.

[tx-robot] updated from transifex

Folder: Keep files option after aboutToRemoveAllFiles should not clear selective sync list (#5531)

We were removing the wholme journal db when the user wanted to keep all files,
But that would also remove the selective sync lists.
We should only remove the metadata table.

Issue #5484

Update windows docker file to use 2.3 OBS repo

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

[tx-robot] updated from transifex

Merge remote-tracking branch 'origin/2.3'

Disable tests uncompatible with Qt4 for that build

Fix the UNIT_TESTING build on Windows

- Put all tests in the bin directory so that DLLs can be loaded
- Add missing exports
- Skip tests that use code depending on zlib
- The "GMT" timezone is named differently, use the int constructor instead

5 tests are still failing, it's not really worth fixing at the moment
since no developper is currently using Windows as its main platform.

Fix FolderManTest on macOS

On macOS /var is a symlink to /private/var and we have to make sure that we
use the canonical path before and after it enters the code to make sure we
compare paths correctly.