confess is better here than die.

Maybe I should use it throughout?
This commit is contained in:
Bradley M. Kuhn 2015-12-30 16:09:59 -08:00
parent 724cb77605
commit 195732b6b0

View file

@ -1077,7 +1077,7 @@ Returns: scalar boolean, which is true iff. the $id is valid and already in the
sub _verifyId($$) { sub _verifyId($$) {
my($self, $id) = @_; my($self, $id) = @_;
die "_verifyId() called with a non-numeric id" unless defined $id and looks_like_number($id); confess "_verifyId(): called with a non-numeric id" unless defined $id and looks_like_number($id);
my $val = $self->dbh()->selectall_hashref("SELECT id FROM donor WHERE id = " . my $val = $self->dbh()->selectall_hashref("SELECT id FROM donor WHERE id = " .
$self->dbh->quote($id, 'SQL_INTEGER'), 'id'); $self->dbh->quote($id, 'SQL_INTEGER'), 'id');