Merge pull request #2063 from nextcloud/fix/mac-bundleinfo-dark-mode-optin

macOS: Opt In to Dark Mode for building against older macOS SDKs < 10.14

macOS: Opt In to Dark Mode for building against older macOS SDKs < 10.14

Required for our build server that uses an older SDK to support previous macOS versions.

See: https://developer.apple.com/documentation/appkit/nsappearancecustomization/choosing_a_specific_appearance_for_your_macos_app

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

Merge pull request #1749 from nextcloud/fix/mac-2.7-qml-tray-build

macOS: Use macdeployqt instead of the old admin/osx/macdeployqt.py

macOS: Build nextcloudcmd after gui and run macdeployqt with the -executable option

- src/CMakeLists.txt: Switch build order to build cmd before gui
- src/gui/CMakeLists.txt: Use the -executable option for a combined run of macdeployqt

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

Copy nextcloudcmd post-build instead of macdeployqt when building macOS bundle

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

macOS: Use macdeployqt instead of the old admin/osx/macdeployqt.py

In order to build the new 2.7 releases (new QML dependencies) without manually
modifying the old Python script each time, rely on Qt's standard tool instead.

This should streamline the build systems for upcoming dependency changes.

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

Merge pull request #2061 from nextcloud/enh/version-display-cmd

Improve command line version info and help display (gui & cmd)

Improve command line version info and help display to streamline gui and cmd

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

[tx-robot] updated from transifex

Merge pull request #2060 from nextcloud/fix/qt-5.15.0-build-error

Fix Qt 5.15.0 build error (missing include)

Fix Qt 5.15.0 build error (missing include)

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

[tx-robot] updated from transifex

Rename rebase.yaml to rebase.yml for consistency

YAML files here use the .yml extension, not the .yaml extension.
Hence the GitHub workflow file is renamed.

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

[tx-robot] updated from transifex

Add some more make_unique calls

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

[tx-robot] updated from transifex

Cleanup the missing braced init list returns

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

Add install instructions for nextcloudcmd

Signed-off-by: shile <13051960278@163.com>

Use default member init when applicable

This also fixes a couple of warnings at places (out of order init for
instance) and a potential bug in the webflow credentials / qtkeychain
integration.

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

Add missing auto opportunities

Somehow forgot to run it on the updater code

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

Merge pull request #2047 from nextcloud/clang-cppcoreguidelines-init-var

Do not declare local variables without an initial value.

Do not declare local variables without an initial value.

Signed-off-by: Camila San <hello@camila.codes>

[tx-robot] updated from transifex

Merge pull request #1257 from ivan-cukic/ivan/icon-for-unencrypted-subdir

Show a broken lock icon for unencrypted subdirectory

Show broken encryption icon only in non-empty folders

This should address Tobias' concerns regarding the icon being
misleading. Now we basically do the following inside an encrypted folder
parent:
 * encrypted folders get the encrypted icon;
 * non-encrypted empty folders get the regular folder icon;
 * non-encrypted non-empty folders get the broken encryption icon.

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

Show a broken lock icon for unencrypted subdirectory

The E2E application allows creating unencrypted subdirectories
in an encrypted parent. This is a big privacy problem.

This patch shows a red broken lock icon for these subdirectories
in the NC client UI.

Signed-off-by: Ivan Čukić <ivan.cukic@kde.org>

Merge pull request #1235 from ivan-cukic/ivan/fixing-leaks

Fixing memory leaks in the encryption module

Introduced RAII classes for other OpenSSL resources

Fixing memory leaks in the encryption module

Due to usage of early-returns, combined with malloc/free,
several buffers that get allocated are leaked when an error
occurs.

Several functions had potential leaks:

 - `encryptStringSymmetric` leaked `ctext`
 - `EncryptionHelper::fileDecryption` leaked `out`
 - `EncryptionHelper::fileEncryption` leaked `out`

Most of the functions had leaks of the cypher context.

This patch uses `QByteArray` as the handler for the dynamically
allocated buffers for openssl to operate on. This also removes
the need for conversions from malloc'd buffers to `QByteArray`
variables previously present in the code.

It also introduces a `CypherCtx` thin wrapper class to provide
a leak-free handling of `EVP_CIPHER_CTX`.

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2051 from nextcloud/remove-submodule-binary

Remove Submodule binary (old upstream binaries)

Remove Submodule binary (old upstream binaries)

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #1940 from nextcloud/traywindow-listview-actions

Implemented share functionality in tray window and changed buttons

ShareDialog: Pointer cosmetics

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

Merge branch 'master' into traywindow-listview-actions

ShareDialog: Add further checks for the _manager member ptr

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

ShareDialog: Fix crash, properly init member pointers

_manager ptr only got initialized in the constructor when sharingPossible was set to true

Changes:
- Move member ptr inits from constructor to the header file's class definition
- Init _manager: set to nullptr too
- Add check for _manager to avoid crash in showSharingUi()

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

[tx-robot] updated from transifex

Try to fix the rebase action

I think I got the if clause wrong regarding limiting the command only to
member users. Let's move this out of the way for now.

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

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>

Get rid of redundant parentheses

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>

Merge remote-tracking branch 'origin/master' into traywindow-listview-actions

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.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