Fix HTML entities not decoded in comment just added

The XML data received from the comments endpoint has an inconsistent
encoding; some entities are encoded once and others are encoded twice.
When the comment list is loaded the comments are fetched using
GetComments, which handles all that, and therefore shows the messages
and author names as expected.

However, when a new comment is posted the list is not got again; instead
the new comment is loaded from the comment data returned after posting
it. This is done in NewComment, which did not decode the messages nor
the author names, and therefore showed, for example, "&" instead of
"&".

To solve that now the same decoding logic used in GetComments is applied
too in NewComment.

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

Extract "decodeHtmlEntities" function to its own file

This will make possible to use it from files other than
"GetComments.js".

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

Merge pull request #34613 from nextcloud/enh/noid/fix-checkmark-icon

fix checkmark for theme settings in dark mode

Merge pull request #32211 from nextcloud/feat/repair_share_ownership

Add repair command to fix wrong share ownership

update autoloader

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

fix repairing non user shares when repairing all shares

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

remove no longer needed code

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

split repairing into two stages to prevent long open transaction

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

update shares directly in db

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

More verbose output for repair share ownership cmd

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

move share owner repair to occ command

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

Add repair step to fix wrong share ownership

Signed-off-by: Louis Chemineau <louis@chmn.me>

Merge pull request #35028 from nextcloud/fix/dns-pin-middleware-throws-public-ip

Fix DNS Pin Middleware throwing for public IPs

Compile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix checkmark for theme settings in dark mode

Signed-off-by: szaimen <szaimen@e.mail.de>

Fix DNS Pin Middleware throwing for public IPs

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

Merge pull request #32838 from markuszeller/bugfix/32837/implicitFloatConversion

Cast to int

Merge pull request #34985 from nextcloud/dependabot/npm_and_yarn/nextcloud/l10n-1.6.0

Bump @nextcloud/l10n from 1.4.1 to 1.6.0

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Bump @nextcloud/l10n from 1.4.1 to 1.6.0

