https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29033

--- Comment #9 from Marcel de Rooy <m.de.r...@rijksmuseum.nl> ---
(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 ;)

-- 
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/

Reply via email to