When paying a fine (in 19.05.x) and lower branchcode is not properly logged. I believe this is fixed in 19.11.x by bz23321 To Recreate: 1. Have some fines. Pay them using the Pay Amount or Pay Selected buttons. 2. SELECT branchcode FROM accountlines where borrowernumber = '{borrowernumber}'; 3. See that branchcode is NULL
Created attachment 98734 [details] [review] Bug 24637: Add brancode in all pay circumstances To TEST: 1. Have some fines. Pay them using the Pay Amount or Pay Selected buttons. 2. SELECT branchcode FROM accountlines where borrowernumber = '{borrowernumber}'; 3. See that branchcode is NULL 4. Apply patch and restart_all 5. Make a partial payment using both the Pay select and Pay amount buttons. 6. SELECT branchcode FROM accountlines where borrowernumber = '{borrowernumber}'; 7. Now you should see the branchcodes in the table and no NULL values.
Lucas, I am getting this error: Could not compile /kohadevbox/koha/members/paycollect.pl: Global symbol "@acc" requires explicit package name (did you forget to declare "my @acc"?) at /kohadevbox/koha/members/paycollect.pl line 214. Global symbol "$select" requires explicit package name (did you forget to declare "my $select"?) at /kohadevbox/koha/members/paycollect.pl line 214. Global symbol "$note" requires explicit package name (did you forget to declare "my $note"?) at /kohadevbox/koha/members/paycollect.pl line 215. Global symbol "@lines" requires explicit package name (did you forget to declare "my @lines"?) at /kohadevbox/koha/members/paycollect.pl line 217. Global symbol "$branch" requires explicit package name (did you forget to declare "my $branch"?) at /kohadevbox/koha/members/paycollect.pl line 234. Global symbol "@lines" requires explicit package name (did you forget to declare "my @lines"?) at /kohadevbox/koha/members/paycollect.pl line 235. Global symbol "$note" requires explicit package name (did you forget to declare "my $note"?) at /kohadevbox/koha/members/paycollect.pl line 236. syntax error at /kohadevbox/koha/members/paycollect.pl line 241, near ">>" Global symbol "$branch" requires explicit package name (did you forget to declare "my $branch"?) at /kohadevbox/koha/members/paycollect.pl line 251. syntax error at /kohadevbox/koha/members/paycollect.pl line 271, near "if" /kohadevbox/koha/members/paycollect.pl has too many errors. at /usr/share/perl5/CGI/Compile.pm line 132
Kelly, I think this is because this patch is intended for 19.05.x, and maybe you're attempting to apply it to master?
Created attachment 99279 [details] [review] Bug 24637: Add brancode in all pay circumstances Signed-off-by: Benjamin Daeuber <bdaeuber@fargolibrary.org>
Created attachment 99285 [details] [review] Bug 24637: Add brancode in all pay circumstances To TEST: 1. Have some fines. Pay them using the Pay Amount or Pay Selected buttons. 2. SELECT branchcode FROM accountlines where borrowernumber = '{borrowernumber}'; 3. See that branchcode is NULL 4. Apply patch and restart_all 5. Make a partial payment using both the Pay select and Pay amount buttons. 6. SELECT branchcode FROM accountlines where borrowernumber = '{borrowernumber}'; 7. Now you should see the branchcodes in the table and no NULL values. Signed-off-by: Benjamin Daeuber <bdaeuber@fargolibrary.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Works as expected.. Passing QA
backported to 19.05.x for 19.05.08