Merge pull request #1989 from nextcloud/remove_redundant_virtual

Remove useless virtual, better use override only

Merge pull request #1988 from nextcloud/nullptr_everywhere

Use nullptr when appropriate

Merge pull request #1992 from NeroBurner/patch-1

Update github links to desktop repository

Use the default parameter: QUrl::path(QUrl::PrettyDecoded = 0x000000)

Change it to path() to be consistent with the other usages in the project :)

See: https://doc.qt.io/qt-5/qurl.html#path

Signed-off-by: Michael Schuster <michael@schuster.ms>

Use nullptr when appropriate

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #1986 from nextcloud/properly_handle_denormalized_href_path

Properly handle denormalized href

Update github links to desktop repository

Update the issue and open PR links with the new repository URL.
The old URL is forwarded, but feels not completely right.

Signed-off-by: Reinhold Gschweicher <pyro4hell@gmail.com>

Merge pull request #1994 from nextcloud/cleanup_stale_files

Remove now unused Qt patch files

Merge pull request #1991 from sbeyer/improve-sanitizer-support

Improve sanitizer support

Remove now unused files

Those files all refer to Qt version older than 5.12, also they seemed
unused for a while now.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #1979 from nextcloud/bump_qt_dependency_to_5_12

Stop supporting Qt older than 5.12

Add -fno-sanitize=vptr for SANITIZE_UNDEFINED=ON

The UndefinedBehaviorSanitizer includes the "vptr" check.  This
check, however, needs typeinfo for OCC::AccountManager because
otherwise its stub for FileManTest leads to undefined references
when linking.  Adding the -frtti flag to enable run-time typeinfo
did not solve the problem.  I do not know another solution, so this
commit disables the vptr check.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

Make sanitizer options more fine-grained

The SANITIZE_ADDRESS option of our CMake configuration activates the
AddressSanitizer (and UBSan in a non-working way) for the whole project
(although, by the way, its documentation pretends that it is only enabled
for tests).

This commit introduces new options SANITIZE_LEAK, SANITIZE_MEMORY,
SANITIZE_UNDEFINED, SANITIZE_THREAD.  Each of these options (including
SANITIZE_ADDRESS) enables only the corresponding sanitizer.

Moreover, we mark all sanitizer options as advanced options, because these
options are only interesting for developers.

Note that some sanitizers are conflicting, that is, not all options can
be enabled simultaneously.  Also, not all sanitizers are available for
all compilers and versions.  We, however, do not check for this, instead
we let the compiler throw its errors in such cases.

The explicit usage of the Google Linker is removed, because it is not
necessary and can lead to problems with clang.

The commit can be considered a rewrite of cmake/modules/SanitizerFlags.cmake.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

[tx-robot] updated from transifex

Remove useless virtual, better use override only

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Properly handle denormalized href

