Use properties to get user details

Convert imperative QML code to declarative code using property bindings

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

Port from context properties to singletons

Context properties are deprecated and will be removed in Qt 6.
QML singletons are the better option here, especially given that
UserModel and UserAppsModel already are singletons.

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

[tx-robot] updated from transifex

Create HeaderButton QML component

Deduplicate the Button code by moving it into a separate file.

Also use the same background code for the apps button since as far as I can tell it didn't behave any different

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

Merge pull request #1939 from nextcloud/fix-tray-multiscreen

Major multi monitor improvements and rewrite of tray window positioning

Add some debug output for the window placement

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

Fix crash caused by setting Tray window screen pointer on the QML side

Seems like Qt sometimes doesn't like the QML window's screen property to be set
to a C++ (QVariant) pointer value, so we use the index: Qt.application.screens[]

See Qt docs: https://doc.qt.io/qt-5/qml-qtquick-window-window.html#screen-prop

This fix returns the matching window's index from the QGuiApplication::screens()
list to the QML side, instead of the window pointer.

Steps to reproduce the crash with the previous code:
- Open the Tray menu and close it a few times,
  or scroll randomly up and down in its activity list.

Tested with Qt 5.12.5

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

Remove Submodule qtmacgoodies again

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

Try make taskbar position heuristic more readable

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

Make the reference point independent of the window

This leads to simplifying the computation code quite a bit as well.
Indeed we're separating concern between what is window size dependent or
not and that shows.

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

Get rid of screenIndex

This was leading to the same logic being duplicated several times. It's
fine to return the QScreen* on the QML side directly but wrapped into a
QVariant.

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

Move all the position computation on the C++ side

The API is just more convenient there, the rect and point types on the
QML side are just pale shadow of their C++ counterparts.

Also improved a bit the constness of the Systray class.

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

Rename tbOrientation into taskbarOrientation

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

Introduce a screenRect as coordinate space

The top left corner of that screenRect is (0,0) or based on
(virtualX,virtualY) depending on the platform. This leads to easier
reduction in code duplication.

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

Use TaskBarPosition enum on the QML side

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

More simplifications, also transfer of more tray positioning logic to C++ backend.

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

Major multi monitor improvements and rewrite of tray window positioning

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2077 from nextcloud/nullptr-everywhere-comparison

Simplify nullptr comparisons where appropriate

Simplify nullptr comparisons where appropriate

Make the codebase consistent, we already have a lot of implicit pointer comparisons.

Exception: Stay explicit on return's, example:
  return _db != nullptr;

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

[tx-robot] updated from transifex

Fix nullptr regression in RegistryUtil::ReadRegistry

Merging PR #2057 caused the Windows build to fail:

shell_integration\windows\OCUtil\RegistryUtil.cpp(43): error C2664: 'LSTATUS RegOpenKeyExW(HKEY,LPCWSTR,DWORD,REGSAM,PHKEY)': cannot convert argument 3 from 'nullptr' to 'DWORD'

The previous implementation prior the PR supplied NULL as the argument 3 to RegOpenKeyEx,
so it was silently accepted and translated to zero, satisfying the DWORD's type requirement.

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

Change default autostart to hide gui

Signed-off-by: Rick van der Zwet <info@rickvanderzwet.nl>

Merge pull request #2058 from nextcloud/sbeyer-fix-nosync-crash

Fix/improve use case without local folder

Replace prefixing of paths with file:// by QUrl::fromLocalFile()

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

Hide openLocalFolderButton if no local folder is configured

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

Fix crash on opening unconfigured local folder

When you try to open the local sync folder (by clicking on the
folder symbol) for an account that has no folder configured,
the client crashes.

This commit changes User::openLocalFolder() to do nothing in
case no local folder is configured.

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

Change width of app menu

The app menu had a width based on the header button size which was
way too narrow to display its content.

This commit changes the width based on the contents (menu items) of
the menu, however, limiting the maximum width to half the window width.

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

[tx-robot] updated from transifex

Replace NULL by nullptr in all C++ files

We keep NULL in the pure C files in src/csync/std and test/csync.

We also replace Doxygen documentation referring to "NULL" to
"\c nullptr" (formatted as code).

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

Use APPLICATION_ICON_NAME for autostart icon

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

Don't show a message box if the entered url returned an error

BEtter to now just display clickable links etc.

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

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>