Merge pull request #3412 from WoltLab/list-unsubscribe

Add List-Unsubscribe header

Fix validating style when editing users in ACP

See #3254

Merge branch '5.2'

Add missing closing quotation mark in English language item

Correctly handle non-numeric exception codes in core.functions.php

Database exceptions use the SQLSTATE as the error code. SQLSTATEs can
contains letters.

Merge pull request #3421 from WoltLab/void-pip

Add <void/> instruction

Merge branch '5.2'

Resolve review requests for List-Unsubscribe

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

Merge pull request #3413 from WoltLab/existingMapping

Improve wording regarding existing import mapping

Updating minified JavaScript files

Merge branch '5.2'

Fix formatting of text/plain notifications

see #325

Merge pull request #3424 from WoltLab/resizer-strip-exif

Strip exif information from loaded image in Resizer#loadFile

Strip exif information from loaded image in Resizer#loadFile

Modern browsers take the Exif orientation into account when showing a JPEG
within an HTMLImageElement. Unfortunately this orientation is not only
visual, but extends to the blob received when reading this image into a
canvas.

The JavaScript based image resizer using within the attachment system takes
care reinsert the original Exif data after fetching the resized blob from
pica.js.

This causes the image to be reoriented multiple times, ultimately leading
to an incorrectly oriented image:

1. The browser rotates the image.
2. The server rotates the image again, because the original Exif information
   has been preserved.

To fix this issue we strip the Exif information before handing the blob over
to the HTMLImageElement, forcing the browser to use the raw pixels instead
of pretending to be smart. When the Exif information is reinserted after
resizing the image that will be uploaded will then be reoriented only once:
On the server.

During fixing of this bug it was also investigated whether one can find out
whether the browser reoriented the image, it looks like one cannot. It was
also tested whether setting `image-orientation: none` will have any effect.

It only has in Firefox: When image-orientation: none is set you will get
the behavior as if no Exif information is present.
In Chrome the source image will not be be reoriented when rendered inside
of the DOM. Reading the pixel values however still returns the reoriented
garbage.

Thus stripping the exif information is the best solution to combat web
browsers attempting to be smart. Unfortunately it comes with an increased
processing requirement, because the raw blob (possible multiple megabytes)
will need to be processed to strip the Exif data.

Fix formatting of wcf.acp.user.activation.mail.plaintext

Fix layout of text/plain version of userRegistration notification mail

Whitespace is significant within the text/plain version, thus remove the
tabs like within all the other notifications.

Merge pull request #3423 from WoltLab/deprecate-uploadhandler-isimage

Deprecation of UploadHandler::isValidImage()

Merge branch '5.2'

Simplify checking of existing of conditions on revoking trophies
See #3422

Incorrect check for legacy attachment dimensions

Deprecation of UploadHandler::isValidImage()
Resolves #3380

Merge branch '5.2'

Fix revoking trophies w/o conditions yield an SQL error
Fixes #3422

Add <void/> instruction

The <void/> instruction may only be used during updates and only when no
other package installation plugin is used. It will result in a noop
when running the upgrade, allowing an upgrade to adjust package metadata
without actually changing anything.

Example usage:

<instructions type="update" fromversion="*">
<void/>
</instructions>

Resolves #3411

Merge branch '5.2'

Merge branch '3.1' into 5.2

Merge pull request #3418 from WoltLab/cronjob-fix

Fix detection of stuck cronjobs

Merge pull request #3416 from WoltLab/upload-acceptable

Allow specifying allowed file extensions for form builder's UploadFormField

Support setting user's style when editing user in ACP

Close #3254

Get rid of useless class attribute CronjobScheduler::$cronjobEditors

Just use a simple return value here. It avoids having to carefully manage
class state.

Fix detection of stuck cronjobs

Consider the following scenario:

A cronjob is scheduled to run `0 * * * *` (i.e. every hour). The cronjob
successfully executes it's 23:00 execution belated on 23:30. Then the
nextExec will be 00:00. afterNextExec will be 01:00.

During night time nothing happens within the community, until at 05:xx when
the first user wakes up and opens their web browser. It restores two tabs
with our community and fetches them. Now the following will happen:

1. Request 1: Notices that the cronjob is due (00:00 < 05:xx). Setting
              the cronjob's state to pending and starting execution.
2. Request 2: Notices that the cronjob is overdue (01:00 < 05:xx,
              state = pending). Logging an error about a stuck job and
              setting the cronjob's state to ready.
3. Request 1: Finishes executing the cronjob and updates nextExec = 06:00,
              afterNextExec = 07:00, also setting the cronjob's state to
              ready.

Fix this issue by updating the times for the next executions together with
the update of the state in a single database query / transaction. This will
prevent other requests from seeing the cronjob with state = pending or
state = executing and an way outdated date.

A side effect of this change is that cronjobs will match the intended
schedule better.

Consider the following:

A cronjob running every minute, taking 5 seconds to execute. The execution
is triggered at 00:00:58. It will finish executing 00:01:03. Previously the
nextExec would be set to 00:02:00, now it will be 00:01:00. Thus the first
request after 00:01:03 (i.e. once the state is set back to ready) will
trigger execution, more closely matching the intended schedule.

