Use OC_Util function for app web path

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

getWebDir function

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Fixed webroot for scss files
Fixed tests

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Fixed scss url rewriting

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #7672 from nextcloud/backport/6977/log-censor

[stable12] Added additional methods for removal of sensitive info

Unit tests for #6977

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

Added additional methods for removal of sensitive info

Signed-off-by: Johannes Schlichenmaier <johannes@schlichenmaier.info>

Merge pull request #7567 from nextcloud/stable-12_fix7428

Respect sharing options when searching for Sharees

Merge pull request #7577 from nextcloud/7361_12

[stable12] check userExists later, saves lookups for appData_INSTANCEID userids

check userExists later, saves lookups for appData_INSTANCEID userids

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

Merge pull request #7568 from nextcloud/stable12-fix-duplicate-session-token

[stable12] Fix duplicate session token after remembered login

Fix duplicate session token after remembered login

On a remembered login session, we create a new session token
in the database with the values of the old one. As we actually
don't need the old session token anymore, we can delete it right
away.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Don't show users which e-mail address match if they belong to a group we may not share with

Signed-off-by: Tobia De Koninck <tobia@ledfan.be>

Merge pull request #7558 from nextcloud/12-7427

[stable12] Fix email buttons for white theme

Fix email buttons for white theme

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

Merge pull request #7464 from nextcloud/cached-mount-info-filepath-12

[12] Allow getting the filepath when getting cached mounts by fileid

Merge pull request #7468 from nextcloud/update-user-agent-regex-stable12

allow 'Nextcloud' in the user agent string of Android

fix dav unit tests

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

allow 'Nextcloud' in the user agent string of Android

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

Allow getting the filepath when getting cached mounts by fileid

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

Merge pull request #7456 from nextcloud/12_7342

[stable12] CardDAV convertor check should not be to wide

Merge pull request #7457 from nextcloud/stable12-7271

[stable12] Fix loading icon position in the app menu

Merge pull request #7455 from nextcloud/12_7230

[stable12] Remove wrong entry in admin_settings that causes 500

Fix loading icon position in the app menu

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

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>

Remove wrong entry in admin_settings that causes 500

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

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