@erik , you can safely accept without testing, because I've removed the code, that was causing the test to fail. Indeed code contained an error, but it's no longer needed.

1. removed Session Key: ... line in Debugger Report, because session key is already visible in the SQL query made before it
2. [bugfix] new sid uniqueness checking during generation was made using SID itself and not it's hash

System log errors exists, but, seems, not related to the current patch. Equal errors count with patch and without patch.

All tests passed, excepting "confirm, that `Session Key: ..." line shows different value, than before (proves, that active sessions in the old format were upgraded and remain active)"

Test failed on Part 1 - Testing Upgrade (for In-Portal 5.2.x installation), in Web Browser open the /core/install.php URL

CS fixes

Sent changes from the wrong changelist

Tested by using PHPStorm+XDebug with breakpoint in the SessionStorage::RemoveFromData method. Before patch applying SQL, that removes the priority_deleted session variable was executed. After patch applying SQL, that removes the priority_deleted session variable was not executed, because SessionSet property is set to false.

Removed excessive session update statement, that happens after it's creation.