Format control structures, classes, methods and function

To continue this formatting madness, here's a tiny patch that adds
unified formatting for control structures like if and loops as well as
classes, their methods and anonymous functions. This basically forces
the constructs to start on the same line. This is not exactly what PSR2
wants, but I think we can have a few exceptions with "our" style. The
starting of braces on the same line is pracrically standard for our
code.

This also removes and empty lines from method/function bodies at the
beginning and end.

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

Merge pull request #20403 from nextcloud/techdebt/utf8-encoding

Use utf8 encoding everywhere

Merge pull request #20402 from nextcloud/techdebt/elseif

Use elseif instead of else if

Use utf8 encoding everywhere

PSR2 likes that and so do I

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

Use elseif instead of else if

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

Merge pull request #20256 from nextcloud/dependabot/composer/nikic/php-parser-4.3.0

Bump nikic/php-parser from 4.2.1 to 4.3.0

Merge pull request #20281 from nextcloud/bugfix/noid/use-primary-element-color

Use the primary element color in case it primary color is too bright

Use the primary element color in case it primary color is too bright

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Merge pull request #20313 from nextcloud/dependabot/npm_and_yarn/underscore-1.10.2

Bump underscore from 1.9.2 to 1.10.2

Merge pull request #20385 from nextcloud/skeeable-http-harden

harden seekable http stream a bit against failures

Merge pull request #20390 from nextcloud/refactor/no-closing-tag

Do not use closing tags

[tx-robot] updated from transifex

Merge pull request #20391 from nextcloud/refactor/spaces-cleanup

Remove all extra whitespace PSR2 does not like

Merge pull request #20384 from nextcloud/techdebt/lowercase-keywords

Use php keywords in lowercase

Remove space between switch case and colon

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

Make sure there is exactly one newline after imports

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

Make sure every file ends with an empty newline

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

Remove trailing whitespaces from comments

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

Remove trailing and in between spaces

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

Remove spaces after method or function call

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

Do not use closing tags

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

Merge pull request #20383 from nextcloud/techdebt/function-spacing

Unify function spacing to PSR2 recommendation

Merge pull request #20376 from nextcloud/fix/tags-ellipsis

Fix systemtags overflow

harden seekable http stream a bit against failures

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

Use php keywords in lowercase

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

Unify function spacing to PSR2 recommendation

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

Merge pull request #20355 from nextcloud/fix/comments/clear-input/19585

Clear comment on successful post

Merge pull request #20379 from nextcloud/techdebt/blank-lines

Use one blank line after the opening tag and namespace declaration

Use a blank line after the opening tag

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

Use exactly one empty line after the namespace declaration

For PSR2

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

Merge pull request #20375 from nextcloud/techdebt/fix-array-indent-style

Fix (array) indent style to always use one tab

Merge pull request #20303 from nextcloud/version/19/beta2

19 beta 2

Fix systemtags overflow

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Fix (array) indent style to always use one tab

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

Merge pull request #20368 from nextcloud/refactor/psr1

Migrate to PSR1 coding style

Merge pull request #20363 from nextcloud/smb-3.2.3

update icewind/smb to 3.2.3

[tx-robot] updated from transifex

Merge pull request #16035 from nextcloud/share-search-hide-on-match

dont show remote and email options if we have an exact match for local user email

Merge pull request #20370 from nextcloud/techdebt/fix-multiline-comments

Fix multiline comments

Fix multiline comments

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

Migrate to PSR1 coding style

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

Bump underscore from 1.9.2 to 1.10.2

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

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

Merge pull request #20321 from nextcloud/enh/noid/check-translation-for-triple-dot

Check for triple dots in translations in CI

Merge pull request #20360 from nextcloud/bugfix/20182/try-to-use-the-displayname-for-file-transfers

Try to use the display name of file transfers

Merge pull request #20319 from nextcloud/enh/noid/embedded-previews

Use embedded previews of ZIP-like files

update icewind/smb to 3.2.3

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

Merge pull request #20343 from nextcloud/bugfix/noid/themed-error-behaviour

Default to dark background in dark mode and better themed navbar

remove Windows config since it is not supported anyways

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

Bump autoloaders

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

Add preview provider for Krita files

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

Extract preview from opendocument files

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

Add abstract class to extract previews from files that are zip archives

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

dont show remote and email options if we have an exact match for local user email

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

Show the displayname instead of the user id

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

Try to use the display name of file transfers

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

Merge pull request #20315 from nextcloud/dependabot/npm_and_yarn/handlebars-4.7.6

Bump handlebars from 4.7.4 to 4.7.6

Bump handlebars from 4.7.4 to 4.7.6

