Merge pull request #2500 from nextcloud/docs-msi

Windows MSI: Update Docs & add SKIPAUTOUPDATE property

Docs: Be slightly more modern and use 64-bit examples

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

Windows MSI: Add SKIPAUTOUPDATE property for the skipUpdateCheck setting

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

Docs: Use our app name in the MSI docs

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

Windows MSI: Stay compatible with the upstream "StartMenuShortcuts" feature

Even though we only create a program shortcut in the Start Menu, try to make administrators lives easier by not diverging feature and option naming.

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

Docs: Adopt latest MSI doc enhancements from upstream master

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

Docs: Adopt upstream commit fe04300 for MSI installation

See: https://github.com/owncloud/client/commit/fe043006c8f4a43b033243a85c294d21c7450238

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

Docs: Update macOS version info

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2497 from nextcloud/fix/mac-folderwatcher-memory-leak

macOS: Fix memory leak in FolderWatcherPrivate::startWatching

Fix source file access modes

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

macOS: Fix memory leak in FolderWatcherPrivate::startWatching

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

[tx-robot] updated from transifex

Merge pull request #2478 from Aldaris1985/patch-1

Update propagateupload.cpp

Update propagateupload.cpp

Typo corrected.

Merge pull request #2493 from nextcloud/kesselb-patch-1

Delete FUNDING.yml

Delete FUNDING.yml

[tx-robot] updated from transifex

Merge pull request #2487 from nextcloud/add-talk-icon

Add a clickable icon for Talk notifications.

Add a clickable icon for Talk notifications.

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

[tx-robot] updated from transifex

Merge pull request #2272 from snake66/issues/2219/deprecate-foreach

Issues/2219/deprecate foreach

Replace find_if with none_of where appropriate.

We're not interested in any found element in these cases, just to check
that none of the elements matches.

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

Don't hold ref to value returned from function.

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

gui/accountmanager: Make isAccountIdAvailable a bit clearer.

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

gui/folderman: Replace foreach (#2219)

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

gui/folder: Replace foreach (#2219)

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

gui/application: replace foreach (#2219)

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

gui/accountstate: Replace foreach (#2219)

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

gui/accountstate: Replace foreach (#2219)

Most instances have been converted to range based for, but std::find_if
has been used where it made sense.

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

gui/accountmanager: Replace foreach/Q_FOREACH (#2219)

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

cmd: Replace foreach with range base for (#2219)

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>

Merge pull request #2486 from nextcloud/fix-notification

Connect signal guiLog to slot to display systray message.

Correct the object name returned by the api.

The id was always 0 so no new systray messages were displayed.

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

Connect signal guiLog to slot to display systray message.

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

Merge pull request #2485 from nextcloud/Valdnet-patch-1

l10n: Remove space from string text

l10n: Remove space from string text

[tx-robot] updated from transifex

Merge pull request #2317 from nextcloud/tray-accessibility

Tray accessibility

emitting signals from Qts Accessibility interaction invocations instead of duplicating actions

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

KISS. Removed description for now, instead clarified some Accessible.name

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

Added missing qsTr() calls - again

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

Added missing qsTr() calls

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

Fix typo

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

Typo fixes, minor additions

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

Re-add wrongly delete action

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

Initial accessibility declarations for tray window qml src

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

Merge pull request #2477 from nextcloud/free_icon_job

Free IconJob after use

Free IconJob after use

Signed-off-by: Dan Griffin <github.mk@xiragon.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2470 from nextcloud/fix_qt_5_12_tooltip_wrap

Force tooltip wrap mode because it's not enabled in Qt 5.12...

Force tooltip wrap mode because it's not enabled in Qt 5.12...

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

[tx-robot] updated from transifex

Merge pull request #2409 from jplsek/native-text-render

Use native text rendering for qml windows

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>