don't die()

Tobias Maier - February 5, 2007 - 11:16
Project:Pdfview
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:minor
Assigned:Unassigned
Status:closed
Description

I read that we should not use die() or exit() without calling module_invoke_all('exit');

you can see it at
drupal_goto() http://api.drupal.org/api/5/function/drupal_goto
views does this, too
http://drupal.org/node/97497

#1

Tobias Maier - February 5, 2007 - 12:47
Status:active» needs review

and here comes the patch

AttachmentSize
pdfview.image__2.patch 619 bytes

#2

Egon Bianchet - February 5, 2007 - 15:33
Status:needs review» fixed

Committed to HEAD, DRUPAL-5 and DRUPAL-4-7, thanks

#3

Anonymous - February 19, 2007 - 15:45
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.