https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37429
Emily Lamancusa <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |emily.lamancusa@montgomeryc | |ountymd.gov --- Comment #4 from Emily Lamancusa <[email protected]> --- (In reply to Andreas Roussos from comment #3) > Interestingly, I cannot reproduce this if I disable and then > stop Plack (followed by a restart of memcached and apache2). > > The following quick and dirty patch seems to fix things > (even when Plack and memcached are enabled): > > diff --git a/cataloguing/addbiblio.pl b/cataloguing/addbiblio.pl > index 99c20580b5..754aaf175a 100755 > --- a/cataloguing/addbiblio.pl > +++ b/cataloguing/addbiblio.pl > @@ -518,6 +518,7 @@ my $fa_duedatespec = $input->param('duedatespec'); > > my $userflags = 'edit_catalogue'; > > +$changed_framework = 0; > if ( $op eq 'cud-change-framework' ) { > $op = $input->param('original_op'); > $changed_framework = 1; > > Not sure about the quality of the above solution, though? The bigger issue is that $changed_framework (as well as others) shouldn't really be global variables in the first place. I don't quite understand Perl's scoping rules well enough to be able to say how big of a refactor it would be to fix that, though. In the meantime, I'd say your patch is worth doing to fix the major bug. We already do that with $is_a_modif, and it's a simple change. I'll file a follow-up bug to address the underlying issue. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. _______________________________________________ Koha-bugs mailing list [email protected] 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/
