https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23166
--- Comment #57 from Jonathan Druart <jonathan.dru...@bugs.koha-community.org>
---
(In reply to Joonas Kylmälä from comment #38)
> There seems to be nothing wrong with the code, it is just that the QA tool
> doesn't add the "orders" object because this is not being executed from the
> real .pl script, so therefore the $set variable is undef. I think we could
> say this is false positive then, right? Or if we want to prevent the QA tool
> failing maybe such a check can be added:
>
> sub Scalar {
> my ( $self, $set, $method ) = @_;
> + return undef unless $set;
> $set = $set->$method;
> return $set;
> }
>
> What do you think? I think it is safe to ignore this here and we could go
> ahead with the current patches.
I've decided to fix it here, it should not hurt.
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
