Fix undefined class property access after upgrade from 19 to 20

The serialized data in 19 has one property less and this was not
considered in the code. Hence adding a fallback. Moreover I'm changing
the deserialization into an array instead of object, as that is the
safer option.

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

Merge pull request #22589 from nextcloud/bugfix/noid/allow-additional-personal-settings-via-normal-registration

Allow additional personal settings via normal registration

Merge pull request #22637 from nextcloud/bugfix/noid/fix-css-selector-for-status-icon

Fix weather status icon selector with nextcloud/vue update

Merge pull request #22617 from nextcloud/dependabot/npm_and_yarn/babel/core-7.11.6

Bump @babel/core from 7.11.4 to 7.11.6

Fix weather status icon selector with nextcloud/vue update

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

Bump @babel/core from 7.11.4 to 7.11.6

Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.11.4 to 7.11.6.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.11.6/packages/babel-core)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Merge pull request #22136 from nextcloud/design/reasons-to-use-nextcloud

Add 'Reasons to use Nextcloud in your organization' call to action in settings

Merge pull request #21945 from nextcloud/fix-contact-download-picture

Changed Content-Disposition header to download Contact Photo with correct extension

Add 'Reasons to use Nextcloud in your organization' call to action in settings

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

[tx-robot] updated from transifex

Merge pull request #22583 from nextcloud/fix/noid/remove-unused-code

LDAP: remove unused methods and DB values

Merge pull request #22605 from nextcloud-pr-bot/automated/noid/psalm-baseline-update

[Automated] Update psalm-baseline.xml

Update psalm baseline

Signed-off-by: GitHub <noreply@github.com>

[tx-robot] updated from transifex

Change Content-Disposition Header

- Make ALLOWED_CONTENT_TYPES public in order to use
- Add $fileName variable which uses "$node->getName()" to get the proper file name and "$file->getMimeType()" along with the ALLOWED_CONTENT_TYPES array in PhotoCache.php to get the proper file extension
- Make "$fileName" the name of the file in the Content-Disposition header when downloading a Contact's photo
- Add filename to the CardDAV integration image export test header
- Change headers in ImageExportPluginTest to reflect changes

Signed-off-by: Jacob Neplokh <me@jacobneplokh.com>

[tx-robot] updated from transifex

Merge pull request #22531 from nextcloud/fix/save-statuses

Fix dashboard statuses toggling

Merge pull request #22597 from nextcloud/techdebt/noid/allow-to-di-the-settings-manager

Add missing alias for OCP\Settings\IManager and deprecate the old one

Merge pull request #17456 from brad2014/feature/brad2014/12391-improve-imip-mail-message-take-2

Merge pull request #22543 from nextcloud/bugfix/noid/dont-keep-menu-open-for-next-user-after-delete

Don't keep menu open for next user after deleting one

Merge pull request #22596 from nextcloud/build-weather-status

Build weather-status to solve design issues

build weather-status to solve design issues

Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

refs #22468 fix empty php array becoming an array instead of an object in UI

Signed-off-by: Julien Veyssier <eneiluj@posteo.net>

Add missing alias for OCP\Settings\IManager and deprecate the old one

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

Don't keep menu open for next user after deleting one

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

Merge pull request #22577 from nextcloud/bugfix/noid/better-error-when-chunk-assembling-is-denied

Better error message when blocked by access control

Allow additional personal settings via normal registration

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

Merge pull request #22573 from nextcloud/bugfix/noid/tooltip-dashboard-firstrun

Remove tooltip on customize button in dashboard

add repair step to clean up DB off lastFeatureRefresh entries in user prefs

- also removes related app setting "updateAttributesInterval"

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

these code bits were part of old logic that was already refactored out

- only references were in unit tests

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

Merge pull request #22540 from nextcloud/bugfix/noid/fix_status_nc2.6_regressions

Adapt status menu styles to nc/vue 2.6

Merge pull request #22578 from nextcloud/up/files_external/smb

Upgrade icewind/smb to 3.2.7

Merge pull request #22550 from nextcloud/bugfix/13556/use-correct-root-when-compiling-scss

Use the correct root to determinate the webroot for the resource

Upgrade icewind/smb to 3.2.7

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

Better error message when blocked by access control

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

Remove tooltip on customize button in dashboard

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

[tx-robot] updated from transifex

Merge pull request #21082 from nextcloud/enh/noid/apps-settings

Show changelog in apps management

Merge pull request #22524 from nextcloud/fix/apppassword_update