In case of denormalized paths in the dav href (presence of . or .. in
the path) simple string startsWith comparison wasn't enough to know if
said href ended up in the right namespace. That's why we're now using
QUrl (pretending local file since we don't have a full URL in the href)
to normalize the path before comparison.

This could happen with broken proxies for instance where we would
wrongly validate the dav information resulting in potentially surprising
syncing and name collisions.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Stop supporting Qt older than 5.12

This both removes older Qt from the CI and also adjust all the
find_package calls in CMakeLists.txt

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #1974 from nextcloud/dont_silently_kill_debug

Don't silently kill debug messages

Merge pull request #1972 from nextcloud/fix_fileutil_leak

Fix leak in FileUtil::IsChildFile

Don't silently kill debug messages

The message handler globally installed by the logger silently drop
messages if the logger is not logging. On top of it, it doesn't log
debug messages by default.

Anything not logged is currently silently discarded. This can come as a
surprise to a developer trying to contribute for the first time and
adding some debug message for some reason.

We're thus trying to strike a middle ground which is that debug messages
get a regular output if the logger isn't interested in them.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Fix leak in FileUtil::IsChildFile

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #1965 from ivaradi/drop-xenial

Do not build for Xenial anymore

Do not build for Xenial anymore.

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

Merge pull request #1961 from nextcloud/tray_window_is_dialog

Indicate this is a dialog and not a regular window

Indicate this is a dialog and not a regular window

This is necessary with some window managers which would otherwise
consider they can reposition and resize the tray window as they wish
(yes, even though the user can't do it directly).

This is for instance the case if you use the tiling mode scripts of
KWin. It automatically ignores dialogs but might force windows to be
fullscreen (which would badly break the layout in our case).

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #1962 from nextcloud/fix_svg_icon_loading

Force the default QQC2 style

[tx-robot] updated from transifex

Force the default QQC2 style

This works around a bug in KDE's qqc2-desktop-style which breaks buttons
with icons not based on a name. By forcing a style name the KDE Plasma
platformtheme plugin won't try to force qqc2-desktops-style anymore.

Can be removed once the bug in qqc2-desktop-style is gone.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Merge pull request #1963 from nextcloud/drone-remove-qt-5.7-and-5.8

Drone: Remove CI builds for Qt 5.7 and 5.8

Drone: Remove CI builds for Qt 5.7 and 5.8

Housekeeping and the new 2.7 Tray builds need at least Qt 5.9

Signed-off-by: Michael Schuster <michael@schuster.ms>

Merge pull request #1957 from nextcloud/backport/1954/master

[master] Fix typos.

Merge pull request #1949 from veddox/master

Resolve symlinks when checking for registered paths.

Fix typos.

Debian has a list of known typos in their checking tools.

Signed-off-by: Sandro Knauß <hefee@debian.org>

Merge pull request #1955 from ivaradi/debian-cleanup

Remove unnecessary Debian build files

Merge pull request #1956 from nextcloud/update-changelog

Update ChangeLog with 2.6 series.

Update ChangeLog with 2.6 series.

Remove unnecessary Debian build files

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Fixed symlink resolution for menu items.

Signed-off-by: Daniel Vedder <d.vedder@web.de>

Resolve symlinks when checking for registered paths.

Fixes issue #1948: https://github.com/nextcloud/desktop/issues/1948
The client already resolves symlinks when adding new paths to the
sync-list, but the Nautilus plugin did not do this yet, causing it
to not recognise synced folders if they were accessed via a symlink.

Signed-off-by: Daniel Vedder <d.vedder@web.de>

[tx-robot] updated from transifex

Merge pull request #1896 from ivaradi/debian-new

Reorganize Debian build

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Add Groovy build

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

Allow specifying the base commit and version in the configuration file

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

Change Debian build to the new more Debian-y way

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

Merge pull request #1895 from ivaradi/debian-version

Debian versioning updates

[tx-robot] updated from transifex

Merge pull request #1943 from ivaradi/groovy

Add Groovy-specific files

Add Groovy-specific files

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

Merge pull request #1942 from ivaradi/disco-groovy

Remove Ubuntu Disco, add Groovy

Remove Ubuntu Disco, add Groovy

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

[tx-robot] updated from transifex

Merge pull request #1884 from nicolasfella/menuaction

[dolphin] Use menuAction() instead of manual creation

Merge pull request #1919 from taaem/fix-dbus-activation

CloudProviders: Use absolute path in dbus file

Merge pull request #1922 from taaem/fix-recent-files

Fix cloudprovider crashing - don't clear the _recentMenu pointer

Merge branch 'master' into menuaction

Merge pull request #1937 from ivaradi/xenial-compat

Qt 5.5 compatibility #ifdef

Merge pull request #1893 from nicolasfella/override

override specifier missing

Merge pull request #1887 from nicolasfella/colon

[dolphin] Fix overlays when filename has a colon

Merge pull request #1889 from nicolasfella/applicationengine

Use QQmlApplicationEngine instead of QQmlEngine

[tx-robot] updated from transifex

Qt 5.5 compatibility #ifdef

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #1933 from nicolasfella/fixdolphin

Fix build of Dolphin integration

Fix build of Dolphin integration

config.h needs to be included in the header already since we use APPLICATION_ICON_NAME there. This got introduced in 6fc877577c84e2ed3b175ee1425255e52d1d37b4

Signed-off-by: Nicolas Fella <nicolas.fella@gmx.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[dolphin] Use CMake env vars for App Name and Icon strings

Signed-off-by: Michael Schuster <michael@schuster.ms>

Merge pull request #1883 from nicolasfella/contexticon

[dolphin] Add icon to context menu

Merge pull request #1882 from nicolasfella/detach

Fix container detaching

Merge pull request #1881 from nicolasfella/envempty

Use qEnvironmentVariableIsEmpty

Merge pull request #1880 from nicolasfella/timeout

Remove timeout parameter from showMessage

Merge pull request #1879 from nicolasfella/desktopentry

Add desktop-entry hint to notifications

[tx-robot] updated from transifex

Use LINUX_APPLICATION_ID

Signed-off-by: Nicolas Fella <nicolas.fella@gmx.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

CloudProviders: Use absolute path in dbus file

CloudProviders: Don't clear the _recentMenu pointer

The _recentMenu pointer ref should not be freed, when building the
menu.
This fixes #1920 and #1818

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex