[Koha-patches] [PATCH] Bug 5418: Rev-4 patch new itemBarcodeInputFilter for libsuite8 style barcodes

Koustubha Kale kmkale at anantcorp.com
Mon Dec 6 11:33:48 CET 2010


Thanks for looking into this patch.

>
> But:
>
> Your patch modify installer/data/mysql/updatedatabase.pl. It souldn't.
> You're modifying an old DB update which will never been called again on
> new installation.

Suggested by chris while he was guiding me on irc on how to properly
implement a new syspref option. But I'll be happy to remove any
changes to installer/data/mysql/updatedatabase.pl if same is not
required..


>
> You also modify itemBarcodeInputFilter syspref in
> data/mysql/en/mandatory/sysprefs.sql. I'm not sure it's required
> anymore. You add your new syspref in circulation.pref. It's enough.
> Others can correct me if I'm wrong.

The reason to do this chris gave was, this file is translatable while
a db entry is not. Be happy to get rid of the changes if its not
required.

Should I remove changes to these two files, retest and resubmit?


Regards,
Koustubha Kale
Anant Corporation

Contact Details :
Address  : 103, Armaan Residency, R. W Sawant Road, Nr. Golden Dyes
Naka, Thane (w),
                Maharashtra, India, Pin : 400601.
TeleFax  : +91-22-21720108, +91-22-21720109
Mobile     : +919820715876
Website  : http://www.anantcorp.com
Blog : http://www.anantcorp.com/blog/?author=2


More information about the Koha-patches mailing list