Remove useless condition in SessionHandler::create()

`$session` is always `null` at this point since the previous commit.

Remove obsolete call to getSessionByUserID in SessionHandler::create()

This call will always return `null`, since the user always is a guest at that
point, passing a `null` to the database query selecting rows `WHERE userID = ?`.

Incidentally it took me a while to understand why this would not return
existing guest sessions, until I remembered that `NULL` does not equal anything
in SQL and instead `IS NULL` needs to be used. It's good that this slightly
misleading part of the code is gone.

Remove useless condition in SessionHandler::create()

This condition always was true since the removal of the legacy autologin in
commit 95953c7a4b18c70d569c4bafc9de2726a1727ef9.

Merge pull request #3580 from WoltLab/session-password-refactor

Add new password hashing framework

Add explicit namespace for used methods

Add 'Smf2' password algorithm

Add 'Smf1' password algorithm

Add 'CryptMD5' password algorithm

Add 'Phpfox3' password algorithm

Add 'Joomla3' password algorithm

Add 'Joomla2' password algorithm

Add 'Joomla1' password algorithm

Add 'Xf12' password algorithm

Add 'Xf1' password algorithm

Add 'Wcf2' password algorithm

Add 'Wcf1e' password algorithm

Add 'Wcf1' password algorithm

Add 'Wbb2' password algorithm

Add 'Phpass' password algorithm

Add 'Phpbb3' password algorithm

Add trait for Phpass algorithm

Add 'Mybb1' password algorithm

Add 'Ipb3' password algorithm

Add 'Ipb2' password algorithm

Add 'Vb5' password algorithm

Add 'Vb4' password algorithm

Add 'Vb3' password algorithm

Add 'Argon2' password algorithm

Rename IPasswordAlgorithm::needs_rehash() to needsRehash()

Deprecate methods related to DoubleBcrypt in PasswordUtil

Update UserEditor to make use of the new password checking framework

Update User::checkPassword() to make use of the new password algorithm framework

Co-authored-by: Joshua Rüsweg <ruesweg@woltlab.com>

Add PasswordAlgorithmManager

Add 'DoubleBcrypt' password algorithm

Add 'Invalid' password algorithm

Add 'Bcrypt' password algorithm

Add IPasswordAlgorithm

Remove obsolete imports

Merge branch 'master' into session-meta

Most importantly for the argon2 compatibility adjustments.

Merge pull request #3574 from WoltLab/session-remove-autologin

Remove legacy autologin functionality

Remove logic related to automated login in IUserAuthentication

`UsersOnlineList::isVisibleObject()` expects a `UserOnline` object

Improved the UX of the style color palette

The style compatibility notice is now context sensitive. The boxes for the color selection now show a "hand" cursor and use a distinct hover state to highlight them.

Closes #3576

Merge branch 'master' of github.com:WoltLab/WCF

Removed the `*_autoscaled` suffix for images

Merge branch '5.2'

Add missing language variables

Release 5.3.0 Beta 1

Merge pull request #3571 from WoltLab/argon2

Add support for argon2 to PasswordUtil

Defaulting the user password to `invalid:`

Modify the password column during the upgrade

Remove obsolete WCF.User.Login JavaScript

This JavaScript was required back when the login form allowed selecting whether
a login should be performed or whether a new account should be registered.

This select was removed in 5ea921192a4b9759c390ccf5c9378d714e3a2d62 (during
development of 3.0).

Remove checkbox for persistent login (“useCookies”)

Refactor Session Handling: Integration branch

Missing update of the version number

Updating minified JavaScript files

Removed debug code

Internal release 5.3.0 Beta 1

Updating minified JavaScript files

Merge branch '5.2'

Updating minified JavaScript files

New implementation for message preview for i18n editor instances

Incorrect handling of array indices for the list of active package servers

Cleaned up the default package servers

Apply the sort order for the list of users in the birthday list

Merge branch '5.2'

Increased the z-index of the reaction popover

This prevents the reactions from being overlapped by an adjacent tab menu on mobile devices.

Merge branch '5.2'

Do not block the installation of not yet installed projects

Fixes #3572

This commit rectifies the change in d70a577df3662d168612e40035a4237c19bed5b0

Restored the 'mark as read' button for notifications

Updating minified JavaScript files

Change wcf1_user.password to VARCHAR(255)

Add support for argon2 to PasswordUtil

Merge branch '5.2'

Merge branch '3.1' into 5.2

Prevent data being pasted twice in Chromium on Android

Merge branch '5.2'

Use the `selectionchange` event to suppress the selection of the quote tooltip

Unified appeareance of links in interactive dropdowns

Raise an exceptions if a phrase name contains leading or trailing whitespace

Merge branch '5.2'

Prevent the read access to devtools projects without a valid path

Fix property documentation

Remove unused imports

Fix ArticleCommentManager::isContentAuthor()

The comments belong to article contents, not the article itself.

Merge pull request #3565 from WoltLab/highlighter-guess

Extend automated guessing logic of highlighters

Merge branch '5.2'

Merge branch '3.1' into 5.2

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

Fix StyleAddForm::rebuildUploadFields()

Fixes #3569

Removed the unintended font size for collapsed messages headers

The font size change was never applied due to a bug in the previously used version of the SCSS compiler (nested include) which simply discarded those lines. Removing these two declarations restores the original layout.

Removed layout differences between moderation queue list and topic list

Removed obsolete white-space in layout

Merge branch 'master' of https://github.com/WoltLab/WCF

Removed temporary file name from error message

Use consistent hostname in description of mail_smtp_host

Merge pull request #3566 from WoltLab/prism-update

Update prism to 1.21.0

Add csharp to highlighter guessing

Update prism to 1.21.0

Extend automated guessing logic of highlighters