Use native text rendering for qml windows

Signed-off-by: Jeremy Plsek <jeremyplsek@gmail.com>

Merge pull request #2441 from nextcloud/enh/server-url

Allow server URL to be pre-defined without enforcing it

Allow server URL to be pre-defined without enforcing it to be used unmodified

APPLICATION_SERVER_URL can be either empty or be specified. This commit adds the new CMake option APPLICATION_SERVER_URL_ENFORCE to decide whether to enforce the
URL's unmodified use (like before, default: ON) or to allow modification by the user (new).

By default APPLICATION_SERVER_URL_ENFORCE is set to ON, to no break with the previous implementation's expectations.

If APPLICATION_SERVER_URL is empty, APPLICATION_SERVER_URL_ENFORCE will be ignored by the Account Wizard.

The previous behaviour confused me a bit with branded builds. When the URL was (usually) specified but not forced, it was simply discarded, forcing the user to
manually supply it.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2369 from nextcloud/enh/windows-msi

Windows: MSI support & Win32 migration tools

Windows: Win32 migration tools code review

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

Windows MSI: Add MSI scripts and UI resources

Uses CMake to generate and install all required files in the "msi/" directory.

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

Windows: Add Win32 stand-alone migration tool NCNavRemove

Removes all Explorer Navigation Pane entries for a given ApplicationName, specified in NavRemove.ini in the working directory.

Also compiles a DLL with the same behaviour that exports:
- RemoveNavigationPaneEntries

Both tool variants are Mutex-protected.

Statically linked, optimized for binary size, no Qt dependencies.

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

Windows MSI: Add helper DLL and shared migration tools code

The helper DLL will be utilized by Windows Installer with Custom Actions defined in the NCMsiHelper.wxs WiX fragment.

Exports:
- ExecNsisUninstaller
- RemoveNavigationPaneEntries

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

Windows MSI: Let CMake install Shell Extensions WiX fragment to "msi/"

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

Windows MSI: Add Upgrade Code and migration tool build options to NEXTCLOUD.cmake

The Upgrade Code is a GUID (specified without brackets) for the MSI package to allow Windows Installer identify existing installations.

New build options (default: OFF):
- BUILD_WIN_MSI: Build all MSI scripts and a required helper DLL (to uninstall NSIS legacy installations and remove Explorer Navigation Pane entries)
- BUILD_WIN_TOOLS: Build additional migration tools (currently NCNavRemove, a stand-alone tool for Explorer entries removal)

The helper DLL and migration tools are set to be statically linked and optimized for binary size.

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

Merge pull request #2431 from jplsek/remove-this-includes

Remove "This includes" due to tooltip order

Remove "This includes" due to tooltip order

Signed-off-by: Jeremy Plsek <jeremyplsek@gmail.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2437 from nextcloud/remove_no_description_available_messages_in_activity_list

If there's no description just say nothing

If there's no description just say nothing

Otherwise we get lots of "No description available" lines in the
activity list which is basically noise. Also trains the user to ignore
the secondary line.

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

Merge pull request #2445 from nextcloud/fix-mac-catalina-login

Entitlement to sign the QtWebEngineProcess with an exception.

Entitlement to sign the QtWebEngineProcess with an exception.

Fix for #1793: The problem seems to be related enabling hardened runtime.
This exception allows the webview to load.

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

[tx-robot] updated from transifex

Merge pull request #2439 from nextcloud/fix-subfolder-missing-contentsync

Fix missing subdirectory discovery on move operations in macOS

Fix missing subdirectory discovery on move operations in macOS.

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
Add qAsConst to avoid detaching

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
Changed callchain to find and return possibly coalesced paths

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
Removed another qAsConst remnant, more func const-correctness.

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

[tx-robot] updated from transifex

Merge pull request #2435 from nextcloud/fix-journal-upgrade-multiple-syncconnections

Fix incomplete journal upgrade from 2.x to 3.x

Do not break out of outer loop when upgrading folders, continue instead

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
Remove awkward debug line a.k.a. whoops

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

Merge pull request #2432 from nextcloud/fix/nsis-language-encoding

