Merge pull request #3488 from WoltLab/external-link-handling

External link handling

Applied suggestions

Merge branch '5.2' into master

Fix PHP 8 compatibility for WCFSetup's error handler

see 0267fa9af7e18aa6449726f748e672cdac192d12

Applied suggestions

Merge pull request #3487 from WoltLab/improved-package-search-results

Improve results when searching for packages

Merge pull request #3477 from WoltLab/style-cleanup-update

Add update script for style cleanup

Added rel attribute for external links in menus

Add rel="ugc" for links within user generated content

isInternalURL() treats everything as internal that resides on the same subdomain

Removed EXTERNAL_LINK_REL_NOFOLLOW

New method to generate attributes for <a> tags

Use of StringUtil::getAnchorTag()

use StringUtil::getAnchorTag()

Merge pull request #3486 from Krymonota/use-generic-default-cookie-prefix

Use generic value for default cookie prefix

Fixed image path issue

Improve results when searching for packages

Closes #3407

Use generic value for default cookie prefix

... so that it doesn't have to be adjusted for new versions.

Merge branch '5.2'

Added missing informal variant

Merge pull request #3484 from WoltLab/wcfsetup-cookietest

Detect misconfigured hostnames during WCFSetup

Detect misconfigured hostnames during WCFSetup

Misconfigured reverse reverse proxies might rewrite the `host` header to the
upstream's hostname, instead of preserving the `host` as it was sent by the
web browser. Such a misconfiguration will cause WoltLab Suite to generate
incorrect absolute URLs and more importantly this also causes it to specify
an incorrect `domain` within cookies. The latter leads to the browser ignoring
the cookie. At the end of WCFSetup this ultimately leads to the ACP session
cookie being ignored, which in turn leads to failing the transition from
WCFSetup into the package installation. Instead the user will be bounced to
the LoginForm which fails to load, because the necessary option.xml was not
yet installed.

An example HAProxy configuration that reproduces the issue is as follows:

    listen test
     mode http
     bind *:80
     http-request set-header host 172.19.0.5
     server nginx 172.19.0.5:80

If the WCFSetup is accepted via any hostname that is not `172.19.0.5`, e.g.
by using `localhost` then cookies will fail to stick within the web browser.

This commit extends the system requirements step to:

- Compare the HTTP_HOST as seen by the web server against both:
   1) The `Referer` header.
   2) The `window.location.host` value in JavaScript.
  If any of those mismatches, then the web server is not correctly configured.
- Read a cookie that was set earlier.
  If this cookie is missing, then most likely the `domain` property was
  incorrectly specified.

This commit most likely resolves #3024.

Merge branch '5.2' into master

Merge branch '3.1' into 5.2

Fix PHP 5.5 compatibility

see b044815dd9b6509fc44219684d7076cd28a80aa6
see #3480

Merge branch '5.2'

Fix handling of hidden form field values via AJAX

See #3053

Do not hardcode the style's asset path in the update script

Co-authored-by: Alexander Ebert <ebert@woltlab.com>

Merge branch '5.2'

Merge branch '3.1' into 5.2

Merge pull request #3462 from SoftCreatR/patch-14

Add detection for Chromium based Edge browser

Merge branch '5.2'

Merge pull request #3471 from WoltLab/disable-spider-visit-tracking

Disable visit tracking for search engines

Merge branch '5.2' into master

Merge branch '3.1' into 5.2

Do not decrement wcf1_user.articles when deleting unpublished articles

see b044815dd9b6509fc44219684d7076cd28a80aa6
see #3480

Merge branch '5.2' into master

Merge branch '3.1' into 5.2

Update wcf1_user.articles when deleting articles

Fixes #3480

Merge pull request #3479 from WoltLab/jump-to-content

Rename "Mark as Read" button to "Jump To Content" in notification mails

Improved german phrasing

Rename "Mark as Read" button to "Jump To Content" in notification mails

Resolves #3257

Merge pull request #3475 from WoltLab/image-scale-memory

Add checkMemoryLimit() method to ImageAdapter

Prevent the incorrect focus of the close button for confirmation dialogs

Duplicate logo on import of pageLogo = pageLogoMobile

Fixes #3478

Add update script for style cleanup

Resolves #3468

Merge pull request #3456 from WoltLab/style-cleanup

Clean up asset handling of styles

Make use of ImageAdapter::checkMemoryLimit()

Add ImageAdapter::checkMemoryLimit()

Resolves #3229

Added detection for Chromium based Edge browser

Merge branch '5.2'

Add note, that the page.xml PIP instruction is needed with the next update
See #3474

Merge branch '3.1'

Fix updating `requireObjectID` for existing pages

Add all environment to event listener (#3145)

Merge pull request #3472 from WoltLab/user-rank-live-preview

User rank live preview

Add cache buster for pageLogo

Remove old pageLogo(|Mobile) file when logo is deleted or updated

Merge branch 'master' into style-cleanup

Missing semicolons

Support for custom block level tag names

Closes #3270

Prevent auto-focus on search suggestions

Closes #3263

Merge branch '5.2'

The `data-user-id` is already set by the ReactionHandler

Fixes #3463

Live preview for user ranks

Modernize the JavaScript for the live preview for labels

Inlining the JavaScript made it much easier plus prevents increasing the bundle size.

Improved detection of duplicate key errors

Prevent writes to the session rather than reads

Merge branch '5.2'

Disable visit tracking for search engines

Merge pull request #3470 from WoltLab/db-expose-driver-specific-code

Allow retrieving the driver specific error code from DatabaseQueryExe…

Allow retrieving the driver specific error code from DatabaseQueryExecutionException

A single ANSI SQLSTATE can indicate several distinct error conditions. The
driver code appears to be unique for MySQL.

Typo

Typo

Made use of the new method "createOrIgnore"

New method that will ignore duplicate key errors while inserting rows into the database

Revert "Whitelist `unserialize()` when running in enterprise mode"

`unserialize()` is unsafe, because it potentially allows for arbitrary
code execution.

This reverts commit 564ba8525e42c9d4677ee1ddac58c4c9c67fc113.

Merge pull request #3467 from WoltLab/avoid-getFont

Avoid use of getFont.php

Place the code and inline-code button next to each other

Closes #3241

Inject proper getFont() version depending on isMultiDomainSetup()

Use getFont SCSS function

Add getFont() SCSS helper function

Use filename parameter of getFont.php for FontAwesome

Merge branch '5.2'

Clear language cache when clearing cronjob cache

Close #3465

Fix Style::getCoverPhotoUrl()

Make sure to return an absolute URL.

Merge branch 'master' into style-cleanup

Merge pull request #3460 from WoltLab/ship-open-sans

Ship Open Sans by default

Prevent images in comments in the sidebar box from becoming too large

Remove Google Font Embed from ACP

Bake Open Sans into default installation

Resolves #3458

Unbreak WCFSetup

Expand style upload field definitions to store image size constraints

Fix preserveAspectRatio parameter name in ImageAdapters

Add missing PHPDoc tags to StyleAddForm / Style

DRY up handling of existing favicon in StyleEditForm

Co-authored-by: Alexander Ebert <ebert@woltlab.com>

Fix code style in StyleAddForm

Co-authored-by: Alexander Ebert <ebert@woltlab.com>

Fix copy and paste mistake deleting cover photos on Favicon update

Merge branch 'master' into style-cleanup

Merge pull request #3448 from WoltLab/local-google-font

Serve Google Fonts locally