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

Merge pull request #2271 from ivaradi/cstdint-fix

Include cstdint only when compiling C++ code

Merge branch 'master' into cstdint-fix

[tx-robot] updated from transifex

Include cstdint only when compiling C++ code

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

[tx-robot] updated from transifex

Merge pull request #2268 from nextcloud/cherry_pick_testchunkingng_related_improvements

Cherry pick TestChunkingNG related improvements

Size cleanups in TestChunkingNG

Some of the comments didn't match the size or were missing. This also
means reducing one of the 150 MB payloads left behind reducing the
execution time by a few more seconds. This is now around 30s execution
time which is more acceptable.

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

Upload: Store the size in the UploadInfo, and compare it when resolving potential conflict

This is about the conflicts that happens when the file has been uploaded
correctly to the server, but the etag was not recieved because the connection
was closed before we got the reply.

We used to compare only the mtime when comparing the uploaded file and the
existing file.  However, to be perfectly correct, we also should check the
size.

This was found because TestChunkingNG::connectionDroppedBeforeEtagRecieved is
flaky. Example of faillure found in https://drone.owncloud.com/owncloud/client/481/5
while testing PR #6626

(very trimmed log:)

06-29 07:58:02:015 [ info sync.csync.csync ]: ## Starting local discovery ##
06-29 07:58:02:016 [ info sync.csync.updater ]: Database entry found, compare: 1530259082 <-> 1530259051, etag:  <-> 1644a8c8750, inode: 1935629 <-> 1935629, size: 301 <-> 300, perms: 0 <-> ff, type: 0 <-> 0, checksum:  <-> SHA1:cc9adedebe27a6259efb8d6ed09f4f2eff559ad1, ignore: 0
06-29 07:58:02:016 [ info sync.csync.updater ]: file: A/a0, instruction: INSTRUCTION_EVAL <<=
06-29 07:58:02:972 [ warning sync.networkjob ]: QNetworkReply::NetworkError(OperationCanceledError) "Connection timed out" QVariant(Invalid)
.. next sync...
06-29 07:58:02:980 [ info sync.engine ]: #### Discovery start ####################################################
06-29 07:58:02:981 [ info sync.csync.csync ]: ## Starting local discovery ##
06-29 07:58:02:983 [ info sync.csync.updater ]: Database entry found, compare: 1530259082 <-> 1530259051, etag:  <-> 1644a8c8750, inode: 1935629 <-> 1935629, size: 302 <-> 300, perms: 0 <-> ff, type: 0 <-> 0, checksum:  <-> SHA1:cc9adedebe27a6259efb8d6ed09f4f2eff559ad1, ignore: 0
06-29 07:58:02:983 [ info sync.csync.updater ]: file: A/a0, instruction: INSTRUCTION_EVAL <<=
06-29 07:58:02:985 [ info sync.csync.csync ]: ## Starting remote discovery ##
06-29 07:58:02:985 [ info sync.networkjob ]: OCC::LsColJob created for "http://localhost/owncloud" + "" "OCC::DiscoverySingleDirectoryJob"
06-29 07:58:02:987 [ info sync.csync.updater ]: Database entry found, compare: 1530259082 <-> 1530259051, etag: 1644a8c8b26 <-> 1644a8c8750, inode: 0 <-> 1935629, size: 301 <-> 300, perms: ff <-> ff, type: 0 <-> 0, checksum: SHA1:5adcdac9608ae0811247f07f4cf1ab0a2ef99154 <-> SHA1:cc9adedebe27a6259efb8d6ed09f4f2eff559ad1, ignore: 0
06-29 07:58:02:987 [ info sync.csync.updater ]: file: A/a0, instruction: INSTRUCTION_EVAL <<=
06-29 07:58:02:989 [ info sync.csync.csync ]: Update detection for remote replica took 0.004 seconds walking 13 files
06-29 07:58:02:990 [ info sync.engine ]: #### Discovery end ####################################################  9 ms
06-29 07:58:02:990 [ info sync.database ]: Updating file record for path: "A/a0" inode: 1935629 modtime: 1530259082 type: 0 etag: "1644a8c8b26" fileId: "16383ea4" remotePerm: "WDNVCKR" fileSize: 301 checksum: "SHA1:cc9adedebe27a6259efb8d6ed09f4f2eff559ad1"
06-29 07:58:02:990 [ info sync.csync.reconciler ]: INSTRUCTION_UPDATE_METADATA    client file: A/a0
06-29 07:58:02:990 [ info sync.csync.csync ]: Reconciliation for local replica took  0 seconds visiting  13  files.
06-29 07:58:02:990 [ info sync.csync.reconciler ]: INSTRUCTION_UPDATE_METADATA    server dir:  A
06-29 07:58:02:990 [ info sync.csync.csync ]: Reconciliation for remote replica took  0 seconds visiting  13  files.
06-29 07:58:02:990 [ info sync.engine ]: #### Reconcile end ####################################################  9 ms
06-29 07:58:02:990 [ info sync.database ]: Updating local metadata for: "A/a0" 1530259082 302 1935629
FAIL!  : TestChunkingNG::connectionDroppedBeforeEtagRecieved(small file) '!fakeFolder.syncOnce()' returned FALSE. ()

