https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37429 Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |emily.lamancusa@montgomeryc | |ountymd.gov --- Comment #4 from Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> --- (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.