Merge pull request #3744 from Krymonota/patch-18

Add missing JSDoc to `EmailSmtpTest`

Add missing JSDoc to `EmailSmtpTest`

Updating minified JavaScript files

Workaround for a Chromium bug on Windows

Mutable variable access inside a callback

Fix ACL buttons when editing media files successively

Replaces 3a7740a3704339c79d08fd2f5094fa1a1973318b by backporting and using parts of 6ee58cd2865980cd7fa16a1f3ebc660131fc7fd6.

Revert "Fix ACL buttons when editing media files successively"

This reverts commit 3a7740a3704339c79d08fd2f5094fa1a1973318b.

Fix ACL buttons when editing media files successively

Fix BBCodeHandler::setDisallowedBBCodes(...) PHPDoc (#3722)

Obsolete exclusion of the old ACP catpcha

This route does not exist anymore since WoltLab Suite 3.0.

Block search keyword list if permissions for the search function are denied

Closes #3681

Fix de.xml

This fixes up commit 70746ece9789d3b2256495a788ae4875d01e95bc.

Fix typo in de.xml

Fix GDPR export if invalid user id is given

… and stop using deprecated method `UserProfile::getUserProfile()`.

Fix class name in UserRegistrationUtil::__construct() PHPDoc (#3652)

Ignore top-level node_modules/

Use a literal string to avoid escaping

See #3595

Removed obsolete code

There is no article counter for users (wcf1_user.articles) in version 3.1.

Merge pull request #3585 from WoltLab/overdue-cronjob

Improved the wording of the cronjob error message

Improved readability

Co-authored-by: Matthias Schmidt <gravatronics@live.com>

Default value blocked saving of empty date fields

Incorrect parameter order

Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>

Improved the wording of the cronjob error message

Incorrect link to the user edit form

Merge pull request #3577 from WoltLab/filereader-range

Disable 'Range' handling for 0-byte files in FileReader

Disable 'Range' handling for 0-byte files in FileReader

see https://community.woltlab.com/thread/285960-download-eines-leeren-textdokuments-gibt-http-error-416-aus/

Prevent data being pasted twice in Chromium on Android

Missing link target causes buttons to be unresponsive in some touch browers

Prevent Android from collapsing the keyboard on [Enter]

Somehow the linkify redactor plugin causes Android to collapse the on-screen
keyboard if a new line is entered.

This plugin was already removed for 5.2 in commit 19af26fc22bc31d699db6021dee2eda1d23b8a14,
so let's effectively disable it for 3.1 as well.

see https://community.woltlab.com/thread/285573-wsc-3-1-17-smartphone-tastatur-schlie%C3%9Ft-sich-unbeabsichtigt/?postID=1816848#post1816848

Incorrect calculation of the last 2 digits of the year

Fixes #3547

Release 3.1.17

Updating minified JavaScript files

Remove fullscreen mode in template diff
The fullscreen mode does not have any advantage at this point because the diff is also displayed with a height of 700px (just like without fullscreen mode).

Improved visuals of the fullscreen template comparison

Fixes #3473

Fix trophy icon size in badges on groupedUserTrophyList

Adjusted appearance of the "sign in with google" button

Reopened dialogs were hidden behind new dialogs

Merge pull request #3500 from Krymonota/patch-12

Add missing phrases to English 3rd party register language items

Prevent username overflow in the message sidebar

Prevent changes to the showOrder setting of system boxes during an update

Prevent changes to the visibility settings of system boxes during an update/upgrade

Add missing phrases to English 3rd party register language items

Merge pull request #3466 from Sir-Will/patch-1

Fixes exception when not using primary PayPal email

Fix PHP 5.5 compatibility

see b044815dd9b6509fc44219684d7076cd28a80aa6
see #3480

Verify if business argument is set in PayPal callback

Merge pull request #3462 from SoftCreatR/patch-14

Add detection for Chromium based Edge browser

Do not decrement wcf1_user.articles when deleting unpublished articles

see b044815dd9b6509fc44219684d7076cd28a80aa6
see #3480

Update wcf1_user.articles when deleting articles

Fixes #3480

Added detection for Chromium based Edge browser

Fix updating `requireObjectID` for existing pages

Include PayPal business email in exception

Fixes exception when not using primary PayPal email

PayPal allows adding multiple emails, if you are using one of the alternative emails instead of the primary email then WoltLab will throw the exception.
The `receiver_email` is always the primary email while `business` is the secondary in the IPN.

Moved the remark on the cleanup script

Merge pull request #3450 from WoltLab/cms-embedded-object-remove

Removed embedded object references when deleting articles / pages

Add update_com.woltlab.wcf_embeddedObjectOrphans.php

Add note about deletion of orphaned embedded objects

see #3429

Remove embedded object references when deleting pages

see #3429

Remove embedded object references when deleting articles

see #3429

Fix PHP 5.5 compatibility

Fix copy and paste bug

see 3cd8f5f63c1327c1534a4e24908b3e49d4074871

Properly update page search index when editing

see #3430

Properly update article search index when editing

see #3430

Improved detection of the links to twitch's clips

Release 3.1.16

Prevent Chromium browser form injecting base64 encoded images

Merge pull request #3439 from WoltLab/cronjob-fix2

Fix calculation of nextExec of cronjobs

Fix calculation of nextExec of cronjobs

To properly calculate nextExec we must not specify the TIME_NOW parameter,
because if the cronjob is run on the scheduled time the nextExec() will
return the current time.

Not passing the TIME_NOW parameter adds at least 120 seconds of delay to
prevent this issue.

This bug was introduced in commit 485f8e1888824b862823de983e019afcb9bca7ce.
When moving the update of the execution time the calls were not correctly
moved and the explicit passing of TIME_NOW was added.

Merge pull request #3428 from WoltLab/url-is

Fix Url::is()

Merge pull request #3432 from WoltLab/linked-blocked-img

Fix markup for linked, blocked images in UGC

Fix markup for linked, blocked images in UGC

Fixes #3384

Fix Url::is()

`parse_url()` cannot be used to validate an URL, because it will accept
roughly everything. In fact this is documented in the parse_url() docs:

> This function is not meant to validate the given URL, it only breaks
> it up into the above listed parts. Partial URLs are also accepted,
> parse_url() tries its best to parse them correctly.

Fixes #3391

Merge pull request #3418 from WoltLab/cronjob-fix

Fix detection of stuck cronjobs

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

Hide empty tag box on tagged page

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

Fixed 'data too long for column username' issue

Article image did not always have the right content language

Fixed 'Prepared statement contains too many placeholders' error in bulk processing

Message with the content "0" led to empty entries in recent activities

Merge pull request #3368 from WoltLab/twitch-bbcode-fix

Fixed broken twitch bbcode

Applied code suggestions

Fixed broken twitch bbcode

Embedding now requires the new 'parent' parameter

Fix sitemaps added multiple times in index file

Fixed bug with multiple comment widgets on a single page

Typo

Template event for reported user profiles

Merge branch '3.0' into 3.1

Merge pull request #3342 from Krymonota/remove-obsolete-user-signature-properties

Remove obsolete signature properties documentation

Remove obsolete signature properties documentation

See bc7adcda6be815367889eaa8cdecf851119009e4 and 291e9fe55c71023bdf3f18565be1c3ae098f77c6

Improved notifications for expiring paid subscriptions