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>

[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