From d62cf842e4b6b27cbe9f8150276f9df61d7e6e4f Mon Sep 17 00:00:00 2001 From: Tatjana Dubrovica Date: Tue, 24 Feb 2026 01:08:35 +0200 Subject: [PATCH] Remove redundant empty-result check in talksByState MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The if($st->rows < 1) block was unnecessary — when no talks match, the while loop simply doesn't execute and the final render correctly returns an empty videos list. --- lib/SReview/Web/Controller/Event.pm | 3 --- 1 file changed, 3 deletions(-) diff --git a/lib/SReview/Web/Controller/Event.pm b/lib/SReview/Web/Controller/Event.pm index 688e774..fc6814d 100644 --- a/lib/SReview/Web/Controller/Event.pm +++ b/lib/SReview/Web/Controller/Event.pm @@ -130,9 +130,6 @@ sub talksByState { $st = $c->dbh->prepare("SELECT talks.id, title, subtitle, description, starttime, starttime::date AS date, to_char(starttime, 'yyyy') AS year, endtime, rooms.name as room, rooms.outputname as rooms_output, upstreamid, events.name as event, slug FROM talks JOIN rooms on talks.room = rooms.id JOIN events on talks.event = events.id WHERE state=? AND event=?"); $st->execute($state, $eventId); my $speakers = $c->dbh->prepare("SELECT name FROM speakers JOIN speakers_talks ON speakers.id = speakers_talks.speaker WHERE speakers_talks.talk = ?"); - if($st->rows < 1) { - $c->render(openapi => $rv); - } my $mt = Mojo::Template->new; $mt->vars(1); while(my $row = $st->fetchrow_hashref()) {