https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23112 --- Comment #48 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Andrew Isherwood from comment #46)
::: Koha/Illrequest.pm @@ +1211,5 @@
+ # For some reason, AddIssue requires an unblessed Patron + $issue_args[0] = $patron->unblessed; + my $issue = C4::Circulation::AddIssue(@issue_args); + + if ($issue && %{$issue}) {
AddIssue return a Koha::Checkout object. The test will work but I think it's confusing.
Do you mean the variable name $issue is confusing? If so, I thought it made sense since the function returning it is called AddIssue and, internally, AddIssue also calls the returned value $issue.
You should only test for $issue, the "&& %$issue" does not make sense to me. -- You are receiving this mail because: You are watching all bug changes.