Merge pull request #10236 from nextcloud/version/noid/13.0.5RC2

13.0.5 RC 2

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

13.0.5 RC 2

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #10233 from nextcloud/backport/10227/ldap-bind-loop

[stable13] Only bind to ldap if configuration for the first server is set

Fix tests for backup ldap server connection

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Only bind if configuration for the first server is available

Signed-off-by: Julius Härtl <jus@bitgrid.net>

[tx-robot] updated from transifex

Merge pull request #10172 from nextcloud/version/noid/13.0.5RC1

13.0.5 RC 1

13.0.5 RC 1

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #10201 from nextcloud/smb-rename-invalidargument-retry-13

[13] also retry rename operation on InvalidArgumentException

[tx-robot] updated from transifex

Use ILogger for consts

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #10202 from nextcloud/backport/10199/improve-url-detection-in-comments

[stable13] Improve url detection in comments

Improve url detection in comments

Signed-off-by: Joas Schilling <coding@schilljs.com>

also retry rename operation on InvalidArgumentException

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #10195 from nextcloud/backport/10193/sanitize-parameter-13

[stable13] Sanitize parameters in createSessionToken() while logging

Sanitize parameters in createSessionToken() while logging

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #10115 from nextcloud/backport/10098/ldap-image-attribute

[stable13] allow admin to disable fetching of avatars as well as a specific attribute

Merge pull request #10189 from nextcloud/backport/10020/email-address-misparsed-in-comments-and-chat

[stable13] Fix "parsing" of email-addresses in comments and chat messages

Merge pull request #10188 from nextcloud/backport/10078/invalid-actor-and-calendar-for-link-shares

[stable13] Correctly parse the subject parameters for link (un)shares of calendars

Merge pull request #10185 from nextcloud/backport/10161/correctly-handle-users-with-numeric-user-ids-13

[stable13] Correctly handle users with numeric user ids

Merge pull request #10186 from nextcloud/backport/10156/behat-3.3

[stable13] Fix the behat version to 3.3.*

Merge pull request #10170 from nextcloud/backport/9823/stable13

[stable13] Allow updating the token on session regeneration

Fix "parsing" of email-addresses in comments and chat messages

\\b matches any non-word character, including \@ and \-
In order to not detect urls in the middle of email-addresses,
we need to check for white space characters and beginning of the
message instead.

Signed-off-by: Joas Schilling <coding@schilljs.com>

Correctly parse the subject parameters for link (un)shares of calendars

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #10169 from nextcloud/backport/9955/stable13

[stable13] Request a valid property for DAV opendir

Fix the behat version to 3.3.*

Signed-off-by: Joas Schilling <coding@schilljs.com>

Correctly handle users with numeric user ids

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #10180 from nextcloud/lock-negative-13

[13] prevent lock values from going negative with memcache backend

[tx-robot] updated from transifex

prevent lock values from going negative with memcache backend

This can be caused by the code releasing more locks then it acquires,
once the lock value becomes negative it's likely that it will never be able
to change into an exclusive lock again.

Signed-off-by: Robin Appelman <robin@icewind.nl>

[tx-robot] updated from transifex

Allow updating the token on session regeneration

Sometimes when we force a session regeneration we want to update the
current token for this session.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Request a valid property for DAV opendir

Apperently Sabre and Onedrive are not friends when requesting a single
404 property. I need to dig deeper on why this is. Anyways requesting a
valid property makes it work like a charm.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #10166 from nextcloud/backport/10039/codecov-disabling

[stable13] Disable codecov status and comment

Merge pull request #10165 from nextcloud/backport/9914/settings-wording-13

[stable13] Fixed English grammatical error on Settings page.

Merge pull request #10164 from nextcloud/backport/9893/update-message-13

[stable13] Update message shown when unsharing a file

Disable codecov status and comment

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Fixed English grammatical error on Settings page.

Update message shown when unsharing a file

Signed-off-by: Kevin Ndung'u <kevgathuku@gmail.com>

Merge pull request #10121 from nextcloud/stable13-allow-disable-encryption

[stable13] allow to disable encryption

allow to disable encryption

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

adjust and add more unit tests

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

allow admin to disable fetching of avatars as well as a specific attribute

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Merge pull request #10089 from nextcloud/backport/10083/stable13

[stable13] let user set avatar in nextcloud von LDAP provides invalid image data

Merge pull request #10093 from nextcloud/smb-log-connection-errors-13

[13] Improved logging of smb connection errors

Improved logging of smb connection errors

Signed-off-by: Robin Appelman <robin@icewind.nl>

let user set avatar in nextcloud von LDAP provides invalid image data

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Merge pull request #10076 from nextcloud/backport/10034/stable13

[stable13] lower log level for quota manipulation cases

constants belong to Util in 13

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

lower log level for quota manipulation cases

and simplify the forest of ifs a little bit

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Merge pull request #10044 from nextcloud/stable13-10040-fix-filenames-in-sharing-integration-tests

[stable13] Fix filenames in sharing integration tests

Check all paths for own files and shares

Add "PARENT (2)" and its subdirectories to the paths to be checked, as
before only the own "PARENT" folder was being checked, but not the
shared one.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Fix filenames in sharing integration tests