Bumps [handlebars](https://github.com/wycats/handlebars.js) from 4.7.4 to 4.7.6.
- [Release notes](https://github.com/wycats/handlebars.js/releases)
- [Changelog](https://github.com/wycats/handlebars.js/blob/master/release-notes.md)
- [Commits](https://github.com/wycats/handlebars.js/compare/v4.7.4...v4.7.6)

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

Merge pull request #20299 from nextcloud/fix/noid/member-on-null-app-cancel-login

fixes the throttler not checking the user state on postLogin

Merge pull request #18200 from nextcloud/enhancement/psr-logger

Add a PSR-3 logger adapter and make it injectable

Bump nikic/php-parser from 4.2.1 to 4.3.0

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

Merge pull request #20347 from azul/test/fix-failing-acceptance

tests: fix failing header acceptance test

Clear comment on successful post

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

tests: fix failing header acceptance test

The username is listed in a div with class `fullname` now.

Signed-off-by: Azul <azul@riseup.net>

Merge pull request #20188 from nextcloud/bug/19662/add-default-title-for-event-in-invitatios

Add default titles for titleless events in invitations

Merge pull request #20265 from nextcloud/techdebt/legacy-classes-psr4

Move legacy classes from PSR0 to PSR4

Default to dark background in dark mode and better themed navbar

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

Move legacy classes from PSR0 to PSR4

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

Check for triple dots in translations in CI

Signed-off-by: Gary Kim <gary@garykim.dev>

[tx-robot] updated from transifex

Merge pull request #20033 from nextcloud/s3-seekable-stream

Enable fseek for files in S3 storage

Merge pull request #19346 from nextcloud/refactor/external-toast-deprecate

Use toasts from @nextcloud/dialogs, deprecate OCP.Toast and fix usages

Merge pull request #20331 from nextcloud/irc-to-talk

Remove freenode badges

Remove freenode badges, Add link to Talk

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

Remove toastify-js

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

Inline usages of OCP.Toast in bundled files

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

Use external Toast implementation and deprecate the OCP API

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

Merge pull request #20276 from nextcloud/files-external-param-tooltop

allow setting tooltips for external storage config options

Merge pull request #20310 from nextcloud/dependabot/npm_and_yarn/build/handlebars-4.7.6

Bump handlebars from 4.7.3 to 4.7.6 in /build

Merge pull request #20330 from nextcloud/dependabot/npm_and_yarn/nextcloud/l10n-1.2.2

Bump @nextcloud/l10n from 1.1.1 to 1.2.2

Bump @nextcloud/l10n from 1.1.1 to 1.2.2

Bumps [@nextcloud/l10n](https://github.com/nextcloud/nextcloud-l10n) from 1.1.1 to 1.2.2.
- [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.1.1...v1.2.2)

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

Merge pull request #19761 from nextcloud/bugfix/19658/add_missing_index_to_original_migration

Add missing index to original migration

[tx-robot] updated from transifex

Merge pull request #20314 from nextcloud/dependabot/npm_and_yarn/eslint-loader-3.0.4

Bump eslint-loader from 3.0.3 to 3.0.4

Merge pull request #20309 from nextcloud/rakekniven-patch-2

l10n: Triple dot to ellipsis.

Bump eslint-loader from 3.0.3 to 3.0.4

Bumps [eslint-loader](https://github.com/webpack-contrib/eslint-loader) from 3.0.3 to 3.0.4.
- [Release notes](https://github.com/webpack-contrib/eslint-loader/releases)
- [Changelog](https://github.com/webpack-contrib/eslint-loader/blob/master/CHANGELOG.md)
- [Commits](https://github.com/webpack-contrib/eslint-loader/compare/v3.0.3...v3.0.4)

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

Merge pull request #20316 from nextcloud/dependabot/npm_and_yarn/eslint-plugin-import-2.20.2

Bump eslint-plugin-import from 2.20.1 to 2.20.2

Merge pull request #20308 from nextcloud/rakekniven-patch-1

 l10n: Changed spelling of WebAuthn

Bump handlebars from 4.7.3 to 4.7.6 in /build

Bumps [handlebars](https://github.com/wycats/handlebars.js) from 4.7.3 to 4.7.6.
- [Release notes](https://github.com/wycats/handlebars.js/releases)
- [Changelog](https://github.com/wycats/handlebars.js/blob/master/release-notes.md)
- [Commits](https://github.com/wycats/handlebars.js/compare/v4.7.3...v4.7.6)

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

Merge pull request #20312 from nextcloud/dependabot/npm_and_yarn/jsdom-16.2.2

Bump jsdom from 16.2.1 to 16.2.2

Compile assets

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

[tx-robot] updated from transifex

Bump eslint-plugin-import from 2.20.1 to 2.20.2

Bumps [eslint-plugin-import](https://github.com/benmosher/eslint-plugin-import) from 2.20.1 to 2.20.2.
- [Release notes](https://github.com/benmosher/eslint-plugin-import/releases)
- [Changelog](https://github.com/benmosher/eslint-plugin-import/blob/master/CHANGELOG.md)
- [Commits](https://github.com/benmosher/eslint-plugin-import/compare/v2.20.1...v2.20.2)

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

Bump jsdom from 16.2.1 to 16.2.2

Bumps [jsdom](https://github.com/jsdom/jsdom) from 16.2.1 to 16.2.2.
- [Release notes](https://github.com/jsdom/jsdom/releases)
- [Changelog](https://github.com/jsdom/jsdom/blob/master/Changelog.md)
- [Commits](https://github.com/jsdom/jsdom/compare/16.2.1...16.2.2)

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

l10n: Triple dot to ellipsis.

Signed-off-by: rakekniven <mark.ziegler@rakekniven.de>

l10n: Changed spelling of WebAuthn

Signed-off-by: rakekniven <mark.ziegler@rakekniven.de>

l10n: Changed spelling of WebAuthn

Signed-off-by: rakekniven <mark.ziegler@rakekniven.de>

l10n: Changed spelling of WebAuthn

Signed-off-by: rakekniven <mark.ziegler@rakekniven.de>

19 beta 2

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

Merge pull request #20139 from nextcloud/unify-default-value-for-restricting-user-enumeration-with-settings

Unify default value for restricting user enumeration with settings