CardDAV convertor check should not be to wide

Case: email is set to null, but the avatar is set. In the old case the
email would set $emptyValue but $noImage would still be false. This we
would set the empty string as email.

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

Merge pull request #7451 from nextcloud/12-7433

[stable12] Fix constructor spy in unit test with Sinon 4.1.3

Fix constructor spy in unit test with Sinon 4.1.3

When a constructor is spied using Sinon it is wrapped by a proxy
function, which calls the original constructor when invoked. When "new
Foo()" is executed a "Foo" object is created, "Foo" is invoked with the
object as "this", and the object is returned as the result of the whole
"new" expression.

Before Sinon 4.1.3 the proxy called the original constructor directly
using the "thisValue" of the spied call; "thisValue" was the object
created by the "new" operator that called the proxy. The proxy assigned
"thisValue" to "returnValue", so it was also the value returned by the
proxy and, in turn, the value returned by the whole "new" expression.

Since Sinon 4.1.3 (see pull request 1626) the proxy calls the original
constructor using "new" instead of directly. The "thisValue" created by
the outermost "new" (the one that called the proxy) is no longer used by
the original constructor; the internal "new" creates a new object, which
is the one passed to the original constructor and returned by the
internal "new" expression. This object is also the value returned by the
proxy ("returnValue") and, in turn, the value returned by the whole
outermost "new" expression.

Thus, now "returnValue" should be used instead of "thisValue" to get the
object created by the spied constructor.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #7409 from nextcloud/swift-retry-12

[12]add retry wrapper when reading files from swift

Merge pull request #7407 from nextcloud/12_7348

[stable12] Check if owner of share exists

Merge pull request #7323 from nextcloud/shared-jailed-source-root-12

[12]Use the correct root for shared jail when the source storage is also a jail

[tx-robot] updated from transifex

[tx-robot] updated from transifex

add retry wrapper when reading files from swift

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

Fix tests

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

A failed storage is a not available storage

We have to double check. Since getting the info of the root returns a
generic entry. But actually the stroage is not available. Else we get
very weird sync and web behavior.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

delay calculating the shared cache root until it's used

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

Merge pull request #7144 from nextcloud/rename-locks-stable12

Unlock files even if an exception occurs (stable12)

Merge pull request #7322 from nextcloud/12-6303

[stable12] Set primary action button color to same as theming color

Merge pull request #7353 from nextcloud/12-7346

[stable12] Fix translation of federation scope menu

Merge pull request #7362 from nextcloud/12-7347

[stable12] Allow to skip data dir permission checks by config

Merge pull request #7355 from nextcloud/12.0.4-final

Release 12.0.4

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Allow to skip data dir permission checks by config

Enables installation on services that don't allow/support permission
changes.

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

[tx-robot] updated from transifex

Release 12.0.4

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

Fix translation of federation scope menu

* l10n in Nextcloud works by extracting the values only passed on their location and not based on the first parameter
* we need to change the translation pool from `core` to `settings` then
* fixes #7345

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #7330 from nextcloud/1204-rc3

12.0.4 RC3

12.0.4 RC3

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

Merge pull request #7328 from nextcloud/backport/7327/access-list-regression-for-not-current-accesss

[stable12]  Only in case of $currentAccess the array uses the id as index

Only in case of $currentAccess the array uses the id as index

Otherwise its a normal string[] with the user ids, in that
case the array_merge did it's job just fine, apart from it
not being deduplicated.
The array+array is only needed when the user id is the key,
so integer only user ids are kept as they are instead of being
reindexed.

Regression from 3820d6883dffcaa49deb1054ae0f32ab3d3e39b1

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

Fix current access list test and add one for $currentAccess === false

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

Use the correct root for shared jail when the source storage is also a jail

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

Set primary action button color to same as theming color

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

[tx-robot] updated from transifex

Merge pull request #7261 from nextcloud/prepare-12.0.4-RC2

Prepare 12.0.4 RC2

Merge pull request #7257 from kyrofa/bugfix/5289/apps_outside_webroot

[stable12] CSSResourceLocator: handle SCSS in apps outside root

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #7277 from nextcloud/12-7268

[stable12] Update CRL due to files_frommail

Update CRL due to files_frommail

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

[tx-robot] updated from transifex

