[Koha] [Discussion tech] Generalise the use Template Toolkit plugins

Paul Poulain paul.poulain at biblibre.com
Wed Feb 22 02:22:04 NZDT 2012

Le 13/02/2012 08:12, Chris Cormack a écrit :
> On 13 February 2012 20:11, Marcel de Rooy <M.de.Rooy at rijksmuseum.nl> wrote:
>> I would say here that a general rule is hard to make. Even disapproving a patch only for the reason of not using a plugin is "discussionable".
> Current Koha does a lot of things in a lot of ways. Standardizing with
routines and plugins is a continuous educational project..
> I agree completely.
I also agree completely too: Koha does a lot of things in a lot of ways.
Maybe we don't conclude the same things: my conclusion is that we must
coordinate & organize things as much as possible.

So, my preference would go to:
* it's too hard to make a general rule for display plugins. There would
be too many possibilities to imagine.
* for bug 4078, we should ask for a plugin though: looking at the code,
it introduces dozens of:
+    $something  = PlaceCurrencySymbol($something);

Those will add complexity to the code, if one day $something must be
reused after the PlaceCurrencySymbol has been applied, there will be
side effects.

There are *bad* things like:
+		$$item{field} = PlaceCurrencySymbol($$item{field}) if ($$item{field}
=~ /^\d+\.\d\d$/);
in the patch : they'll display a currency just in case something is
numeric with 2 decimals. That can be the case of a dewey code,
librarians won't like seeing a $ or € on their dewey ;-)

I'll ask for an alternate patch on the bug itself, it's really a nice
feature but Aleksa probably missed the T::T plugin (great) feature !

Expert en Logiciels Libres pour l'info-doc
Tel : (33) 4 91 81 35 08

More information about the Koha mailing list