Remove the last redundant void arg I left behind

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

Add a GitHub Action to easily rebase PRs

This will make it easier to have a semi-linear commit history

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

[tx-robot] updated from transifex

Merge pull request #2035 from nextcloud/add_clang_tidy_pipeline_to_drone

Add clang tidy pipeline to drone

Run clang-tidy from the 5.12-clang pipeline

This should pass, with the minimal .clang-tidy which was provided. Other
PRs should be done to enable further checks.

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

Upgrade to the latest client image

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

Merge pull request #2032 from nextcloud/insulate_testutility_from_user_settings

testLaunchOnStartup shouldn't fail on dev setup

Merge pull request #2023 from nextcloud/use_return_braced_init_list

Use the return braced init list pattern

testLaunchOnStartup shouldn't fail on dev setup

This test was failing locally for me. Indeed, through QStandardPaths it
was finding the user settings of my production client and not having the
initial state it expected. Using QStandardPaths test mode then it starts
from a clean slate every time.

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

Use the return braced init list pattern

This way we avoid repeating the return type while it is already known.

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

Merge pull request #2024 from nextcloud/use_raw_string_literals

Use raw string literals when appropriate

[tx-robot] updated from transifex

Merge pull request #2021 from nextcloud/use_default_for_trivial_ctor_and_dtor

Use = default for trivial ctors and dtors

Merge pull request #2020 from nextcloud/use_make_unique

Use make_unique when appropriate

Merge pull request #2019 from nextcloud/remove_redundant_void_argument

Remove redundant void argument list

Merge pull request #2018 from nextcloud/use_using

Use using instead of typedef

Merge pull request #2017 from nextcloud/remove_redundant_initializer

Remove redundant initializer

Use raw string literals when appropriate

This is especially nice for Windows paths.

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

Use = default for trivial ctors and dtors

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

Use make_unique when appropriate

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

Remove redundant void argument list

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

Use using instead of typedef

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

Remove redundant initializer

This is the same value than the initializer value in the header, no need
to repeat it.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2006 from nextcloud/sbeyer-improve-wizard-url-field

Improve URL input field in Wizard

Replace "https://..." hint by "https://host:port"

The new hint is a little more verbose and provides more information
than the old one.

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

Show URL placeholder hint in wizard

In owncloudsetupnocredspage.ui, the URL input field leUrl has a
placeholder text saying "https://..." which is a very useful hint
for the user.  However, in the OwncloudSetupPage constructor, the
placeholer text is overwritten by the return string of the theme's
wizardUrlHint() method.  The NextcloudTheme class does not override
this virtual method, so an empty string is used.

To make available the "https://..." hint, it is moved from the
UI file to NextcloudTheme::wizardUrlHint().  Note that, if a
theme is used which does not allow a custom server URL, the
placeholder text is now empty.  This makes sense because the
input field is disabled in that case.

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

Change vertical size policy of URL input field in wizard

The input field for the nextcloud server URL is vertically too
big (because of the icon next to it in the same horizontal layout).
This commit solves this issue by changing its vertical size policy
from Ignored to Fixed.

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

[tx-robot] updated from transifex

Merge pull request #2003 from ivaradi/debian-qt512

Do not build master for distributions without Qt 5.12 or later

[tx-robot] updated from transifex

Do not build master for distributions without Qt 5.12

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

Merge pull request #1987 from sbeyer/fix-leaks

Fix a few leaks

Merge pull request #2001 from nextcloud/sbeyer-member-tidy

Use default member initialization

[tx-robot] updated from transifex

Use default member initializers in OCC::FolderStatusModel sub-structs

This allows to remove their explicit default constructor definitions.

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

Use default member initializers in OCC::SyncJournalErrorBlacklistRecord

This allows to remove its explicit default constructor definition.

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

Use default member initializers in OCC::SyncJournalDb sub-structs

This allows to remove their explicit default constructor definitions.

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

Fix leak in OCC::AccountManager stub for FolderManTest

We simply use a static QObject using Q_GLOBAL_STATIC()
instead of allocating a leaking QObject on the heap.

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

Fix leak in Systray

The member _trayEngine, allocated in the constructor, was leaking.

This commit sets the Systray instance as the Qt object tree parent of
the _trayEngine to fix the leak.

While at it, a few unused header includes have been removed.

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

Fix leak in CloudProviderManager

The member _map, allocated in the constructor, was leaking.

The ad-hoc way to fix the leak would have been to add a destructor
that deletes _map.  However, there is no good reason why _map has
to be a pointer to a QMap (instead of a QMap).
Hence this commit changes the type of the _map member from a QMap
pointer to a QMap.

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

Fix leak in CloudProviderWrapper

The member _recentlyChanged, allocated in the constructor, was leaking.

The ad-hoc way to fix the leak would have been to add a destructor
that deletes _recentlyChanged.  However, there is no good reason why
_recentlyChanged has to be a pointer to a QList (instead of a QList).
Hence this commit changes the type of the _recentlyChanged member
from a QList pointer to a QList.

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

Use default member initializers in OCC::FolderDefinition

Using default member initializers (available since C++11) in
OCC::FolderDefinition allows to remove its explicit default
constructor, which is favorable.

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

Remove unused member in OCC::FolderDefinition

OCC::FolderDefinition has a member called isClientSideEncrypted
that was introduced by commit 9318c487b9 but has not been used
in any way since.

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

Don't use int literals for bools

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

[tx-robot] updated from transifex

Merge pull request #1990 from nextcloud/use_auto_to_avoid_duplicated_type_names

Use auto to avoiding repeating type names

Fix spacing

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

Use auto to avoiding repeating type names

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

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>