Bumps [@nextcloud/l10n](https://github.com/nextcloud/nextcloud-l10n) from 1.4.1 to 1.6.0.
- [Release notes](https://github.com/nextcloud/nextcloud-l10n/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-l10n/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-l10n/compare/v1.4.1...v1.6.0)

---
updated-dependencies:
- dependency-name: "@nextcloud/l10n"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

Merge pull request #34986 from nextcloud/dependabot/npm_and_yarn/handlebars-loader-1.7.2

Bump handlebars-loader from 1.7.1 to 1.7.2

Cast to int #32837

Signed-off-by: Markus Zeller <git@markuszeller.com>

Merge pull request #35007 from nextcloud/fix/lower-logging-level-user-session-cookie-login

Fix unsuccessful token login logged as error

Merge pull request #35000 from nextcloud/bugfix/34959/fix-cron-warning-with-time-insensitive-jobs

Fix cron warning with time insensitive jobs

Fix unsuccessful token login logged as error

The condition of a non-existent login token can happen for concurrent
requests. Admins can not do anything about this. So this is to be
expected to happen occasionally. This event is only bad if none of the
requests is able to re-acquire a session. Luckily this happens rarely.

If a login loop persists an admin can still lower the log level to find
this info. But a default error log level will no longer write those
infos about the failed cookie login of one request.

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

Merge pull request #34996 from nextcloud/fix/bump-microsoft/azure-storage-blob-to-1.5.4

Bump microsoft/azure-storage-blob to 1.5.4

Merge pull request #32530 from nextcloud/dependabot/npm_and_yarn/nextcloud-vue-collections-0.10.0

Bump nextcloud-vue-collections from 0.9.0 to 0.10.0

Merge pull request #34929 from nextcloud/chore/deprecate-app-container-aliases

Clean up and deprecate app container parameter aliases

Align with 3rdparty master

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Recompile

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

Increase the "bad waiting time" for cron mode to 24h to account for "time insensitive" jobs

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

Show the correct warning for system cron vs. webcron/ajax

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

Merge pull request #34937 from nextcloud/fix/do-not-crash-on-extension-without-version

Avoid a crash when a PHP extension has no version

Bump microsoft/azure-storage-blob to 1.5.4

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Suppress false-positives from psalm, waiting for fix upstream

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #34987 from nextcloud/automated/noid/master-update-psalm-baseline

[master] Update psalm-baseline.xml

Update psalm baseline

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

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #18210 from dbw9580/master

Limit number of concurrent preview generations

Limit-number-of-concurrent-preview-generations

Signed-off-by: Bowen Ding <dbw9580@live.com>
Signed-off-by: szaimen <szaimen@e.mail.de>

Bump handlebars-loader from 1.7.1 to 1.7.2

Bumps [handlebars-loader](https://github.com/pcardune/handlebars-loader) from 1.7.1 to 1.7.2.
- [Release notes](https://github.com/pcardune/handlebars-loader/releases)
- [Changelog](https://github.com/pcardune/handlebars-loader/blob/main/CHANGELOG.md)
- [Commits](https://github.com/pcardune/handlebars-loader/compare/v1.7.1...v1.7.2)

---
updated-dependencies:
- dependency-name: handlebars-loader
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

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

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #34775 from nextcloud/bugfix/noid/mailer-set-to

Do not throw errors when invalid setTo email is provided

Merge pull request #33181 from nextcloud/dependabot/composer/build/integration/behat/behat-approx-3.11.0

Merge pull request #34702 from nextcloud/fix/clean-ldap-wizard-typing

Clean up LDAP Wizard typing

Merge pull request #34834 from nextcloud/bugfix/noid/skeleton-size-calc

Fix size calculation on copying the skeleton files

Merge pull request #34909 from nextcloud/fix/21370/duplicate-notifications-2

Fix duplicate event email notifications

Bump nextcloud-vue-collections from 0.9.0 to 0.10.0

Bumps [nextcloud-vue-collections](https://github.com/juliushaertl/nextcloud-vue-collections) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/juliushaertl/nextcloud-vue-collections/releases)
- [Commits](https://github.com/juliushaertl/nextcloud-vue-collections/compare/v0.9.0...v0.10.0)

---
updated-dependencies:
- dependency-name: nextcloud-vue-collections
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

Update behat/behat requirement in /build/integration

Updates the requirements on [behat/behat](https://github.com/Behat/Behat) to permit the latest version.
- [Release notes](https://github.com/Behat/Behat/releases)
- [Changelog](https://github.com/Behat/Behat/blob/master/CHANGELOG.md)
- [Commits](https://github.com/Behat/Behat/compare/v3.10.0...v3.11.0)

---
updated-dependencies:
- dependency-name: behat/behat
  dependency-type: direct:development
...

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

Merge pull request #33884 from nextcloud/dependabot/composer/build/integration/guzzlehttp/guzzle-7.5.0

Bump guzzlehttp/guzzle from 7.4.5 to 7.5.0 in /build/integration

Fix duplicate event email notifications

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>

Merge pull request #34963 from nextcloud/followup/34522/fix-invalid-date

Fix invalid date in security.txt

Fix invalid date in security.txt

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

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #34954 from nextcloud/bugfix/noid/autoloader-update-imip

Dump autoloaders for new IHandleImipMessage

Merge pull request #34934 from nextcloud/bugfix/noid/session-reopen-read

Read encrypted session data again on reopen

Dump autoloaders for new IHandleImipMessage

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

Do not throw errors when invalid setTo email is provided

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

Merge pull request #34678 from pulsejet/patch-jpeg

Interlace all output JPEG

Merge pull request #34893 from nextcloud/fix/move-imip-to-separate-interface

Move iMIP to its own interface

Merge pull request #34330 from nextcloud/fix/fix-contacts-test-php8

Fix ContactsStoreTests under PHP 8.2

Merge pull request #32565 from nextcloud/chore/modernize-error-handler

Modernize the error handler

Merge pull request #34933 from nextcloud/automated/noid/master-update-psalm-baseline

[master] Update psalm-baseline.xml

Merge pull request #33605 from nextcloud/perf/user-getdisplayname-cache

Use cached getDisplayName method

Do not remove complete encrypted session key when just a key should be removed

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

Read encrypted session data again on reopen

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

Avoid a crash when a PHP extension has no version

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #34852 from nextcloud/enhancement/host-validator-api

Add remote host validation API

Switch to DisplayNameCache for CardDAV

Signed-off-by: Anna Larch <anna@nextcloud.com>

Switch to DisplayNameCache for CalDAV

Signed-off-by: Anna Larch <anna@nextcloud.com>

Move iMIP to it's own interface

and clean up the code a bit.

Added:
- author

Removed:
- unused imports

Changed:
- moved iMIP handling to new interface
- pointed code to new implementation

Signed-off-by: Anna Larch <anna@nextcloud.com>

Update psalm baseline

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

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Clean up and deprecate app container aliases

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

Merge pull request #34868 from nextcloud/fix/toasts-styling

Update @nextcloud/dialogs and fix toasts styling

Update @nextcloud/dialogs@ and fix toasts styling

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

Merge pull request #34880 from nextcloud/dependabot/npm_and_yarn/babel/node-7.20.0

Bump @babel/node from 7.17.10 to 7.20.0

Merge pull request #34907 from nextcloud/enh/noid/remove-32bit-compatibilitiy

remove 32-bit workarounds

Merge pull request #34871 from nextcloud/bugfix/34695/fix-internal-expiredate-visibility

Fix visibility of internal expire date

Merge pull request #34908 from nextcloud/enh/noid/disable-26-updates-32bit

disable upgrades to 26 for 32-bit

fix psalm

Signed-off-by: szaimen <szaimen@e.mail.de>

remove 32-bit workarounds

Signed-off-by: szaimen <szaimen@e.mail.de>

Merge pull request #34906 from nextcloud/enh/noid/improve-64-bit-check

Log all deprecations with debug level

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

Refactor the ErrorHandler into a dynamic class

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

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

disable upgrades to 26 for 32-bit

Signed-off-by: szaimen <szaimen@e.mail.de>

update 64-bit check wording

Signed-off-by: szaimen <szaimen@e.mail.de>

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Bump @babel/node from 7.17.10 to 7.20.0

Bumps [@babel/node](https://github.com/babel/babel/tree/HEAD/packages/babel-node) from 7.17.10 to 7.20.0.
- [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.20.0/packages/babel-node)

---
updated-dependencies:
- dependency-name: "@babel/node"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

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

Add remote host validation API

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

Merge pull request #34894 from nextcloud/enhancement/talk-api-delete-conversation

Add Talk converstation delete API

Merge pull request #34895 from nextcloud/fix/build-api-reference

Use the official phpDocumentor release

Use the official phpDocumentor release

The previous cached version download URL was broken and only a
non-functional version was downloaded leading and so the API reference
was not updated since NC24.

This is fixed by simply using the latest stable phpDocumentor release.

Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>

Add Talk converstation delete API

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

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

[tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #34884 from nextcloud/automated/noid/master-update-psalm-baseline

[master] Update psalm-baseline.xml

Merge pull request #34883 from nextcloud/dependabot/composer/build/integration/phpunit/phpunit-9.5.26