Happy new year everybody!
Last month I asked a question regarding pdf templates for basketgroups, but nobody answered.
So I reformulate :-)

Pdf templates are untranslatable, and the 2 defaut layouts is not convenient at all for our library. So we want to define our own template. I have done the job, but I can not activate it in system preferences without removing some lines in /acqui/basketgroup.pl. So, do you think it would be a good idea to give libraries the ability to define their own templates? And what would be the best solution for that : removing these lines in /acqui/basketgroup.pl, or creating a more flexible security (I don't now how...)?


Regards,
M. Saby
Rennes 2 university


Mathieu Saby a écrit :
Hello

Our library has created a new templates for exporting basketgroups in pdf (translated in french, and showing fields more usefull for us in the tables than "layout2pages" and "layout3pages"). Of course, we could name this template "layout3pages", and replace the standard template. But I think it would be "cleaner" to add this give this new template a name of his own, so it won't be altered when our Koha is updated.

The problem is some code in /acqui/basketgroup.pl prevents us to do that. The only 2 valid templates for exporting basketgroups in pdf are be "layout2pages" and "layout3pages". And we want to edit perl files as less as possible. http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=acqui/basketgroup.pl;h=6b7e5bf192a7b5cfa6761dc750bfc9fe5904bf60;hb=HEAD 188 if ($pdfformat eq 'pdfformat::layout3pages' || $pdfformat eq 'pdfformat::layout2pages'){
189         eval {
190         eval "require $pdfformat";
191             import $pdfformat;
192         };
193         if ($@){
194         }
195     }
196     else {
197 print $input->header; 198 print $input->start_html; # FIXME Should do a nicer page
199         print "<h1>Invalid PDF Format set</h1>";
200 print "Please go to the systempreferences and set a valid pdfformat";
201         exit;
202     }

This code was added by this Chris Cormack's commit : "Bug 6679 Fix scripts in admin & acqui to pass Perl::Critic" http://git.koha-community.org/gitweb/?p=koha.git;a=commit;h=7cdea5de355e853f25300821cc641672443177de added

So, here is my question :
Is this "security" in code really necessary ? And if it is, is there a way to make it more flexible ?


Regards,

M. Saby
Rennes 2 University



--
Mathieu Saby
Service d'Informatique Documentaire
Service Commun de la Documentation
Université Rennes 2
Téléphone : 02 99 14 12 65
Courriel : [email protected]

_______________________________________________
Koha-devel mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to