From f29a243db30021e8d85818a3865122ab509148b0 Mon Sep 17 00:00:00 2001 From: "Bradley M. Kuhn" Date: Sun, 20 Dec 2015 17:02:44 -0800 Subject: [PATCH] Extract $requestId from getRequest() return. of course, getRequest() returns a hash so we must properly extract the value we want. This fix causes this test to now pass: ok 73 - fulfillRequest: databse entry from successful return is correct --- Supporters/lib/Supporters.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Supporters/lib/Supporters.pm b/Supporters/lib/Supporters.pm index a1d9737..dcd6294 100644 --- a/Supporters/lib/Supporters.pm +++ b/Supporters/lib/Supporters.pm @@ -706,7 +706,9 @@ sub fulfillRequest($$) { die "fulfillRequest: undefined who" unless defined $params->{who}; die "fulfillRequest: undefined requestType" unless defined $params->{requestType}; - my $requestId = $self->getRequest($supporterId, $params->{requestType}); + my $req = $self->getRequest($supporterId, $params->{requestType}); + return undef if not defined $req; + my $requestId = $req->{requestId}; return undef if not defined $requestId; my $fulfillLookupSql = "SELECT id, request_id FROM fulfillment WHERE request_id = " .