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

Code review changes implemented

Code review changes implemented for bandwidthmanager.cpp
1. Use auto instead of size_t
2. Check for empty instead of size comparisions

Signed-off-by: V.C <c******@rediffmail.com>

Support client builds on VS2019

Scenario: Build fails on Qt 5.15.0 + VS2019 16.7.2
Root cause: QLinkedList seems to be depreciated. Advise to use std::list instead.
Fix: Used std::list insead of QLinkedList.

Signed-off-by: V.C <c******@rediffmail.com>

Merge pull request #2309 from nextcloud/update-docs-page-title

Update copyright date and version displayed in the page title.

Update copyright date and version displayed in the page title.

- This is the title displayed at https://docs.nextcloud.com/desktop/x.y

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

Merge pull request #2313 from nextcloud/update-issue-template

Add --logdebug parameter to instructions in issue_template for Github.

Update issue_template with information about logs, e2ee versions and code of conduct.

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

Add --logdebug parameter to instructions in issue_template for Github.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2302 from ivaradi/no-eoan

Do not build for Eoan

Do not build for Eoan

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

[tx-robot] updated from transifex

Merge pull request #2288 from nextcloud/enh/windows-shellext

Windows shell extensions: Use custom GUIDs and prepare for MSI

Fix spacing

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

Windows shell extensions: Add WiX (MSI) fragment

Use CMake to generate a WXI fragment to handle the DLL registration and file deployment for the shellext components.

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

Windows shell extensions: Change NCUtil Pipe name from ownCloud to APPLICATION_EXECUTABLE

Avoid interference with foreign pipes as this is always a bad idea ;p

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

Windows shell extensions: Rename all files and classes from OC* to NC*, update version info

This also ensures a clear separation in the system registry.

SelfReg is not recommended by Microsoft and will be handled by the MSI package to allow proper Repair and Uninstall.
However, we keep it for backward compatibility with the NSIS installer.

For details see:
https://stackoverflow.com/questions/364187/how-do-you-register-a-win32-com-dll-file-in-wix-3#364210
https://docs.microsoft.com/en-us/windows/win32/msi/selfreg-table#remarks

Another fix by this commit:
The "Version" registry value in the NCOverlays self reg should be a key and not a value.

Details: https://wixtoolset.org/documentation/manual/v3/xsd/wix/class.html

Example:

  [HKCR\CLSID\{01234567-89AB-CDEF-0123-456789ABCDEF}\Version]
  @="1.0.0.0"

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

Windows shell extensions: Use custom GUIDs via CMake and prepare MSI reg

Previously side by side installation with ownCloud or other NC custom builds would break the shell integration because the same GUIDs and registry keys were used.

Now we specify our custom GUIDs in NEXTCLOUD.cmake and use CMake to generate a header file and WiX (MSI) include file with these constants.

Note: Using generators like "guidgen" or "uuidgen" ensures that GUIDs are unique, as manual changes are not guaranteed to be.

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

Remove dead code

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

2.7 has less automatic includes on Windows

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

Cleanup Windows shell extensions

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

Remove outdated visual studio projects, we now use cmake

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

Avoid warning because /MT overrides /MD flag

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

Make OCUtil helper lib static and link it statically against crt

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

Windows Shell Integration: Don't limit the size of the buffer

Otherwise we can't have operation that has many many filename

As reported in #6780

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

Windows shellext: Update copyrights and company name

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

windows shell extension: add OCUtil/resource.h

It's a copy from OCContextMenu/resources.h

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

Windows shellext: Add rc for OCUtil #6554

To set the dll's metadata.

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

Merge pull request #2292 from nextcloud/prevent_sharing_for_e2ee_folders

Disable context menu actions for E2EE files and folders

Present sharing of E2EE files from the main dialog too

The visibility of the share button for the activity list is tied to the
path role in the ActivityListModel, so make sure we don't return a path
for E2EE files and folders.

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

Disable context menu actions for E2EE files and folders

Those files and folders are not shareable and the "Edit" or "Open in
browser" actions will lead to showing an error in the web GUI. No need
to lead users there so just disable them.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Show the "Display mnemonic" button only when we know e2e is supported

Turns out that showing the button straight from the page ctor is a bit
too early. At that point the account might not be connected yet and thus
we wouldn't have proper information. Currently we were displaying that
button all the time, now we wait for the account to be connected to
decide to show it or not.

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

Minor fix in text of warning and debug message.

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

Make sure we don't assert when calling fileStatus

It happens that sometimes we leak a directory path ending with a slash,
but that violates fileStatus' precondition so let's catch it early and
skip such path.

Of course the right fix would be a larger swipe in the sync engine and
around it to not use naked strings anymore but rely on the typesystem.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex