Merge pull request #7674 from nextcloud/12_7644

[stable12] Do no run SyncJob in cron

Merge pull request #7666 from nextcloud/stable12-7527-fix-drag-shadow-not-visible-when-dragging-a-file-on-a-narrow-screen

[stable12] Fix drag shadow not visible when dragging a file on a narrow screen

Merge pull request #7742 from nextcloud/stable12-7716-hide-favourite-icon-in-details-view-if-favourite-action-is-not-available

[stable12] Hide favourite icon in details view if favourite action is not available

[tx-robot] updated from transifex

Merge pull request #7694 from nextcloud/stable12-7611

[stable12] Don't attempt to translate login names to uids when uids are provided

Hide favourite icon in details view if favourite action is not available

When the favourite icon in the details view is clicked the "Favorite"
action is triggered. However, if the action name given to
"triggerAction" is not found then the "Download" action is triggered
instead. As the "Favorite" action is not available in some file lists
(like "Recents") the "Download" action was executed instead in those
cases, which was a strange behaviour. Now the favourite icon is
hidden if its action is not available.

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

[tx-robot] updated from transifex

Merge pull request #7717 from nextcloud/stable12-7662

[stable12] throw ServerNotAvailableException when LDAP is caught shutting down

[tx-robot] updated from transifex

[tx-robot] updated from transifex

throw ServerNotAvailableException when LDAP is caught shutting down

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

Merge pull request #7688 from nextcloud/stable12-fix-scss-webroot-and-url-rewrite

[Stable12] fix scss webroot and url rewrite

Revert rebaseurl function edit

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

Rollback caching fix

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

Fix faulty path in theming scss generation

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

[tx-robot] updated from transifex

test adoption for stable12

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

Fixup! Proberly detect variable changes

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

Proberly detect variable changes

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

Fixed url rebase with theming

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

update autoloader

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

removes invalid tests, adapts another one

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

never translate login names when requiring with a user id

where appropriate, the preLoginNameUsedAsUserName hook should be thrown.

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

extend tests for status quo

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

[tx-robot] updated from transifex

Fixed tests

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

Fixed tests

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

Correctly remove temp test directory tree

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

Added tests for various installations types
- With root installation
 - Core css
 - App inside server root
 - Secondary apps directory outside server root
- With an installation in a sub directory
 - Core css
 - App inside server root
 - Secondary apps directory outside server root

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

Fixup! getWebdir return

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

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

Do no run SyncJob in cron

Fixes #7184

The SyncJob can be very resource intensive. Since it requests all users
on the system to create the system addressbook. In order to do this it
creates a vcard for every user and updates the addressbook.

There is no need for this job since the proper signals are emitted and
handled in the carddav backend to update the addressbook live.

Worst comes to worst there is always the occ command to bring the
address book in sync again.

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

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>

Fix drag shadow not visible when dragging a file on a narrow screen

When a file from the file list is dragged a drag shadow (a copy of the
file row that follows the cursor position) is created. The drag shadow
element is created as a direct child of the body element, so it needs a
higher "z-index" than the one used for the file list to be visible.

In narrow screens the "#app-content" uses a "z-index" of 1000 in order
to be visible over the "#navigation-bar" when they overlap, so the
"z-index" of the drag shadow must be at least 1000 to be visible over
the file list.

Instead of updating the hardcoded "z-index" it was removed and replaced
by CSS rules for ".dragshadow" elements to ease theming.

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

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