https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22849
Katrin Fischer <katrin.fisc...@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #89577|0 |1 is obsolete| | --- Comment #4 from Katrin Fischer <katrin.fisc...@bsz-bw.de> --- Created attachment 89689 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=89689&action=edit Bug 22849: Do not share data (Mana) without agreement A couple of things wrong in serials/subscription-add.pl 1. 447 if ( defined( $query->param('mana_id') ) ) { It's always defined, you wanted to test with "" 2. 375 if ( (C4::Context->preference('Mana')) and ( grep { $_ eq "subscription" } split(/,/, C4::Context->preference('AutoShareWithMana'))) ){ Mana == 2 == let me think about it => I am not expecting to reach Koha::SharedContent in that case So we are sharing data whereas the library did not decide yet! Test plan: You need to be familiar you Mana to test this patch. JD: I did not test this patch Signed-off-by: Liz Rea <wizzy...@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer...@web.de> -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/