When a file is shared and the receiver of the share already has a file
with the same name that file is left untouched, and "(2)" is appended to
the name of the shared file.

As "textfile0.txt" is included in the user folder skeleton all the users
in the integration test have that file, so when it is shared the
receiver sees the share as "/textfile0 (2).txt", and her own file as
"/textfile0.txt".

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

[tx-robot] updated from transifex

Merge pull request #10032 from nextcloud/backport/10031/stable13

[stable13] LDAP backup server should not be queried when auth fails

[tx-robot] updated from transifex

LDAP backup server should not be queried when auth fails

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #9986 from nextcloud/dav-upload-no-partfile-lock-13

[13] properly lock the target file on dav upload when not using part files

[tx-robot] updated from transifex

properly lock the target file on dav upload when not using part files

Signed-off-by: Robin Appelman <robin@icewind.nl>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #9958 from nextcloud/backport/7818/stable13

[stable13] Log full exception in cron instead of only the message

[tx-robot] updated from transifex

Log full exception in cron instead of only the message

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #9843 from nextcloud/backport/9724/stable13

[stable13] Add parent index to share table

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #9837 from nextcloud/stable13-9807-fix-highlighting-of-the-upload-drop-zone

[stable13] Fix highlighting of the upload drop zone

Merge pull request #9839 from nextcloud/backport/8221/stable13

[stable13] Apply ldapUserFilter on members of group

[tx-robot] updated from transifex

Merge pull request #9844 from nextcloud/backport/9276/remove-unused-config-sample

[stable13] Remove unused config option

Remove unused config option

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #9841 from nextcloud/backport/9814/stable13

[stable13] Make the DELETION of groups match greedy on the groupID

Add parent index to share table

Fixes #9327

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Make the DELETION of groups match greedy on the groupID

fixes #6032

Now since the match is greedy it will also eat the /

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

dn2ocname: also apply group filter to readAttribute()

Signed-off-by: Roland Tapken <roland@bitarbeiter.net>

Apply ldapUserFilter on members of group

Refers to issue #8220

user_ldap configured with custom filters for active directory access
(group-member-association is "member"). Then it can happen that the
members of a group contain members that don't belong to the users
available in Nextcloud (the most trivial reason is that the user filter
contains "(!(UserAccountControl:1.2.840.113556.1.4.803:=2))" to exclude
disabled users from being imported).

This can be fixed by applying the ldapUserFilter when resolving the UID
for a DN fetched from the group's member list.

Signed-off-by: Roland Tapken <roland@bitarbeiter.net>

Do not show an error message when draging and dropping text

When the browser reports a drag of items other than files (for example,
text) and then triggers a drop event with no files no error message
should be shown to the user, as in that case there would be no highlight
of the drop zone and no indication that the drop would be valid (except
for the mouse cursor); the error message should be shown only when
the drop event with no files follows a file drag.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Remove no longer needed special handling for Firefox

The highlighting was removed in Firefox when the cursor was no longer
moving to handle the behaviour of reporting a file drag and then
providing no files in the drop event. That behaviour (which was only
present in Firefox 48 and 49) is already handled with the "dropnofiles"
callback, so that special handling is no longer needed.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Use "dropnofiles" callback to disable the drop state in the UI

When a file is dragged from the desktop to the file list the file list
is highlighted, and when the file is finally dropped or the drag
operation is cancelled the highlighting is removed. In some cases, due
to a wrong implementation, a browser may end a file drag with a drop
with no files (for example, when a folder or text is dragged), which
would cause the highlight to not be removed. Now those cases are handled
with the "dropnofiles" callback, which restores the UI and also shows a
message to the user.

The error message is just a generic one, as in some cases it is not even
possible to know whether the problem came from a text drag or a folder
drag, and whether the problem appears or not depends on the browser,
version and even operating system.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Add callback to clean up after misbehaved drag and drop events

The jQuery Plugin triggers the "dragover" callback when the browser
triggers the "dragover" event and the types in their DataTransfer
include a "Files" item. It also triggers the "drop" callback when the
browser triggers the "drop" event and the list of files in its
DataTransfer is not empty.

Unfortunately some browsers may trigger "dragover" events with a
DataTransfer that includes a "Files" item and then trigger a "drop"
event with an empty list of files. When that happens the actions
performed in the "dragXXX" callbacks could be left hanging if they were
expected to be finished in the "drop" callback (for example, if the drop
zone was highlighted during the drag to be then restored when the file
was finally dropped). This commit adds the "dropnofiles" callback to be
able to handle those situations.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Remove duplicated code

"disableDropState" was set as the event handler in 8d4e5747f386a0, but
the duplicated code was accidentally added back in 786e858d23c4a4.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

[tx-robot] updated from transifex

Merge pull request #9821 from nextcloud/version/noid/13.0.4

13.0.4

13.0.4

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #9820 from nextcloud/backport/9798/stable13

[stable13] enable caldav for webdav subtree public-calendars

Merge pull request #9803 from nextcloud/backport/9802/stable13

[stable13] Make the token expiration also work for autocasting 0