Windows: Fix NSIS language encoding (use UTF-8)

Windows: Remove unused and outdated NSIS Transifex settings

The last translation update from Transifex was done in 2017, so the scripts and settings were never used for Nextcloud.

This commit removes the unused scripts and "Auto-generated" comments in the translation files.

Some files have been modified over time either way and MSI will replace NSIS soon.

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

Windows: Fix NSIS language encoding (use UTF-8 instead of Win code pages)

The old conversion to specific code pages caused display errors (see issue #2393 for Russian).

Based on bin/l10n.sh, the files in this commit were converted to UTF-8 using iconv with the following parameters:

  iconv -f CP1252 -t UTF-8 -o German.nsh German.nsh
  iconv -f CP1252 -t UTF-8 -o Galician.nsh Galician.nsh
  iconv -f CP1253 -t UTF-8 -o Greek.nsh Greek.nsh
  iconv -f CP1250 -t UTF-8 -o Slovenian.nsh Slovenian.nsh
  iconv -f CP1257 -t UTF-8 -o Estonian.nsh Estonian.nsh
  iconv -f CP1252 -t UTF-8 -o Italian.nsh Italian.nsh
  iconv -f CP1252 -t UTF-8 -o PortugueseBR.nsh PortugueseBR.nsh
  iconv -f CP1252 -t UTF-8 -o Spanish.nsh Spanish.nsh
  iconv -f CP1252 -t UTF-8 -o Dutch.nsh Dutch.nsh
  iconv -f CP1252 -t UTF-8 -o Finnish.nsh Finnish.nsh
  iconv -f CP932 -t UTF-8 -o Japanese.nsh Japanese.nsh
  iconv -f CP1250 -t UTF-8 -o Slovak.nsh Slovak.nsh
  iconv -f CP1254 -t UTF-8 -o Turkish.nsh Turkish.nsh
  iconv -f CP1252 -t UTF-8 -o Norwegian.nsh Norwegian.nsh
  iconv -f CP852  -t UTF-8 -o Czech.nsh Czech.nsh

The installer script nextcloud.nsi in the client-building repo needs the following line (to be added on top):
  Unicode true

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

[tx-robot] updated from transifex

Merge pull request #2424 from nextcloud/enable_ci_for_stable_branches

Try to enable the CI on stable branches as well

Try to enable the CI on stable branches as well

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

Merge pull request #2423 from nextcloud/disable_bugprone_forward_declaration_namespace_clang_tidy_check

Disable bugprone-forward-declaration-namespace check

Disable bugprone-forward-declaration-namespace check

Turns out this clang-tidy check can give false positives coming from
headers outside the project while it shouldn't.

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

Merge pull request #2412 from jplsek/tooltip-update

Include list info in tooltip

Include list info in tooltip

Also extend the tooltip timeout

Signed-off-by: Jeremy Plsek <jeremyplsek@gmail.com>

Merge pull request #2417 from nextcloud/fix_menu_popup_in_main_dialog

Fix menu popup in main dialog

Make sure menu are closed when button is clicked again

This way we also made sure all the Menu are currently on the same close
policy.

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

Delay binding on the menu width

Indee the MenuItem might not be linked to its Menu at creation time
which will make the binding fail and give a warning. Delay for the menu
availability.

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

Remove pre-QQC2.3 workaround

Now that we depend on Qt 5.12 anyway, the count property is available
just fine on the Menu item.

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

Merge pull request #2418 from nextcloud/use_primary_screen_as_fallback_when_qcursor_pos_search_failed

Use QGuiApplication::primaryScreen() as fallback for Systray::currentScreen()

Use QGuiApplication::primaryScreen() as fallback for Systray::currentScreen()

Under Wayland QCursor::pos() is unlikely to give us anything meaningful,
so fallback to the primary screen information.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Don't use a confusing "royal you" in the sync messages

Users get rightfully confused with the "You changed ..." messages in the
activity list for syncs. Indeed, some of those changes might be coming
from the server in which case we don't really know who did the change.

So now we use the old "Synced ..." messages for changes pulled from the
server and we have a more precise "You changed ..." (renamed, deleted,
created) when the changes were initiated locally (since there we know
the user reading the message did it).

Also changed how the messages are constructed so that they can be
properly translated.

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

[tx-robot] updated from transifex

Click on activity list in systray open file in the file manager.

Clean up code, add more const and auto.

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

Fix tooltip for activity info.

Also improves readability of the items in the list.

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

l10n: Add a period

[tx-robot] updated from transifex

l10n: Extract the button name

l10n: Delete unnecessary sentence

l10n: A few more fixes

l10n: Word correction

l10n: Corrected translation

Leave the implicit uint to int cast

I wrongly (and stupidly) assumed the int in QBitArray was treated as a
hash as well but it is an index so it must be positive. Also to make
things even worse I misread on which expression clang-tidy was
complaining regarding the implementation specific narrowing
conversion... This is happening after the modulo operator and not
before. We're in a safe range of values at that point, so it's fine to
let the narrowing happen.

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

Merge pull request #2387 from nextcloud/update-troubleshooting

Update troubleshooting

Add README with instructions on how to build the documentation.

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

Update troubleshooting instructions with the newly added config file options.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2357 from thaytan/metadata-index-e2eMangledName

syncjournaldb: index e2eMangledName column in metadata

syncjournaldb: index e2eMangledName column in metadata

Add an index on the e2eMangledName column in the metadata table
to speed up file sync by orders of magnitude on directories with
a large number of files.

Signed-off-by: Jan Schmidt <jan@centricular.com>

[tx-robot] updated from transifex

Merge pull request #2351 from nextcloud/fix_clang_tidy_errors_in_csync_tests

Fix clang tidy errors in csync tests

Switch to newer CI image to build csync tests

This new image brings the cmocka dependency and thus will make the CI
build the csync tests which depend on it.

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

Fix bugprone-narrowing-conversion errors in csync tests

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

Fix modernize-use-nullptr errors in csync tests

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

Fix modernize-use-auto errors in csync tests

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

Fix modernize-deprecated-headers errors in csync tests

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

[tx-robot] updated from transifex

Merge pull request #2283 from nextcloud/enable_bugprone_clang_tidy_checks

Enable bugprone clang tidy checks

Enable all the other bugprone clang-tidy checks

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

Enable bugprone-terminating-continue clang-tidy check

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

Enable bugprone-too-small-loop-variable clang-tidy check

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

Enable bugprone-narrowing-conversions clang-tidy check

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

Enable bugprone-macro-parentheses clang-tidy check

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

Enable bugprone-forward-declaration-namespace clang-tidy check

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

Enable the bugprone-branch-clone clang-tidy check

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

Enable the bugprone-argument-comment clang-tidy check

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

[tx-robot] updated from transifex

Merge pull request #2345 from nextcloud/use_httpd_unix-directory_mimetype_for_e2ee_folders

Use httpd/unix-directory mimetype for E2EE folders

Use httpd/unix-directory mimetype for E2EE folders

We don't do much with that mimetype on our end, but other clients
somehow don't expect inode/directory to let's lie. ;-)

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

Enable all the misc clang-tidy check except one

This flagged mostly unused parameters. Didn't enable the
misc-non-private-member-variables-in-classes check as we got a lot of
those. Hopefully we'll get to fix them at some point but that feels too
early and too much work for now.

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 #2336 from nextcloud/jospoortvliet-patch-1

Some changes to the template...

Some changes to the template...

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Remove remaining QLinkedList includes in addition to #2300

Now the only remaining reference is in:

src/3rdparty/libcrashreporter-qt/3rdparty/drkonqi-parser/backtraceparser.cpp:269:        || line.functionName().startsWith(QLatin1String("QLinkedList")

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

Adapt code style in addition to #2300

- Use "!empty()" instead of "size() > 0" in std::list
- Add comments for namespace brackets

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

Merge pull request #2300 from chenecharry/master

Support client builds on VS2019 (QLinkedList is deprecated)

Merge branch 'master' into master

[tx-robot] updated from transifex