Merge pull request #7262 from nextcloud/backport/7168/files-access-list-with-integer-userid

[stable12] Fix accesslist when a user has an ID only containting 0-9

Merge pull request #7245 from nextcloud/feature/noid/allow-migration-from-comingup-oc-release

Allow migration from upcoming 10.0.4 ownCloud release

Fix accesslist when a user has an ID only containting 0-9

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

Prepare 12.0.4 RC2

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

Merge pull request #7259 from nextcloud/7211-stable12

[stable12] only replace permission popupmenu

only replace permission popupmenu

the contacts popovermenu is also present and is being replaces, ending
up in two permission popupmenus with checkboxes duplicating the id,
breaking further permission changes.

plus, fixing a selector

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

CSSResourceLocator: handle SCSS in apps outside root

Currently static CSS files work fine in apps outside of the root.
However, as soon as an app uses SCSS, Nextcloud starts being unable to
find the web root.

Fix this problem by backporting select snippets from master
specifically targeting this issue, and add a test to ensure it doesn't
regress.

Fix #5289

Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>

[tx-robot] updated from transifex

Allow migration from upcoming 10.0.4 ownCloud release

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #7225 from nextcloud/12.0.4RC-version

12.0.4 RC

Merge pull request #7170 from kyrofa/backport/6028/app_path_realpath

[stable12] {J,CS}SResourceLocator: account for symlinks in app path

12.0.4 RC

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #7207 from nextcloud/fix-object-storage-touch-12

[stable12] touch opertation on object storage, don't create the file cache entry to early

only create the file cache entry after the empty file was created successfully, otherwise file_exists() call on the initial file_put_content() will indicate that the file already exists

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

Merge pull request #7186 from nextcloud/s3-download-stream-fix-12

[12] Fix s3 download and touch

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Use non empty files for object store touch

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

Fix contructing headers for s3 download

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

[tx-robot] updated from transifex

{J,CS}SResourceLocator: account for symlinks in app path

Currently, if the app path includes a symlink, the calculated webDir
will be incorrect when generating CSS and URLs will be pointing to the
wrong place, breaking CSS.

Use realpath when retrieving app path, and these issues go away.

Fix #6028

Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Unlock files even if an exception occurs

Signed-off-by: Kristof Hamann <korelstar@users.noreply.github.com>

[tx-robot] updated from transifex

Merge pull request #6464 from nextcloud/proper-english-nowhave-backport-stable12

[stable12] Improve text: 'you have now' -> 'you now have'

[tx-robot] updated from transifex

Merge pull request #7120 from nextcloud/12-6988

[stable12] Improve performance of UserMountCache with external storage folders

Merge pull request #7116 from nextcloud/12-fix-6033

[stable12] Fix icon for security settings

Merge pull request #7118 from nextcloud/12-7109

[stable12]  If for some reason the json can't be decoded it is not cached

[tx-robot] updated from transifex

Improve performance of UserMountCache with external storage folders

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

If for some reason the json can't be decoded it is not cached

Should fix #6898

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

Merge pull request #6847 from nextcloud/filelist-confict-path-12

[12] still trigger conflict resolution for existing entries when the curre…

Fix welcome text for better grammar in themed instances

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

Fix icon for security settings

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

[tx-robot] updated from transifex

Merge pull request #7114 from nextcloud/12-7110

[stable12] theme flow redirection page

Theming: theme flow redirection page

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

Merge pull request #7102 from nextcloud/revert-6774-backport-6645-only-colons-for-ipv6

Revert "[stable12] Only allow colons in db host for IPv6 addresses"

[tx-robot] updated from transifex

Revert "[stable12] Only allow colons in db host for IPv6 addresses"

Merge pull request #7096 from nextcloud/backport-stable12-7042

[stable12] ensure uid for calendar objects is unique

never show '/' as path for conflicting file names

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

still trigger conflict resolution for existing entries when the current entry doesn't need it

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

Merge pull request #6699 from nextcloud/backport-6522-stable12

[stable12] fix LDAP User deletion (cleanup), fixes #3365

Merge pull request #7095 from nextcloud/12-7059

[stable12] hide spinner for initial install

ensure uid for calendar objects is unique

Signed-off-by: Georg Ehrke <developer@georgehrke.com>