[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2126 from nextcloud/sbeyer-simplify-systray-positioning

Slightly simplify code for Systray positioning

Hide Systray's auxiliary positioning methods from public and Qml

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

Simplify tray window positioning

The Qml position setting code is moved to C++, which allowed to get rid
of the currentScreenIndex() method (which seemed to be just a detour to
pass the screen from C++ to Qml).

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

[tx-robot] updated from transifex

Fix crash in UserModel::currentUser() and add more sanity checks

Commit 07bede8 (PR #1892) introduced a new helper method currentUser()
that didn't check for _users.count() thus causing to throw an
"index out of range" exception when no accounts are configured.

This commit uses the opportunity to add more sanity checks to UserModel.

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

[tx-robot] updated from transifex

Merge pull request #2119 from nextcloud/sbeyer-fix-segv-on-exit

Fix SEGV (due to circular ownership) at exit

Make Systray's _trayEngine a QPointer

This is just safer.

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

Fix SEGV (by circular ownership) at exit

Commit a12205f322d43476230881192616e47c9cf786ef (PR #1891) introduced
a circular ownership: qmlRegisterSingletonType<Systray>(...) makes the
QQmlEngine own the resulting singleton Systray instance, however, the
QQmlEngine _trayEngine itself is owned by the Systray instance. This
circular ownership results in a crash when the destructor of Systray
calls the destructor of _trayEngine which attempts to call the destructor
of Systray.

This commit solves this problem by making ownCloudGui, which is the
parent of Systray, the parent of the _trayEngine.

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

Let Systray constructor call its base class constructor

Although I see no behavioral difference, this is probably
a good idea.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Fix issue #2108: Default system proxy not working with Account Wizard

Discovered on Windows in conjunction with PAC scripts:
- Already configured accounts worked
- Fresh client account setup did not work

Reason:
- Proxy was reset over and over again in Account::setCredentials

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

Merge pull request #2110 from nextcloud/sbeyer-menu-border-colors

Darken menu border color

Make sure the menu separator uses Style.menuBorder color

In case the default color value will ever change, this
commit makes sure that the colors of the menu border and
the menu separator coincide.

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

Darken menu border color

The lightgray menu border color #ededed introduced in
commit f147e5a66f43657805542904b43e0b4e662ffe04 (by PR #2095)
is way too light for my display (and probably others).
Thus the menus have no clear border which looks odd and broken.

This commit simply darkens the menu border to #bdbdbd, which
coincides with the (current default) color of the menu separator.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2106 from nextcloud/fix/proxy-build-libsync

Fix libsync exports for proxy changes in #1886

Fix libsync exports for proxy changes in #1886

Build failed on Windows, missing libsync export for printQNetworkProxy after
moving ClientProxy class from gui to libsync.

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

Merge pull request #1886 from nextcloud/enh/proxy

Move the proxyfiles to libsync where they make more sense

Improve proxy logging: Log type string instead of abstract ID

The format in the client log was: 3://host:port
Now it is shown like:             HttpProxy://host:port

The old ID display was confusing and misleading users (and people debugging it ;p)

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

Move the proxyfiles to libsync where they make more sense

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: Michael Schuster <michael@schuster.ms>

[tx-robot] updated from transifex

Merge pull request #2093 from nicrame/patch-1

Add photothumb.db to sync-exclude.lst

Merge branch 'master' into patch-1

Merge pull request #2099 from nextcloud/ease_access_to_main_dialog

Ease access to main dialog

Bring forward the main systray dialog on second execution

Previously we were showing up the settings dialog but it doesn't quite
make sense anymore now that we got two dialogs.

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

Also show the window on right click

Note this won't work on all platforms. KDE Plasma and GNOME Shell (with
systray extension) assume that right click is necessarily for a context
menu exposed via D-Bus, there's not nice way to make the right click
popup the main dialog on those platforms.

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

Merge branch 'master' into patch-1

[tx-robot] updated from transifex

Don't call method on potentially null object

We were calling accountState() on a "folder" member which could be
nullptr. In fact this would happen any time one right click on a file
outside of a sync dir under Windows, this thus led to a crash.

Since the capabilities variable was unused anyway, we just removed it.

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

Make activity items with a link clickable

We then get more items clickable, in particular the bottom one proposing
to open the activity application. Now we can click it and it opens the
activity application straight away.

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

Don't use a bright blue color for menus

As advised, use a light gray instead, this electric blue was a bit too
much.

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

Don't use a weirdly truncated icon size for header buttons

This led to blurry icons. Now use a fixed size of 32 which also makes
them a bit more pleasing... at least to me. ;-)

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

Don't duplicate file name in the info line

For the "Activity" type activities, displayPath contains the file name
which is also in the subject. This was redundant, so now we don't
display an info line at all for that type of activities.

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

Close all menus on window show

We previously missed the app menu which wasn't closed.

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

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

Add photothumb.db to sync-exclude.lst

Add photothumb.db file that is thumbnails generated by Photoscape application (generated in every folder with graphic files).

[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>