Improve error message for stuck cronjobs

Use consistent wording 'acceptable' in Upload.js

Add support for acceptable types to UploadFormField of form builder

Resolves #3414

Add support for acceptable types to lib/system/file/upload/ functionality

see #3414

Add support for acceptable types to WoltLabSuite/Core/Upload

see #3414

Merge branch 'registerActivationMethodByUserAndAdmin'

Merge pull request #3323 from WoltLab/registerActivationMethodByUserAndAdmin

Register activation method by user and admin

Improve wording regarding existing import mapping

Resolves WoltLab/com.woltlab.wcf.exporter#43

Add List-Unsubscribe header

Resolves #3379

Redirect NotificationDisableAction to NotificationUnsubscribeForm

see #3379

Add NotificationUnsubscribeForm

see #3379

Add UserNotificationEvent::getObjectType()

Add support for the List-Unsubscribe header to Email

see #3379

Add List-ID header to notification mails

see #3379

Add support for the List-ID header to Email

see #3379

Use `RegisterNewActivationCode` as link for need email confirmation lang var

Fixed documentation issue

New version number: 5.3.0 Alpha 1

Merge branch '5.2'

Merge pull request #3397 from WoltLab/media-caption-width

Limit media images' caption size to image width

Merge pull request #3396 from WoltLab/composer-deps

Update composer dependencies

Merge pull request #3409 from WoltLab/php8-handleError

Fix PHP 8 compatibility for WCF::handleError()

Fix PHP 8 compatibility for WCF::handleError()

Quoting from the UPGRADING manual (https://github.com/php/php-src/blob/php-8.0.0alpha1/UPGRADING):

> The @ operator will no longer silence fatal errors (E_ERROR, E_CORE_ERROR,
> E_COMPILE_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR, E_PARSE). Error handlers
> that expect error_reporting to be 0 when @ is used, should be adjusted to
> use a mask check instead:

Merge branch '5.2'

Merge branch '3.1' into 5.2

Hide empty tag box on tagged page

Move sidebar on the TaggedPage to the right

Close  #3377

Merge branch '5.2'

Fix validation of url form field value

Close #3390

Release 5.2.7

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Merge pull request #3374 from WoltLab/exception-log-dedup

Deduplicate Exceptions in ExceptionLogView

Release 3.1.15

Updating minified JavaScript files

Merge branch '3.0' into 3.1

Preparing the release 3.0.25

Revert "Preparing the release 3.1.15"

This reverts commit bd4d9de5fc28d17643c109fc87ae88f2d9265817.

Preparing the release 3.1.15

Incorrect encoding of conditions

Merge branch '3.0' into 3.1

Incorrect encoding of conditions

Merge branch '5.2'

Fix checking current value of radio button form fields

Merge branch '5.2'

Whitelist `parse_url` in enterprise mode

Limit media images' caption size to image width

Resolves #3228

Update composer dependencies

Merge pull request #3385 from WoltLab/amp-ext-img

Output external images with unknown dimensions in AMP pages

Output external images with unknown dimensions in AMP pages

The `layout="fill"` attribute allows to embed an `<amp-img>` with unknown
dimensions. This comes with the drawback that the image will stretch as
far as possible, not maintaining the aspect-ratio and taking up the whole
page.

To solve this a container with known dimensions needs to be put around it
and the `<img>` within needs to have the `object-fit: contain` property.
This property will contain the image within the container while maintaining
aspect ratio at the drawback that the empty space of the container will be
filled with void.

This container was selected to be a 16:9 aspect ratio, with a size of
384×216 Pixels (one fifth of Full HD in each dimension), hopefully
minimizing the void on common image formats.

The resulting AMP was checked against the AMP validator and passed.

Fixes #3264

Merge branch '5.2'

Merge branch '3.1' into 5.2

Fixed 'data too long for column username' issue

Fix leaking globals

Merge pull request #3383 from WoltLab/audio-attachment

Add audio player for embedded audio attachments

Add audio player for embedded audio attachments
Closes #3231

Merge pull request #3372 from WoltLab/video-attachment-rework

Improve layout for embedded videos in ugc

Merge branch 'master'

Merge pull request #3358 from WoltLab/user-profile-special-trophies

Indicate the existance of further trophies in special trophy list on …

Reverted changes to UserProfile::getSpecialTrophies()

Update com.woltlab.wcf/templates/user.tpl

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

Fix JavaScript error for ACL form fields in non-ajax forms

See 6cce1152a855f636ac22689cbf531f3928f97506

Merge branch '5.2'

Merge branch '5.2' of github.com:WoltLab/WCF into 5.2

Check for missing length of varchar columns

Fix updating decimal fields with default value (#3382)

MySQL stores the default value using the specified number of decimals so that when comparing the existing default value with the new default value, the same format should be used.

Merge pull request #3375 from WoltLab/notification-settings

New UI for the notification settings

Visually disable the email settings for suppressed notifications

Refresh the aria label when the tooltip value is updated