Enhancement: Enable `trailing_comma_in_multiline` fixer (#647)

* Enhancement: Enable and configure trailing_comma_in_multiline fixer

* Fix: Run 'make coding-standards'

Enhancement: Enable `no_mixed_echo_print` fixer (#864)

Use `white-space: pre-wrap` for code formatting (#810)

* Use `white-space: pre-wrap` for code formatting

Apparently code formatting is used for the entire user comments. Use
`wh

Use `white-space: pre-wrap` for code formatting (#810)

* Use `white-space: pre-wrap` for code formatting

Apparently code formatting is used for the entire user comments. Use
`white-space: pre-wrap` to avoid horizontal scrolling of the plaintext
description.


see 7d4038829e601bfdff3c434f5d8f20d84b344e27

* Unify `highlight_php()` with PhD

see php/phd@01d6beb366bfb5e5b99bfdcc44393d8d1ee2d433

* Fix markup for notes in add-note.php

show more ...

Fix GH-712: link to mirrors is broken on confirmation page

The mirror program has been retired, so it makes no sense to mention
it. Still, user notes may take a while to be shown.

Introduce UserNote DTO and refactor Sorter (#594)

Enhancement: Enable `binary_operator_spaces` fixer

Closes GH-667.

Enhancement: Enable and configure `concat_space` fixer

Closes GH-657.

Enhancement: Enable `array_syntax` fixer

Co-authored-by: MathiasReker <mathias@reker.dk>

Closes GH-659.

Enhancement: Enable `constant_case` fixer

Co-authored-by: MathiasReker <mathias@reker.dk>

Closes GH-660.

Enhancement: Enable `trim_array_spaces` fixer

Closes GH-644.

Enhancement: Enable `single_space_after_construct` fixer

Closes GH-640.

HTML A11y improvements in add-note.php, my.php and vote-note.php

Also use strict comparison in pages that are changed.

Closes GH-610.

Multiple micro-optimizations

* Replace `ob_get_contents();ob_clean()` with `ob_get_clean()`

`ob_get_clean()` is equivalent to `ob_get_contents()` followed by `ob_clean()`.

Multiple micro-optimizations

* Replace `ob_get_contents();ob_clean()` with `ob_get_clean()`

`ob_get_clean()` is equivalent to `ob_get_contents()` followed by `ob_clean()`.

* Replace `intval()` calls with `(int)` type cast

This is a micro-optimization because `intval()` is a function call, and the type cast is about 6 times fast.

* Replace `preg_replace` call that could be done with an `rtrim()` call

In `./error.php`, there is a `preg_replace('!/+$!', '', $URI);` call that essentially is equivalent to `rtrim()`, that both calls removing trailing slash characters in `$URI`.
The `rtim()` call is more legible and faster.

* Combine consecutive `str_replace` calls to a single `str_replace` call

* Use short ternary operator where possible

Improves code readability.

* Cascade various `else` statements where possible

Cleans up the code by removing unnecessary `else` blocks and moving the code to the parent context if the previous `if` block exits the function by either terminating the script, or with a `return` statement.

* Combine multiple `isset()` calls to a single `isset()`

`isset()` accepts multiple parameters and returns `true` only if all of the parameters are `isset`. It makes sense to combine multiple individual `isset` calls to a single call for better readability.

* Replace `for` loop with a `foreach` loop

* Remove unnecessary character escapes in regular expressions

Regular expression special characters are context-sensitive. For example, special characters such as `.` are not considered special within square braces (`[]`).
This removes several of such instances that certain characters are escaped, but it is not strictly necessary within the context. This improves the readability of the expression.

See more information at [PHP.Watch: Writing better Regular Expressions in PHP](https://php.watch/articles/php-regex-readability#reduce-escape)

* Remove unnecessary break statement

* Remove unnecessary PHP close tags

* Remove redundant JSON_ERROR_NONE check

Remove unnecessary `json_last_error() == JSON_ERROR_NONE` where the decoded object is inspected already.

Closes GH-603.

show more ...

Fix: Remove extra closing tag

Closes GH-589.

Use elseif instead of else if

Closes GH-534.

Ternary to null coalescing

Use null coalescing operator ?? where possible.

Closes GH-528.

Replace master -> main in more places

Goodbye Magic Quotes

Swap uses of _SERVER[DOCUMENT_ROOT] for __DIR__

2/4 - Update remaining except archives/ and release/.

Trim trailing whitespace

This patch cleans all redundant trailing whitespace across the
repository except for the icalendar files.

Remove unused Git attributes ident

The $Id$ keywords were used in Subversion where they can be substituted
with filename, last revision number change, last changed date, and last
use

Remove unused Git attributes ident

The $Id$ keywords were used in Subversion where they can be substituted
with filename, last revision number change, last changed date, and last
user who changed it.

In Git this functionality is different and can be done with Git attribute
ident. These need to be defined manually for each file in the
.gitattributes file and are afterwards replaced with 40-character
hexadecimal blob object name which is based only on the particular file
contents.

This patch simplifies handling of $Id$ keywords by removing them since
they are not used anymore.

show more ...

Revert "Replace <small> with <span style"

This reverts commit 12f04c0d4a005342080eacf8eb5440037cd3288d.

Replace <small> with <span style

Replace <small> with <span style="font-size:0.8em"> because the
definition of <small> is a complete other with HTML5

Replace <tt> with <code>

The <tt>-tag doesn't exists anymore in HTML5

Kill off XHTML

Long live HTML5! =)