[Bps-public-commit] r12958 - RT-BugTracker/html/Dist

ruz at bestpractical.com ruz at bestpractical.com
Fri Jun 6 06:26:29 EDT 2008


Author: ruz
Date: Fri Jun  6 06:26:28 2008
New Revision: 12958

Modified:
   RT-BugTracker/html/Dist/Display.html

Log:
* don't use abort as it hides footer and breaks validity of html

Modified: RT-BugTracker/html/Dist/Display.html
==============================================================================
--- RT-BugTracker/html/Dist/Display.html	(original)
+++ RT-BugTracker/html/Dist/Display.html	Fri Jun  6 06:26:28 2008
@@ -88,15 +88,11 @@
 
 my $QueueObj = RT::Queue->new($session{CurrentUser});
 $QueueObj->Load($Queue);
-
-if (not $QueueObj->Id) {
-    $m->comp('/Elements/ErrorNoDist', Queue => $Queue, %ARGS);
-    $m->abort();
-}
+return $m->comp('/Elements/ErrorNoDist', Queue => $Queue, %ARGS)
+    unless $QueueObj->id;
 
 if ($Status !~ /\w/) {
-    $m->comp('/Elements/Error', Why => q{no such status as '} . $Status . q{'});
-    $m->abort();
+    return $m->comp('/Elements/Error', Why => q{no such status as '} . $Status . q{'});
 }
 
 my $admincc = $QueueObj->AdminCc->UserMembersObj;



More information about the Bps-public-commit mailing list