https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34532 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |victor@tuxayo.net Status|Signed Off |Failed QA --- Comment #6 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Thanks Lucas, more extract variable refactoring :) Thanks Marcel to also look at this, I started to look at this yesterday and wondered if there was a reason to have the $llx & $lly default values set in the method instead of the constructor: https://git.koha-community.org/Koha-community/Koha/src/commit/e26d7c783eb8c7... Currently there are likely the same warns in the other functions if llx & lly are undef. And now I notice it's actually done twice, once in each method, so there is duplication. And one occurrence is `my $lly = $self->{'llx'} || 0;` which very likely should be $self->{'lly'} -- You are receiving this mail because: You are watching all bug changes.