http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10855
--- Comment #82 from Jonathan Druart <[email protected]> --- (In reply to Jacek Ablewicz from comment #78) Hello Jacek, I squashed all fixes into one. > 79c79,80 > < if ( defined ( my $filter_value = $query->param('additional_field_' . > $field->{id} . '_filter') ) ) { > --- > > my $filter_value = $query->param('additional_field_' . $field->{id} . > > '_filter'); > > if ( defined( $filter_value ) && $filter_value ne '' ) { > > searching works again like it should. I'm not sure if that may be a proper > fix, but it corrected this particular problem. Yep, it is fine. It is included in the squashed patch. (In reply to Jacek Ablewicz from comment #79) > It was enough, no need to extend additional_fields table.. Also, in case > extra fields are really needed, one can always add additional fields to > additional_fields using Koha::AdditionalFields ;). It would be fun but I discourage you to do that :) > Yet subscription-add.* script & template are (?) still using field name for > <input name=""> etc. isntead of field id - that's where I encountered > aftermentioned problems. Another possibility is that while applying patches > I simply may have missed one (or two).. Yes, good catch. Fixed in the last patch too. Thank you very much for testing! -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] 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/
