https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29033
--- Comment #10 from Tomás Cohen Arazi <tomasco...@gmail.com> --- (In reply to Marcel de Rooy from comment #9) > (In reply to Tomás Cohen Arazi from comment #8) > > (In reply to Marcel de Rooy from comment #7) > > > +sub multivalue_preference { > > > + my ( $self, $preference ) = @_; > > > + > > > + my $syspref = $self->preference($preference) // q{}; > > > + my $values = [ split qr{\|}, $syspref ]; > > > + > > > + return $values; > > > +} > > > > > > I agree that we need a parameter here to pass other separators. > > > Multi value, multi sepa :) > > > > Counter proposal: > > 1. Accept this as-is > > 2. Implement a nice UI to abstract users from the separator being used > > 3. Migrate sysprefs into using this > > Point 1: No ! :) > Point 2: Fine with me. > Point 3: Great. (Just gradually move away from sending the sepa param.) > > You need 5 mins to do the thing. We need 5 years for the rest ;) Well, it is always the same chicken-egg situation: one proposes something, for enhancing things, and waits for feedback before moving into changing all the things. And some people want you to do all the things to get convinced of the idea. It's a hard one. I'm fine with ad-hoc splitting code for now. But this is something we could do better, specially point 2. Plus, I don't think there are that many sysprefs that require this (many == more than 20). -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/