Fix app password updating out of bounds

Show changelog in apps management

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Fix app password updating out of bounds

When your password changes out of bounds your Nextcloud tokens will
become invalid. There is no real way around that. However we should make
sure that if you successfully log in again your passwords are all
updates

* Added event listener to the PostLoggedInEvent so that we can act on it
  - Only if it is not a token login
* Make sure that we actually reset the invalid state when we update a
  token. Else it keeps being marked invalid and thus not used.

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

Fix status menu styles after nc/vue 2.6 upgrade

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

Merge pull request #22455 from nextcloud/version/20.0.0/beta3

20 Beta3

Merge pull request #22556 from nextcloud/bugfix/noid/cleanup-part-file

Do not keep the part file if the forbidden exception has no retry set

Merge pull request #22557 from nextcloud/bugfix/noid/group-share-failure

Do not fail if share for mountpoint is no longer available

Do not fail if share for mountpoint is no longer available

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

Do not keep the part file if the forbidden exception has retry set to false

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

Merge pull request #22111 from nextcloud/enh/unified-search-filters

Merge pull request #22528 from nextcloud/bug/22475/trash-bin-size

Change free space calculation

Merge pull request #22545 from nextcloud-pr-bot/automated/noid/psalm-baseline-update

[Automated] Update psalm-baseline.xml

Adjust acceptance tests to changes in @nextcloud/vue 2.6.3

The menu button and the menu of a share are no longer direct childs of
the actions of the share row. The menu button is now a child of a
".trigger" element, while the menu is a direct child of the body and has
an id defined in the "aria-describedby" attribute of the ".trigger"
element.

In XPath 1.0 it does not seem possible to "backreference" a value or
create variables, so when the share menu or one of its item is needed
now the ".trigger" element is first found and then its XPath expression
is used to compose its "aria-describedby" attribute in the XPath
expression for the menu.

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

Bump @nextcloud/vue to v2.6.3

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

Fix search placeholder animation & dark theme compatibility

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

Allow unified search filtering

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

Update psalm baseline

Signed-off-by: GitHub <noreply@github.com>

[tx-robot] updated from transifex

Minor cleanup:  php-cs-fixer, tests, interface consistency

IMipPlugin.php Removed blank lines to make php-cs-fixer happy.
Minor cleanup:  bugs found by Psalm static checker
IEMailTemplate: The public interface to addBodyListItem also needs to include the new plainIndent parameter.
IMipPlugin: Fixes an undefined variable for events that do not have DTEND.  Also use explicit string conversion for parameters and properties in several places.

The new email template adds an additional blank line before "button" links in plain text, so the tests were fixed to include that additional blank line.

Signed-off-by: Brad Rubenstein <brad@wbr.tech>

Use the correct root to determinate the webroot for the resource

Since all the compiled routes are based on the server webroot,
we have to use this, independent from which app this belongs to.

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

Merge pull request #22526 from nextcloud/feature/unified-search/emit-query

Emit unified search query

Emit unified search query

Signed-off-by: Raimund Schlüßler <raimund.schluessler@mailbox.org>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

Merge pull request #22423 from nextcloud/bugfix/noid/direct-editing-encryption

Do not expose direct editing if no master key is available

Merge pull request #22535 from nextcloud/fix/zipstreamer_external_storage

Only disable zip64 if the size is known

Only disable zip64 if the size is known

Else we might run into the issue that for an external storage where the
size is not known yet we do not use zip64. Which then of course fails on
large zip files.

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

Merge pull request #22533 from nextcloud/fix/noid/rm-matcher-magic

Flow: do not hide "matches" and "does not match" checkers

Do not expose direct editing if no master key is available

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

Tests only: Forward calls to getUserValue and getAppValue

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

do not hide "matches" and "does not match" checkers

- they show up previously with / typed – end user needs to know about it
- inconsistent with some other fields, that show it directly

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

Merge pull request #22491 from nextcloud/dependabot/npm_and_yarn/dompurify-2.0.14

Merge pull request #22116 from nextcloud/bugfix/noid/transfer-ownership-share-root

Fix share transfer of single files and on the transfered node

Merge pull request #16632 from nextcloud/bugfix/external-reshare

Set proper root path for single file shares originating from other storages

Bump dompurify from 2.0.12 to 2.0.14

