Dblog event page for non-existing event returns 200, should be 404

Fixed coding standards and re-rolled. Tested manually. RTBC +1.

Thanks for working on this. I see the opportunity to simplify the nesting of the eventDetails method if we check first if the log record exists and throw the exception early. The else there is really far away from the if so is hard to read. Also this approach simplifies the entire method because removes a lot of indentation.

patch at #6 look good to me and it is working properly.

+++ b/core/modules/dblog/tests/src/Functional/DbLogTest.php
@@ -136,6 +136,24 @@ public function testLogEventPage() {
+
+    // Random (not existing) event ID.
+    $connection = Database::getConnection();
+    $wid = $connection->query('SELECT MAX(wid) FROM {watchdog}')->fetchField();
+    $wid += rand(100, 999);
+

I think this should be non-random - can we just make it 999999 or something?

Thanks @Krzysztof Domański

So the issue is fixed then.

Committed 08582f7 and pushed to 9.0.x. Backported to 8.9.x Thanks!