TestChunkingNG: Make it faster

Use smaller files so the test run faster.
Particulary usefull for TestChunkingNG::connectionDroppedBeforeEtagRecieved
Which had become so much slower after 2638332dc63cd79410b6fea5a24449696d63d3f2
increased the timeout for bigger files

Upload: Add more chunk resuming unit tests #6405

Merge pull request #2269 from nextcloud/enable_modernize_loop_convert

Enable the modernize-loop-convert check on clang-tidy

Enable the modernize-loop-convert check on clang-tidy

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

Remove unused method

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

Enable the modernize-deprecated-headers check on clang-tidy

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

Enable the modernize-use-equals-default check on clang-tidy

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

[tx-robot] updated from transifex

Enable the modernize-user-nullptr check on clang-tidy

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

Merge pull request #2264 from nextcloud/enable_modernize_use_using

Enable the modernize-use-using check on clang-tidy

Get rid of useless using constructs

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

Make QTokenizer use simpler by having optional second parameter

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

Enable the modernize-use-using check on clang-tidy

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

Merge pull request #2262 from nextcloud/enable_trivial_clang_tidy_modernize_checks

Enable "trivial" modernize clang-tidy checks

Enable "trivial" modernize clang-tidy checks

This is a bunch of modernize checks which we don't hit at all. This is
an easy win and also make sure those old constructs won't creep up
behind our back.

Note that some of them won't trigger until we bump our C++ baseline
standard to C++17. But I'd say that's OK, that'll force our hand to do
those C++17 related cleanups when the time comes.

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

Merge pull request #2037 from nextcloud/enable_modernize_raw_string_literal

Enable modernize raw string literal

We don't really need two \ here

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

Add more raw string literals missed previously

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

Merge pull request #2247 from nextcloud/improve_drone_config

Improve drone config

Make sure wget looks for the certificates at the right place

Might happen because we got too many copies of OpenSSL around

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

Switch to Ninja for the clang build

Clang is generally slower to compile than GCC so use a faster Make for
that build variant. This also allows us to verify our CMake files don't
do anything Make specific.

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

Parallelize the clang-tidy runs

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

Split cmake and compile into separate pipeline steps

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

Switch to actual commands inside of the steps

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

Split the CI pipelines into proper steps

Will make it easier to navigate the logs

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

Now make sure we're not picking up generated headers

This could happen (and started to happen more with clang-tidy 10) that
clang-tidy picked up errors in headers from the build directory. Now
that we moved the build directory out of source, we can simply filter
headers based on the pwd.

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

Move the build in a volume to be reused between steps

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

Remove unneeded environment variables

Those are set by the base CI image anyway, no need to do the work again.

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

Switch to the newer base CI image

We can now spare building QtKeychain over and over shortening a bit the
build. We also update to latest gcc and clang.

Note that I didn't switch the AppImage pipeline to the newer image to
keep the binary compatibility promise required by linuxdeployqt.
Probably would make sense to do a specific image for AppImage with some
other of the cleanups I did there (in particular to avoid building
qtkeychain every time).

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

Parallelize the CI build

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

[tx-robot] updated from transifex

Merge pull request #2258 from nextcloud/have_a_way_to_display_e2ee_mnemonic

Keep the E2EE info message around to allow displaying mnemonic

Keep the E2EE info message around to allow displaying mnemonic

We lost the ability to display the E2EE mnemonic during the GUI
redesign and the info message wasn't displayed again on restart. So now
we display it every time, it still can be dismissed and the button text
is different in such a case to make the intent clearer.

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

[tx-robot] updated from transifex