Bumps [dompurify](https://github.com/cure53/DOMPurify) from 2.0.12 to 2.0.14.
- [Release notes](https://github.com/cure53/DOMPurify/releases)
- [Commits](https://github.com/cure53/DOMPurify/compare/2.0.12...2.0.14)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

Change free space calculation

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

Merge pull request #22514 from nextcloud/fix/s3/only_delete_new_failed

Fix S3 error handling

Merge pull request #22424 from nextcloud/feature/noid/dnd_will_mute_notifications

User Status: Display hint that DND mutes all notifications

Merge pull request #22517 from nextcloud/external-mount-id-before-wrapper

set the mount id before calling storage wrapper

User Status: Display hint that DND mutes all notifications

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

Merge pull request #22494 from nextcloud/dependabot/npm_and_yarn/underscore-1.11.0

set the mount id before calling storage wrapper

this allows the storage wrapper to use the mount id for it's own logic

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

Bump underscore from 1.10.2 to 1.11.0

Bumps [underscore](https://github.com/jashkenas/underscore) from 1.10.2 to 1.11.0.
- [Release notes](https://github.com/jashkenas/underscore/releases)
- [Commits](https://github.com/jashkenas/underscore/compare/1.10.2...1.11.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Merge pull request #22459 from nextcloud/dependabot/npm_and_yarn/nextcloud/dialogs-2.0.1

Merge pull request #22472 from nextcloud/fix/contacts-interaction-blob-postgres

Fix writing BLOBs to postgres with recent contacts interaction

Transfer shares of the transferred root node

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

Only update the filecache entry once the file has been written to S3

If we already update before we have no way to revert if the upload
fails.

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

Don't lose filecache entry on s3 overwrite error

If the object store errors we should not always delete the filecache
entry. As this might lead to people losing access to their files.

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

Bump @nextcloud/dialogs from 1.4.0 to 2.0.1

Bumps [@nextcloud/dialogs](https://github.com/nextcloud/nextcloud-dialogs) from 1.4.0 to 2.0.1.
- [Release notes](https://github.com/nextcloud/nextcloud-dialogs/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-dialogs/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-dialogs/compare/v1.4.0...v2.0.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Merge pull request #22511 from nextcloud/fix/oca-search-monkey-patch

Declare OCA.Search  directly, not via a monkey patch

Declare OCA.Search  directly, not via a monkey patch

Apps could use OCA.Search very early on a page load without a problem
prior to Nextcloud 20 (current master). Since the unified search work
moved some code around, it now was added later, resulting in `TypeError:
OCA.Search is not a constructor` errors if the apps did not wrap their
calls in a `DOMLoaded` event handler.

To ensure existing code continues to work without modification, this
patch moves the declaration of the `OCA.Search` API to where we defined
`OCA`, instead of monkey-patching it later on.

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

Merge pull request #22502 from nextcloud/fix/12302/oauth2_table

Pimp Oauth2 table

Pimp Oauth2 table

Fixes #12302

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

[tx-robot] updated from transifex

Merge pull request #22476 from nextcloud/setting-sections-unique

ignore duplicate setting sections

Merge pull request #22481 from nextcloud/fix-clicks-on-actions-menu-of-non-opaque-file-rows-in-acceptance-tests

Fix clicks on actions menu of non opaque file rows in acceptance tests

Merge pull request #22490 from nextcloud/dependabot/npm_and_yarn/webpack-node-externals-2.5.2

Bump webpack-node-externals from 2.5.1 to 2.5.2

[tx-robot] updated from transifex

Bump webpack-node-externals from 2.5.1 to 2.5.2

Bumps [webpack-node-externals](https://github.com/liady/webpack-node-externals) from 2.5.1 to 2.5.2.
- [Release notes](https://github.com/liady/webpack-node-externals/releases)
- [Commits](https://github.com/liady/webpack-node-externals/compare/v2.5.1...v2.5.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

[tx-robot] updated from transifex

Fix clicks on actions menu of non opaque file rows in acceptance tests

When a row is added to the file list the opacity of the file row is
animated from transparent to fully opaque. As the file actions menu is a
descendant of the row but overflows it when the row is not fully opaque
clicks on the menu entries "fall-through" and are received instead by
the rows behind.

The opacity animation is a CSS animation, and it is not possible to know
if a row is appearing or not except from its opacity (the row will have
the "appear" CSS class even after the animation ended). Therefore it
should be waited until the row of the file is fully opaque before using
the menu.

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

ignore duplicate setting sections

this prevents some 'Section with the same ID already registered' errors in the log

also includes an improvement of the error message to make other cases easier to find

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